myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ji Kim <ji....@oracle.com>
Subject Re: [Trinidad] New API for UIXEditableValueTemplate
Date Wed, 19 Mar 2014 17:56:29 GMT
Cool thx, must have got mixed up w/ the previous patch.

Made the change for addValueUpdatedListener/removeValueUpdatedListener

Thanks!

On 3/19/2014 10:47 AM, Mark Yvanovich wrote:
> I have one small quibble with this.  I like symmetry in add/remove 
> methods, so I'd prefer either:
> addValueUpdatedListener/removeValueUpdatedListener
> or
> addValueUpdatedChangeListener/removeValueUpdatedChangeListener
>
> The former probably makes more sense, since the parameter is a 
> ValueUpdatedListener class.
>
> --Marky
>
> On 3/17/2014 6:03 PM, Ji Kim wrote:
>> Hi,
>>
>> This thread is to discuss JIRA 
>> [https://issues.apache.org/jira/browse/TRINIDAD-2459] which is to add 
>> two new APIs of
>>
>> public void addValueUpdatedListener(ValueUpdatedListener listener)
>> {
>>   super.addFacesListener (listener);
>> }
>>
>> public void removeValueUpdatedChangeListener(ValueUpdatedListener 
>> listener)
>> {
>>   super.removeFacesListener (listener);
>> }
>>
>> This is to allow component's and etcetera to add NEWLY added 
>> ValueUpdatedListener listener to the component where 
>> ValueUpdatedEvent is queued within the updateModel method/phase.
>>
>> Thanks!
>


Mime
View raw message