tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 48941] New: Make EL api and servlet container compatible with other Unified EL implementations
Date Thu, 18 Mar 2010 20:50:48 GMT
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


Mime
View raw message