On 18/01/2014 22:02, Jeremy Boynes wrote: > On Jan 18, 2014, at 11:33 AM, [email protected] 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
Thanks. I think I see what is going on here. Working on a patch now... Mark --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
