Re: trimSpaces removing whitespace from html

[Mark Thomas <ma...@apache.org>]

On 29/05/17 09:43, Mark Thomas wrote:
> On 29/05/17 09:09, David Kavanagh wrote:
>> On 26 May 2017 at 22:33, Christopher Schultz <ch...@christopherschultz.net>
>> wrote:
>>
>> Mark,
>>
>> On 5/26/17 6:59 AM, Mark Thomas wrote:
> 
> <snip/>
> 
>>>>> This is a potential use case for a recently closed (as WONTFIX)
>>>>> enhancement request:
>>>>>
>>>>> https://bz.apache.org/bugzilla/show_bug.cgi?id=45931
>>>>>
>>>>> Whether this use case is sufficient to justify re-opening that
>>>>> enhancement request, reviewing the patch and updating it for 9.0.x
>>>>> is TBD. How realistic is it for you to apply the fix (using ${'
>>>>> '}) described above?
>>
>> Would it be reasonable to change trimSpaces so that spaces within
>> element attributes would be normalized to a single space, and others
>> would be removed? I'm not sure how aware the parser is of the
>> surrounding document structure.
>>
>> -chris
> 
> <snip/>
> 
>> Hey guys,
>>
>> Thanks again for the help. For the Tomcat setup we currently have, the
>> first link sent my Mark is a good fix for us.
>> The other option Mark posted that would require modifying the .jsp files is
>> unfortunately not realistic because we currently use thousands of .jsps.
>> The possible fix that Christopher suggested in the last message would be
>> the perfect solution.
> 
> BZ 45931 it is then. Please re-open that enhancement request and link to
> this thread as the use case that justifies it. If you wanted to take a
> look at porting the patch as well, that would be great.

This has now been fixed for 9.0.x and will be included in 9.0.0.M22 onwards.

Mark

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tomcat.apache.org
For additional commands, e-mail: users-help@tomcat.apache.org