ExpressionFactoryImpl.createMethodExpression() is not EL 2.2 Spec conform

[glassfishrobot]

com.sun.el.ExpressionFactoryImpl.createMethodExpression(ELContext context, String expression, Class expectedReturnType, Class[] expectedParamTypes)

does not correctly impl EL Specification 2.2 (Maintenance Release 2 - December 10, 2009)

The spezfication say at site 72 (last paragraph): expectedParamTypes - The expected parameter types for the method to be found. Must be an array with no elements if there are no parameters expected. It is illegal to pass null, unless the method is specified with arugments in the EL expression, in which case these arguments are used for method selection, and this parameter is ignored.

The part ", unless the method is specified...." was added in 2.2 and is not implemented.

If i call the method with expectedParamTypes == null i get NullPointerException, although expression #

{bean.sayHello('World')}

has arguments.

The tomcat 7.0 el (org.apache.el.ExpressionFactoryImpl) has the same defect. The code looks identical. JUEL (http://juel.sourceforge.net/) works fine.

Environment

Operating System: All Platform: All

Affected Versions

[current]

[glassfishrobot]

• Issue Imported From: https://github.com/javaee/uel-ri/issues/18
• Original Issue Raised By:@glassfishrobot
• Original Issue Assigned To: @glassfishrobot
• Original Issue Closed By:@glassfishrobot

[glassfishrobot]

@glassfishrobot Commented Reported by sascha_gross

[glassfishrobot]

@glassfishrobot Commented sascha_gross said: Bug at Tomcat https://issues.apache.org/bugzilla/show_bug.cgi?id=49655

[glassfishrobot]

@glassfishrobot Commented kchung said: Fixed. The fix should be in the next EL promoted build, and eventually in the glassfish 3.1 build.

Thanks for reporting the bug.

[glassfishrobot]

@glassfishrobot Commented Was assigned to uel-issues

[glassfishrobot]

@glassfishrobot Commented This issue was imported from java.net JIRA UEL-18

[glassfishrobot]

@glassfishrobot Commented Marked as fixed on Thursday, July 29th 2010, 6:05:18 am