You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ar...@apache.org on 2011/02/24 19:23:27 UTC

svn commit: r1074249 - /myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java

Author: arobinson74
Date: Thu Feb 24 18:23:27 2011
New Revision: 1074249

URL: http://svn.apache.org/viewvc?rev=1074249&view=rev
Log:
TRINIDAD-2045 - Commit patch made by Venkata Guddanti

Modified:
    myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java

Modified: myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java?rev=1074249&r1=1074248&r2=1074249&view=diff
==============================================================================
--- myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java (original)
+++ myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java Thu Feb 24 18:23:27 2011
@@ -253,6 +253,15 @@ public abstract class UIXCollection exte
       setRowKey(currencyKey);
     }
 
+    // Finally clean up any internal model state that we might be holding on to. We do not want to hold onto any 
+    // application data in between requests 
+    InternalState iState = _getInternalState(false);
+    if (iState != null)
+    {
+      iState._value = null;
+      iState._model= null;
+    }
+    
     return savedState;
   }
 
@@ -272,10 +281,6 @@ public abstract class UIXCollection exte
     {
       stampState = iState._stampState;
       clientKeyMgr = iState._clientKeyMgr;
-      
-      // Make sure that we are not holding on to any application data in our state
-      iState._value = null;
-      iState._model= null;
     }
     else
     {