[MYFACES-4099] Allow resolve #{cc} inside templates called from a composite component

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.12
• Fix Version/s: 2.3.0-beta
• Assignee: Leonardo Uribe
• Reporter: Thomas Andraschko
• Created: 21/Feb/17 21:54
• Updated: 09/Jun/17 01:24
• Resolved: 24/Feb/17 00:38

Description

It has been reported this scenario:

<cc:implementation>
     <ui:include src="#{cc.page}"/>
</cc:implementation>

In the included page:

<ui:composition ...>
     <test:anoterCompositeComponent locale="#{cc.someProperty}"/>
</ui:composition>

The problem right now is such kind of indirection has not been taken into account for #

{cc} EL resolution algorithm. It is supposed that any resolution of #{cc}

happens inside the composite component .xhtml and not indirectly through a template like is happening in the previous example.

This code leads to an StackOverflowException, because the algorithm cannot find the right composite component and by default it takes the closest one.

In MyFaces it is possible to pass the values as parameters of ui:include, but the point is it is better if we allow these example to work.

Please note this is not a bug. This looks like an improvement. The challenge here is expressions using #

{cc}

outside a composite component xhtml are not cacheable, so we need to study if this improvement can be done or not.