Mailing-List: contact dev-help@myfaces.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "MyFaces Development" <dev@myfaces.apache.org>
Message-ID: <76645004.1208847448163.JavaMail.jira@brutus>
Date: Mon, 21 Apr 2008 23:57:28 -0700 (PDT)
From: "Bernd Bohmann (JIRA)" <dev@myfaces.apache.org>
To: dev@myfaces.apache.org
Subject: [jira] Resolved: (MYFACES-1825) duplicate ids not detected in
 JspStateManagerImpl.checkForDuplicateIds()
In-Reply-To: <2123783207.1204038893926.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/MYFACES-1825?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bernd Bohmann resolved MYFACES-1825.
------------------------------------

    Resolution: Fixed

> duplicate ids not detected in JspStateManagerImpl.checkForDuplicateIds() 
> -------------------------------------------------------------------------
>
>                 Key: MYFACES-1825
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1825
>             Project: MyFaces Core
>          Issue Type: Bug
>    Affects Versions: 1.1.5
>            Reporter: gaston
>            Assignee: Bernd Bohmann
>             Fix For:  1.1.6-SNAPSHOT, 1.2.3-SNAPSHOT
>
>
> Some duplicate id are not detected in my MyFaces(1.1.5)/Facelets(1.1.12) based application.
> I think there's a bug in MyFaces's detection of duplicate ids in JspStateManagerImpl.checkForDuplicateIds() : 
> this code seems to be wrong :
>         boolean namingContainer = component instanceof NamingContainer;
>         while (it.hasNext()) {
>             UIComponent kid = (UIComponent) it.next();
>             if (namingContainer) {
>                 checkForDuplicateIds(context, kid, new HashSet());
>             }
>             else {
>                 checkForDuplicateIds(context, kid, ids);
>             }
>         }
> It should be :
>         while (it.hasNext()) {
>             UIComponent kid = (UIComponent) it.next();
>             boolean namingContainer = kid instanceof NamingContainer;
>             if (namingContainer) {
>                 checkForDuplicateIds(context, kid, new HashSet());
>             }
>             else {
>                 checkForDuplicateIds(context, kid, ids);
>             }
>         }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.