You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2010/04/29 08:21:39 UTC

svn commit: r939206 - /myfaces/core/branches/1.2.x/api/src/main/java/javax/faces/component/UIComponentBase.java

Author: lu4242
Date: Thu Apr 29 06:21:39 2010
New Revision: 939206

URL: http://svn.apache.org/viewvc?rev=939206&view=rev
Log:
MYFACES-2684 UIComponentBase.saveAttachedState save transient elements on List case

Modified:
    myfaces/core/branches/1.2.x/api/src/main/java/javax/faces/component/UIComponentBase.java

Modified: myfaces/core/branches/1.2.x/api/src/main/java/javax/faces/component/UIComponentBase.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/1.2.x/api/src/main/java/javax/faces/component/UIComponentBase.java?rev=939206&r1=939205&r2=939206&view=diff
==============================================================================
--- myfaces/core/branches/1.2.x/api/src/main/java/javax/faces/component/UIComponentBase.java (original)
+++ myfaces/core/branches/1.2.x/api/src/main/java/javax/faces/component/UIComponentBase.java Thu Apr 29 06:21:39 2010
@@ -952,7 +952,11 @@ public abstract class UIComponentBase
             List<Object> lst = new ArrayList<Object>(((List)attachedObject).size());
             for (Iterator it = ((List)attachedObject).iterator(); it.hasNext(); )
             {
-                lst.add(saveAttachedState(context, it.next()));
+                Object value = it.next();
+                if (value != null)
+                {
+                    lst.add(saveAttachedState(context, value));
+                }
             }
             return new _AttachedListStateWrapper(lst);
         }