You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mm...@apache.org on 2006/09/21 13:31:55 UTC

svn commit: r448521 - /myfaces/core/trunk/api/src/main/java/javax/faces/webapp/UIComponentTag.java

Author: mmarinschek
Date: Thu Sep 21 04:31:54 2006
New Revision: 448521

URL: http://svn.apache.org/viewvc?view=rev&rev=448521
Log:
(re)fix for [MYFACES-1029] : Duplicate sibling ids allowed

Modified:
    myfaces/core/trunk/api/src/main/java/javax/faces/webapp/UIComponentTag.java

Modified: myfaces/core/trunk/api/src/main/java/javax/faces/webapp/UIComponentTag.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/api/src/main/java/javax/faces/webapp/UIComponentTag.java?view=diff&rev=448521&r1=448520&r2=448521
==============================================================================
--- myfaces/core/trunk/api/src/main/java/javax/faces/webapp/UIComponentTag.java (original)
+++ myfaces/core/trunk/api/src/main/java/javax/faces/webapp/UIComponentTag.java Thu Sep 21 04:31:54 2006
@@ -586,8 +586,7 @@
             // Sun RI just issues a warning...
             if(parentTag._childrenAdded != null && parentTag._childrenAdded.contains(id))
             {
-                if(log.isWarnEnabled())
-                    log.warn("There is more than one JSF tag with an id : " + id);
+                throw new FacesException("There is more than one JSF tag with id : " + id+" for parent component with id : '"+parent.getId()+"'");
             }
             
             _componentInstance = findComponent(parent,id);