On 4/30/07, Anahide Tchertchian <[EMAIL PROTECTED]> wrote:
Hi,
This is on the 1.1 trunk.

There is no UIComponent.getContainerClientId() method
in JSF 1.1.  It was added in 1.2, and I'm pretty sure our
1.2 code properly calls it.  The trunk is strictly 1.1.  So
I don't think you've found a bug.

I'd be happy to test the 1.2 branch if I could find a maven repository
for it :)

You have to build it yourself, I'm afraid.  Ongoing problems
with continuum have prevented us from putting it in a maven
repository.

-- Adam



Adam Winer a écrit :
> Is this on the 1.2 branch or the 1.1 trunk?
>
> -- Adam
>
>
> On 4/25/07, Anahide Tchertchian <[EMAIL PROTECTED]> wrote:
>> Hi,
>>
>> I noticed that trinidad components client ids are not set up correctly
>> when used inside a naming container that redefines its children client
>> ids.
>>
>> I believe this could be fixed applying the following changes, which is
>> not very visible as the default implementation returns getClientId when
>> calling getContainerClientId.
>>
>> $ svn di
>> ./src/main/java/org/apache/myfaces/trinidad/component/UIXComponentBase.java
>>
>> Index:
>> src/main/java/org/apache/myfaces/trinidad/component/UIXComponentBase.java
>> ===================================================================
>> ---
>> src/main/java/org/apache/myfaces/trinidad/component/UIXComponentBase.java
>>    (revision 532384)
>> +++
>> src/main/java/org/apache/myfaces/trinidad/component/UIXComponentBase.java
>>    (working copy)
>> @@ -270,7 +270,7 @@
>>       {
>>         if (containerComponent instanceof NamingContainer)
>>         {
>> -        clientId = (containerComponent.getClientId(context) +
>> +        clientId = (containerComponent.getContainerClientId(context) +
>>                       NamingContainer.SEPARATOR_CHAR +
>>                       clientId);
>>           break;
>>
>>
>>
>> Do you think this could be made available for the next version?
>> Thanks,
>>
>> --
>> Anahide Tchertchian, Nuxeo
>> Mail: [EMAIL PROTECTED] - Tel: +33 (0)1 40 33 79 87
>> http://www.nuxeo.com - http://www.nuxeo.org
>>
>

Reply via email to