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
signature.asc
Description: Message signed with OpenPGP using GPGMail