dev@jsf-extensions.java.net

Re: [JSF-EXT] redundant phase listener diff

From: Ed Burns <ed.burns_at_sun.com>
Date: Tue, 4 Sep 2007 13:32:18 -0700

>>>>> On Tue, 04 Sep 2007 13:16:31 -0700, Matthew Bohm <Matthew.Bohm_at_Sun.COM> said:

MB> listeners of the same type redundant. Note that this is, generally
MB> speaking, a wider definition of redundancy than the logic employed
MB> in jsf 1.2_05, which effectively tests for redundancy using the
MB> equals method of the phase listeners. (Shale remoting's phase
MB> listener, for instance, does not override equals, so only the exact
MB> same instance is considered redundant.) Using this slightly
MB> different definition of redundancy in
MB> PartialTraversalLifecycle.execute maintains consistency with
MB> PartialTraversalLifecycle.addPhaseListener and ensures that phase
MB> listeners are tested for both defintions of redundancy as of jsf
MB> 1.2_05.

Can you add an || that uses the equals() method?

Ed

-- 
| ed.burns_at_sun.com  | office: 408 884 9519 OR x31640
| homepage:         | http://purl.oclc.org/NET/edburns/
| aim: edburns0sunw | iim: ed.burns_at_sun.com