ok - i just added the patchfile.
hope this is formal correct.

thanks for your help
elmar


2009/9/11 Matthias Wessendorf <[email protected]>:
> https://issues.apache.org/jira/browse/TRINIDAD-1397
>
> Operations
>
> => Attach file
>
> On Fri, Sep 11, 2009 at 1:12 PM, Elmar Kretzer <[email protected]> 
> wrote:
>> Hi Matthias,
>>
>> of course - but i haven`t done this yet.
>> can you give me link with a how to?
>>
>>
>> elmar
>>
>> 2009/9/11 Matthias Wessendorf <[email protected]>:
>>> can you upload a diff file ?
>>>
>>> On Fri, Sep 11, 2009 at 1:05 PM, Elmar Kretzer <[email protected]> 
>>> wrote:
>>>> Hi,
>>>> i got a question concerning a possible patch for
>>>> https://issues.apache.org/jira/browse/TRINIDAD-1397.
>>>>
>>>> The issue is, that the SelectManyShuttleRenderer uses
>>>> rc.setSkinResourceKeyMap(getResourceKeyMap()); to render the
>>>>  SelectManyShuttle styleClasses.
>>>> But at the end of encodeElementContent() the skinResourceKeyMap gets
>>>> not cleared.
>>>>
>>>> Therefore a panelBox containing a selectManyShuttle gets corrupted 
>>>> styleClasses.
>>>>
>>>> I could provided a possible patch for this issue, but honestly i don`t
>>>> know how to do this.
>>>> So could anyone advice me?
>>>>
>>>> thx
>>>> elmar
>>>>
>>>> Patch Information:
>>>>
>>>> file 
>>>> org.apache.myfaces.trinidadinternal.renderkit.core.xhtml.SelectManyShuttleRenderer
>>>>
>>>> Line 293:
>>>>
>>>>    change:
>>>>    rc.setSkinResourceKeyMap(getResourceKeyMap());
>>>>
>>>>    to:
>>>>   Map<String, String> originalSkinResourceMap = rc.getSkinResourceKeyMap();
>>>>    rc.setSkinResourceKeyMap(getResourceKeyMap());
>>>>
>>>> Line 326:
>>>>
>>>>   change:
>>>>   _clearContext(rc);
>>>>
>>>>  to:
>>>>  rc.setSkinResourceKeyMap(originalSkinResourceMap);
>>>>  _clearContext(rc);
>>>>
>>>
>>>
>>>
>>> --
>>> Matthias Wessendorf
>>>
>>> blog: http://matthiaswessendorf.wordpress.com/
>>> sessions: http://www.slideshare.net/mwessendorf
>>> twitter: http://twitter.com/mwessendorf
>>>
>>
>
>
>
> --
> Matthias Wessendorf
>
> blog: http://matthiaswessendorf.wordpress.com/
> sessions: http://www.slideshare.net/mwessendorf
> twitter: http://twitter.com/mwessendorf
>

Reply via email to