You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2010/03/18 21:50:48 UTC

DO NOT REPLY [Bug 48941] New: Make EL api and servlet container compatible with other Unified EL implementations

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

           Summary: Make EL api and servlet container compatible with
                    other Unified EL implementations
           Product: Tomcat 6
           Version: 6.0.26
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Servlet & JSP API
        AssignedTo: dev@tomcat.apache.org
        ReportedBy: contact@ruslan.org


Created an attachment (id=25149)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=25149)
Patch which makes proposed modifications

Make EL api and servlet container compatible with other Unified EL
implementations

Currently tomcat instantiates EL ExpressionFactory
not by using standard ExpressionFactory.newInstance()
but by directly calling org.apache.el.ExpressionFactoryImpl()
constructor.

See
org.apache.jasper.runtime.JspApplicationContextImpl
for example.

This proposal is simple. 

1. Modify bundled EL api to add newInstance method to
javax.el.ExpressionFactory which calls org.apache.el.ExpressionFactoryImpl()
constructor.

2. Replace all invocations of org.apache.el.ExpressionFactoryImpl() with
standard ExpressionFactory.newInstance() call.

This way if EL api is replaced, new version can be used without modification.

However, to fully remove dependency on org.apache.el changes need to be made to 
org.apache.jasper.compiler.Validator which directly static method in ELSupport
class. This is much more complex matter which can be addressed later.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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


DO NOT REPLY [Bug 48941] Make EL api and servlet container compatible with other Unified EL implementations

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=48941

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

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

--- Comment #1 from Mark Thomas <ma...@apache.org> 2010-03-19 07:56:54 UTC ---
ExpressionFactory.newInstance() is an EL 2.2 method. Tomcat 6 supports EL 2.1.

Note that Tomcat 7 will support EL 2.2 and that the necessary API changes have
already been made.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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