https://issues.apache.org/bugzilla/show_bug.cgi?id=57136

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|INVALID                     |---

--- Comment #5 from Mark Thomas <ma...@apache.org> ---
Test cases that avoid JSPs and go directly to the ELProcessor are good since
they avoid a lot of the complexities. The only catch is that you still have to
use Java escaping for the String (which you have in your example above).

I've re-read that section of the EL spec and I see what you mean. That language
actually seems pretty clear although it would be better if it said the escape
was "\$}" or "\#{". In this type of case, I usually look at section 1.24 for
clarity/confirmation. Unfortunately that doesn't help. It is clear that "\$500"
is a literal expression but it offers to guidance as to whether it evaluates to
"$500" or "\$500".

I took a look at the previous EL spec and there is some helpful language there.

<quote>
LiteralExpression::= (LiteralComponent)* ([$#])?

i.e. a string of any characters that doesn't include ${ or #{ unless escaped by
\${ or \#{
</quote>

The 2.2 spec also include the language about using \$ or \# as the escape.

Given all of this, I think you are right and "\$500" should be evaluated as
"\$500" not "$500".

I've converted your example to a test case and I'll tae a look at a fix.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to