[jsr-314-open] composite:insertFacet target facet name

[Alexander Smirnov]

On 09/16/2009 10:59 AM, Ed Burns wrote:
>>>>>> On Tue, 15 Sep 2009 19:29:22 -0400, Andy Schwartz<andy.schwartz at oracle.com>  said:
>
> AS>  Gang -
> AS>  As currently specified, composite:insertFacet's "name" attribute serves
> AS>  two purposes:
>
> AS>  1. It identifies the name of the facet on the containing composite
> AS>  component to insert.
> AS>  2. It identifies the name of the facet on the target component into
> AS>  which the facet is being inserted.
>
> [...]
>
> AS>  1. Add an attribute to composite:insertFacet that allows a target facet
> AS>  name to be specified:
>
> AS>    <h:panelGrid>
> AS>      <composite:insertFacet name="backupCaption" targetName="caption"/>
> AS>    </h:panelGrid>
>
> AS>  2. Specify that the target facet name can be picked up from a wrapping
> AS>  <f:facet>  tag:
>
> AS>    <h:panelGrid>
> AS>      <f:facet name="caption">
> AS>        <composite:insertFacet name="backupCaption"/>
> AS>      </f:facet>
> AS>    </h:panelGrid>
>
> AS>  I prefer #2 since is consistent with typical facet usage.
>
> I happen to prefer #1, but everyone else favors #2, we'll go with #2.
Sorry for wrong decigion, I've mixed 'composite:insertFacet' and 
'composite:renderFacet' tags. The second option is makes sense for 
'renderFacet' but 'insertFacet' seems better to use an additional attribute.
PS. I really don't see an advantage to have two so alike tags, except 
performance purposes. As you already saw even EG members are messed up 
with them :-)
>
> Andy, can you please file a spec issue and share the number with the
> group?  Once you have it, I'll add an entry in the changelog wiki.
>
> Ed
>