dev@javaserverfaces.java.net

[REVIEW] Ensure FacesContextImpl.release() sets ThreadLocal to null

From: Ryan Lubke <Ryan.Lubke_at_Sun.COM>
Date: Wed, 14 Dec 2005 10:19:26 -0800

Ensure the FacesContext ThreadLocal is set to null when release is called.

This is for JSF_1_1_ROLLING and HEAD.

SECTION: Modified Files
----------------------------
M portlet-lib/src/java/com/sun/faces/portlet/FacesContextImpl.java


SECTION: Diffs
----------------------------
Index: portlet-lib/src/java/com/sun/faces/portlet/FacesContextImpl.java
===================================================================
RCS file:
/cvs/javaserverfaces-sources/jsf-portlet/portlet-lib/src/java/com/sun/faces/portlet/FacesContextImpl.java,v
retrieving revision 1.4
diff -u -r1.4 FacesContextImpl.java
--- portlet-lib/src/java/com/sun/faces/portlet/FacesContextImpl.java
 22 Aug 2005 22:09:59 -0000 1.4
+++ portlet-lib/src/java/com/sun/faces/portlet/FacesContextImpl.java
 14 Dec 2005 18:17:50 -0000
@@ -262,6 +262,8 @@
     public void release() {
         assertNotReleased();
         released = true;
+ // Make sure to clear our ThreadLocal instance.
+ setCurrentInstance(null);
     }