[MYFACES-2740] MethodExpression wrapping of vdl.retargetMethodExpression does not take into account #{cc} resolution

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.1
• Component/s: JSR-314
• Assignee: Leonardo Uribe
• Reporter: Leonardo Uribe
• Created: 28/May/10 22:20
• Updated: 30/Jul/12 13:59
• Resolved: 28/May/10 22:24

Description

On vdl.retargetMethodExpression, it is done something like this to "retarget" methodExpressions:

if (CompositeComponentELUtils.isCompositeComponentExpression(attributeExpressionString))

{ methodExpression = new ValueExpressionMethodExpression(attributeNameValueExpression); }

This strategy causes problems when complex EL expressions are resolved.

I think it is better to "unwrap" the original expression, trying to find LocationValueExpression wrappers and "rewrap" the method expression to be created with a LocationMethodExpression. In this way, complex EL expressions are resolved correctly.

I did some tests and it works well. Minor fixes were included with this patch too.

Issue Links

• is related to: MYFACES-2561 StackOverflowError if a composite component implementation uses another composite component