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/09/03 00:52:51 UTC

svn commit: r1380084 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java

Author: lu4242
Date: Sun Sep  2 22:52:51 2012
New Revision: 1380084

URL: http://svn.apache.org/viewvc?rev=1380084&view=rev
Log:
MYFACES-3601 Avoid call UIViewRoot.setViewId(...) when it is not necessary to prevent state creation 

Modified:
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java?rev=1380084&r1=1380083&r2=1380084&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/view/facelets/FaceletViewDeclarationLanguage.java Sun Sep  2 22:52:51 2012
@@ -385,9 +385,17 @@ public class FaceletViewDeclarationLangu
         }
 
         // setup our viewId
-        String renderedViewId = getRenderedViewId(context, view.getViewId());
+        String previousViewId = view.getViewId();
+        String renderedViewId = getRenderedViewId(context, previousViewId);
 
-        view.setViewId(renderedViewId);
+        if (renderedViewId == null)
+        {
+            view.setViewId(renderedViewId);
+        }
+        else if (!renderedViewId.equals(previousViewId))
+        {
+            view.setViewId(renderedViewId);
+        }
 
         if (log.isLoggable(Level.FINEST))
         {