dev@javaserverfaces.java.net

Re: Seeking Review: 8-ViewLifecycle

From: Ed Burns <Ed.Burns_at_Sun.COM>
Date: Wed, 17 Nov 2004 21:47:12 -0800

>>>>> On Wed, 17 Nov 2004 21:37:14 -0800, Ed Burns <Ed.Burns_at_Sun.COM> said:

>>>>> On Thu, 11 Nov 2004 08:50:30 -0800, Adam Winer <adam.winer_at_oracle.com> said:
AW> Ed,
AW> Shouldn't we be calling all the UIViewRoot phase listener
AW> on all phases, not just encodeBegin() and encodeEnd()?

EB> Ok, but we can't do RestoreView, per EG discussion.

AW> Also, it would greatly simplify the resulting code if
AW> we pulled this requirement out of UIViewRoot and into
AW> LifecycleImpl.

EB> We can't do this, per EG discussion.

EB> Here it is.

EB> Issue: [8-ViewLifecycle]

EB> M src/javax/faces/component/UIViewRoot.java

EB> - Per EG discussion, make the phase listeners be called for every phase.

EB> M test/javax/faces/component/UIViewRootTestCase.java

EB> - lots more permutations!

Also, once this is reviewed, I need to merge this over to the
webtier-alignment repository.

Ed

-- 
| ed.burns_at_sun.com  | {home: 407 294 2468, office: 408 884 9519 OR x31640}
| homepage:         | http://javaweb.sfbay.sun.com/~edburns/
| aim: edburns0sunw | iim: ed.burns_at_sun.com
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_javaserverfaces.dev.java.net
For additional commands, e-mail: dev-help_at_javaserverfaces.dev.java.net