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:53:51 UTC

svn commit: r633084 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java

Author: lu4242
Date: Mon Mar  3 05:53:50 2008
New Revision: 633084

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

Modified:
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java?rev=633084&r1=633083&r2=633084&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/jsp/JspStateManagerImpl.java Mon Mar  3 05:53:50 2008
@@ -336,9 +336,9 @@
                 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());
             }