You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ma...@apache.org on 2008/06/28 18:13:16 UTC

svn commit: r672542 - /myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java

Author: matzew
Date: Sat Jun 28 09:13:11 2008
New Revision: 672542

URL: http://svn.apache.org/viewvc?rev=672542&view=rev
Log:
TRINIDAD-1135 - Flush Cached Model for UIXCollection during broadcast
Thanks to Venkata for his patch.

Modified:
    myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java

Modified: myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java?rev=672542&r1=672541&r2=672542&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java (original)
+++ myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java Sat Jun 28 09:13:11 2008
@@ -134,6 +134,12 @@
   public void broadcast(FacesEvent event)
     throws AbortProcessingException
   {
+    // If the UIXCollection component is used as a stamp of another collection component,
+    // we could be pointing to the wrong model based on the the last stamp state from 
+    // parent's decode/validate/update processing. Flush the cached model to get the correct model 
+    // based on the parent's currency
+    _flushCachedModel();
+    
     // For "TableRowEvents", set up the data before firing the
     // event to the actual component.
     if (event instanceof TableRowEvent)