You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ja...@apache.org on 2010/03/19 14:33:59 UTC

svn commit: r925212 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/jsf/ComponentTagHandlerDelegate.java

Author: jakobk
Date: Fri Mar 19 13:33:59 2010
New Revision: 925212

URL: http://svn.apache.org/viewvc?rev=925212&view=rev
Log:
MYFACES-2598 UIViewParameter does not get an automatic id

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

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/jsf/ComponentTagHandlerDelegate.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/jsf/ComponentTagHandlerDelegate.java?rev=925212&r1=925211&r2=925212&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/jsf/ComponentTagHandlerDelegate.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/tag/jsf/ComponentTagHandlerDelegate.java Fri Mar 19 13:33:59 2010
@@ -51,7 +51,6 @@ import javax.faces.view.facelets.TagHand
 
 import org.apache.myfaces.util.ExternalSpecifications;
 import org.apache.myfaces.view.facelets.AbstractFaceletContext;
-import org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage;
 import org.apache.myfaces.view.facelets.tag.MetaRulesetImpl;
 import org.apache.myfaces.view.facelets.tag.composite.InsertChildrenHandler;
 import org.apache.myfaces.view.facelets.tag.composite.InsertFacetHandler;
@@ -220,6 +219,14 @@ public class ComponentTagHandlerDelegate
                 if (uniqueIdVendor == null)
                 {
                     uniqueIdVendor = facesContext.getViewRoot();
+                    
+                    if (uniqueIdVendor == null)
+                    {
+                        // facesContext.getViewRoot() returns null here if we are in
+                        // phase restore view, so we have to try to get the view root
+                        // via the method in ComponentSupport and our parent
+                        uniqueIdVendor = ComponentSupport.getViewRoot(ctx, parent);
+                    }
                 }
                 if (uniqueIdVendor != null)
                 {