You are viewing a plain text version of this content. The canonical link for it is here.
Posted to adffaces-commits@incubator.apache.org by aw...@apache.org on 2007/02/06 01:54:23 UTC

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

Author: awiner
Date: Mon Feb  5 17:54:23 2007
New Revision: 503953

URL: http://svn.apache.org/viewvc?view=rev&rev=503953
Log:
Fix a possible NPE when calling resetStampState() on a table that hasn't been processed yet.  And switch an Oracle bug number to a description of the bug

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

Modified: incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java
URL: http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java?view=diff&rev=503953&r1=503952&r2=503953
==============================================================================
--- incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java (original)
+++ incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java Mon Feb  5 17:54:23 2007
@@ -184,8 +184,11 @@
     InternalState iState = _getInternalState(true);
     // TODO: this is over kill. for eg, It clears out any toggled showDetails.
     Object initKey = _getCurrencyKeyForInitialStampState();
-    // do not clear the initial stamp state: bug 4862103:
-    iState._stampState.clear(initKey);
+    // do not clear the initial stamp state: a subtle bug could
+    // result where the initial state of each component is gone, so we
+    // fail to roll back to the initial default values
+    if (iState._stampState != null)
+      iState._stampState.clear(initKey);
   }
 
   @Override