[ 
https://issues.apache.org/jira/browse/EL-8?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12521383
 ] 

Henri Yandell commented on EL-8:
--------------------------------

Ack. I hadn't realized that function-binding.txt was a patch. Brain was 
completely turned off.

Guess I need to get my head around that. Given its size - can I get you to 
comment that it's intended to be licensed to the ASF?

> [el] Implementation of parseExpression does not implement spec
> --------------------------------------------------------------
>
>                 Key: EL-8
>                 URL: https://issues.apache.org/jira/browse/EL-8
>             Project: Commons EL
>          Issue Type: Bug
>    Affects Versions: 1.0
>         Environment: Operating System: All
> Platform: All
>            Reporter: Jamie Taylor
>             Fix For: 1.1
>
>         Attachments: EL-8.patch, function-binding.txt, 
> TestFunctionBinding.java, TestFunctionBinding.java
>
>
> The implementation of parseExpression in the EL expression evaluator does not 
> conform to the JSP 2.0 specification.  The javadoc for the fMapper argument 
> states that 
> "The ExpressionEvaluatormustnotholdontotheFunctionMapperreferenceafter 
> returning fromparseExpression(). TheExpression object returned must invoke 
> the same functions regardless of whether the mappings in the 
> providedFunctionMapper instance change between calling 
> ExpressionEvaluator.parseExpression() andExpression.evaluate()."
> The test case which I will attach clearly shows that this is not the case.
> Why hasn't anybody noticed this before, you may ask?  I suspect that the 
> current implementation of Tomcat never updates the bindings in its function 
> mapper once one is constructed, since the functions are defined at web-app 
> load time and can't be updated after that.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to