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 2012/12/22 02:13:19 UTC

svn commit: r1425218 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/RenderFacetHandler.java

Author: lu4242
Date: Sat Dec 22 01:13:19 2012
New Revision: 1425218

URL: http://svn.apache.org/viewvc?rev=1425218&view=rev
Log:
MYFACES-3668 CLONE - UIForm.createUniqueId should take into account prependId logic (do not create component when building component metadata)

Modified:
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/RenderFacetHandler.java

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/RenderFacetHandler.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/RenderFacetHandler.java?rev=1425218&r1=1425217&r2=1425218&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/RenderFacetHandler.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/composite/RenderFacetHandler.java Sat Dec 22 01:13:19 2012
@@ -114,8 +114,15 @@ public class RenderFacetHandler extends 
             }
             
             facetList.add(facetName);
+            
+            // Do not call super.apply(ctx, parent), because it forces component creation,
+            // and in this step it is not necessary. Also, it changes the order of 
+            // the generated ids.
+        }
+        else
+        {
+            super.apply(ctx, parent);
         }
-        super.apply(ctx, parent);
     }
 
     @Override