dev@javaserverfaces.java.net

Re: Seeking Review: 140-FacesContextScope

From: Ed Burns <Ed.Burns_at_Sun.COM>
Date: Mon, 09 Jun 2008 12:08:24 -0700

>>>>> On Mon, 09 Jun 2008 08:57:08 -0700, Ryan Lubke <Ryan.Lubke_at_Sun.COM> said:

RL> Go ahead and check this in, but I will be making some changes:
RL> - getAttributes() implementation will be moved to FacesContext.java
RL> proper to avoid the proliferation of the impl dependency hacks
RL> (unless you saw some reason not to).
RL> - RequestStateManager will be removed in favor of leveraging
RL> the FacesContext attributes Map.

That's fine, but make sure to remove it from InitFacesContext and the
MockFacesContext in jsf-extensions-test-time.

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