[ http://issues.apache.org/jira/browse/JEXL-18?page=comments#action_12426389 ] Kohsuke Kawaguchi commented on JEXL-18: ---------------------------------------
D'oh! This is supposed to be filed against Jelly, not Jexl. But I don't seem to be able to reassign it, or even just to close it. If whoever with enough karma can reassign this to Jelly or just close it, that would be appreciated. > Whitespace normalization in Jelly is wrong > ------------------------------------------ > > Key: JEXL-18 > URL: http://issues.apache.org/jira/browse/JEXL-18 > Project: Commons JEXL > Issue Type: Bug > Reporter: Kohsuke Kawaguchi > > In XML, the convention is to only remove "ignorable whitespaces". Ignorable > whitespaces are defined as a text block that entirely consist of whitespaces, > surrounded by tags. So the following text are ignorable whitespaces: > <foo> <bar> // text between two start tags > <foo> </foo> // text between start and end tag > (Technically speaking, they are ignorable only when DTD says so, but those > are nevertheless people usually consider them pseudo-ignorable even if > there's no DTD.) > But the following text are never ignorable whitespaces: > <foo> x <bar> > <foo> y </foo> > Jelly agressively trims off whitespaces, even if they are not ignorable. So > in the above cases, it becomes: > <foo>x<bar> > <foo>y</foo> > Such mode of whitespace normalization is not seen anywhere else, and IMHO it > is confusing. > It is sometimes even harmful, for example because the following text: > <p> some text <b>here</b></p> > ... becomes: > <p>some text<b>here</b></p> > and you end up losing the significant space character between 'text' and > 'here'. As an example of proper whitespace stripping, consult XSLT > (http://www.w3.org/TR/1999/REC-xslt-19991116#strip) > I do realize that one can control the whitespace stripping behavior by using > TagSupport.setEscapeText(boolean), but nevertheless I thought I would file a > bug to see if the development team is willing to consider this change. I also > do realize that this has some non-trivial compatibility implications. > If you are willing to fix this, I have already patched Jelly locally, so I'm > happy to contribute a patch. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
