You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by aw...@apache.org on 2007/05/30 18:39:28 UTC

svn commit: r542868 - /myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/RowKeySetImpl.java

Author: awiner
Date: Wed May 30 09:39:24 2007
New Revision: 542868

URL: http://svn.apache.org/viewvc?view=rev&rev=542868
Log:
TRINIDAD-44: Fix assertion failures in Trinidad
- Remove an invalid assertion in RowKeySetImpl.getCollectionModel()

Modified:
    myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/RowKeySetImpl.java

Modified: myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/RowKeySetImpl.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/RowKeySetImpl.java?view=diff&rev=542868&r1=542867&r2=542868
==============================================================================
--- myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/RowKeySetImpl.java (original)
+++ myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/RowKeySetImpl.java Wed May 30 09:39:24 2007
@@ -498,7 +498,10 @@
   @Override
   protected CollectionModel getCollectionModel()
   {
-    assert _model != null : "There is no CollectionModel associated with this set";
+    // This code used to contain an assertion that the collection model
+    // was non-null - but a null collection model is a perfectly
+    // legitimate state.  Users of a row key set might want to assert
+    // the collection model is non-null.
     return _model;
   }