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 2008/03/03 14:58:21 UTC

svn commit: r633087 - /myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java

Author: lu4242
Date: Mon Mar  3 05:58:18 2008
New Revision: 633087

URL: http://svn.apache.org/viewvc?rev=633087&view=rev
Log:
fixes MYFACES-1825 duplicate ids not detected in JspStateManagerImpl.checkForDuplicateIds()

Modified:
    myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java

Modified: myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java?rev=633087&r1=633086&r2=633087&view=diff
==============================================================================
--- myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java (original)
+++ myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java Mon Mar  3 05:58:18 2008
@@ -370,10 +370,10 @@
                                             getPathToComponent(component));
         }
         Iterator it = component.getFacetsAndChildren();
-        boolean namingContainer = component instanceof NamingContainer;
         while (it.hasNext())
         {
             UIComponent kid = (UIComponent) it.next();
+            boolean namingContainer = kid instanceof NamingContainer;
             if (namingContainer)
             {
                 checkForDuplicateIds(context, kid, new HashSet<String>());