On Jan 18, 2014, at 11:33 AM, ma...@apache.org wrote:

> Author: markt
> Date: Sat Jan 18 19:33:19 2014
> New Revision: 1559397
> 
> URL: http://svn.apache.org/r1559397
> Log:
> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=56029 a regression in 
> the fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=55198
> Remove the code to skip whitespace from the 'simple'EL parser. It is far 
> simpler to retain the whitespace than it is to teach this simple parser 
> parser when to insert it. This also has the benefit that the EL should remain 
> unchanged.
> A a number of test cases for BZ 56029
> 
> Modified:
>    tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java
>    tomcat/trunk/test/org/apache/jasper/compiler/TestELParser.java

This broke TestELInJSP for me:

    [junit] 18-Jan-2014 13:59:38.635 SEVERE [http-nio-127.0.0.1-auto-7-exec-1] 
org.apache.catalina.core.StandardWrapperValve.invoke Servlet.service() for 
servlet [jsp] in context with path [/test] threw exception [/bug46596.jsp 
(line: 22, column: 4) "${fn:trim
    [junit]                               ('{OK}')}" contains invalid 
expression(s): javax.el.ELException: Function 'fn:trim' not found] with root 
cause
    [junit]  org.apache.jasper.JasperException: /bug46596.jsp (line: 22, 
column: 4) "${fn:trim
    [junit]                               ('{OK}')}" contains invalid 
expression(s): javax.el.ELException: Function 'fn:trim' not found

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

Reply via email to