Checks in TreeMethodExpression are too restrictive about juel HOT 5 CLOSED

[Comment migrated from SourceForge | Submitted by 'beckchr']

Hi! The check is only done if the expression is not an invocation. I think the code is correct here. I suppose that bug #3031795 is the problem. Agree?

from juel.

[Comment migrated from SourceForge | Submitted by 'struberg']

think this is only the half way though ;)
"is illegal to pass null, "

ok so far, BUT:

"unless the method is specified with arguments in the EL expression, in which case these arguments are used for method
selection, and this parameter is ignored."

As far as I read this section it means: it MIGHT be null if the invocation has parameters which the implementation should guess.
wdyt?

LieGrue,
strub

from juel.

[Comment migrated from SourceForge | Submitted by 'beckchr']

agree, but if the invocation has parameters (expression like #{foo.bar(x, y, z)}), the "if (paramType != null) {...}" code is not executed.

from juel.

[Comment migrated from SourceForge | Submitted by 'beckchr']

fixed in trunk

from juel.

[Comment migrated from SourceForge | Submitted by 'beckchr']

created new bug #3104608, closing this one.

from juel.