You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by jw...@apache.org on 2013/05/02 23:38:04 UTC

svn commit: r1478575 - /myfaces/trinidad/trunk/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXTableTemplate.java

Author: jwaldman
Date: Thu May  2 21:38:04 2013
New Revision: 1478575

URL: http://svn.apache.org/r1478575
Log:
TRINIDAD-2380 Collection Component's internal state is not initialized properly in some cases. 
thanks to Jing Wu for the patch

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

Modified: myfaces/trinidad/trunk/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXTableTemplate.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXTableTemplate.java?rev=1478575&r1=1478574&r2=1478575&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXTableTemplate.java (original)
+++ myfaces/trinidad/trunk/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXTableTemplate.java Thu May  2 21:38:04 2013
@@ -642,6 +642,10 @@ abstract public class UIXTableTemplate e
           try
           {
             RowKeySet rowKeys = (RowKeySet) value;
+            
+            // make sure the state is properly initiated first by calling getCollectionModel().
+            getCollectionModel();
+
             // row key sets need the most recent collection model, but there is no one common entry
             // point to set this on the set besides when code asks for the value from the bean
             __flushCachedModel();  //insist that we populate with the very lastest instance of the collection model
@@ -664,6 +668,10 @@ abstract public class UIXTableTemplate e
           try
           {
             RowKeySet rowKeys = (RowKeySet) value;
+
+            // make sure the state is properly initiated first by calling getCollectionModel().
+            getCollectionModel();
+
             // row key sets need the most recent collection model, but there is no one common entry
             // point to set this on the set besides when code asks for the value from the bean
             __flushCachedModel();  //insist that we populate with the very lastest instance of the collection model