[jboss-jira] [JBoss JIRA] Updated: (JBWEB-144) JSP EL ternary operator requires space before colon

[Mike Millson (JIRA)]

     [ https://jira.jboss.org/jira/browse/JBWEB-144?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mike Millson updated JBWEB-144:
-------------------------------

    Attachment: jbweb-144.patch


Proposed CP branch patch from this chain of patches:
1) svn diff -r 562806:562807 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 562807.patch
2) svn diff -r 578465:578466 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 578466.patch
3) svn diff -r 578609:578610 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 578610.patch
4) svn diff -r 607333:607334 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 607334.patch
5) svn diff -r 649194:649195 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 649195.patch
6) svn diff -r 658469:658470 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 658470.patch
7) svn diff -r 677727:677728 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 677728.patch
8) svn diff -r 677728:677729 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 677729.patch
9) svn diff -r 685698:685699 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 685699.patch
10) svn diff -r 685700:685701 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 685701.patch
11) svn diff -r 685707:685708 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 685708.patch
12) svn diff -r 698886:698887 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 698887.patch
13) svn diff -r 698912:698913 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 698913.patch
14) svn diff -r 698917:698918 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 698918.patch
15) svn diff -r 714255:714256 http://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk/ > 714256.patch

Would it be better, given the extent of changes, to rebase some part of the code (e.g. org.apache.el) to 6.0.19?

> JSP EL ternary operator requires space before colon
> ---------------------------------------------------
>
>                 Key: JBWEB-144
>                 URL: https://jira.jboss.org/jira/browse/JBWEB-144
>             Project: JBoss Web
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: Tomcat
>    Affects Versions: JBossWeb-2.0.0.GA_CP09
>            Reporter: Mike Millson
>            Assignee: Remy Maucherat
>         Attachments: jbweb-144.patch, test.jsp
>
>
> JSP EL that worked on JBoss AS 4.0.5.GA does not work on EAP. For example, the following does not work:
> <c:set var="truevalue" value="true" />
> <c:set var="falsevalue" value="false"/>
> <c:out value="${not empty param.yes?truevalue:falsevalue}"/>
> It throws this error:
> 2009-06-11 10:40:57,487 ERROR [org.apache.catalina.core.ContainerBase] Servlet.service() for servlet jsp threw exception
> org.apache.jasper.JasperException: /jsp/case305721.jsp(27,3) "${not empty param.yes?truevalue:falsevalue}" contains invalid expression(s): javax.el.ELException: Error Parsing: ${not empty param.yes?truevalue:falsevalue}
>        at org.apache.jasper.compiler.DefaultErrorHandler.jspError(DefaultErrorHandler.java:40)
> But this works with a space added before the colon:
> <c:out value="${not empty param.yes?truevalue :falsevalue}"/>
> This is a bug[1] that was originally fixed in Tomcat 6.0.18[2], but was subsequently found to cause a regression[3], so was re-implemented in 6.0.19.
> [1]http://issues.apache.org/bugzilla/show_bug.cgi?id=42565
> [2]http://tomcat.apache.org/tomcat-6.0-doc/changelog.html
> [3]http://issues.apache.org/bugzilla/show_bug.cgi?id=45511

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira