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!


Reply via email to