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:22:01 UTC

svn commit: r939207 - /myfaces/core/trunk/api/src/main/java/javax/faces/component/UIComponentBase.java

Author: lu4242
Date: Thu Apr 29 06:22:01 2010
New Revision: 939207

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

Modified:
    myfaces/core/trunk/api/src/main/java/javax/faces/component/UIComponentBase.java

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