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 2009/09/04 01:01:55 UTC

svn commit: r811165 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/lifecycle/RestoreViewExecutor.java

Author: lu4242
Date: Thu Sep  3 23:01:55 2009
New Revision: 811165

URL: http://svn.apache.org/viewvc?rev=811165&view=rev
Log:
MYFACES-2345 Add View Parameters feature (add TODOs)

Modified:
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/lifecycle/RestoreViewExecutor.java

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/lifecycle/RestoreViewExecutor.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/lifecycle/RestoreViewExecutor.java?rev=811165&r1=811164&r2=811165&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/lifecycle/RestoreViewExecutor.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/lifecycle/RestoreViewExecutor.java Thu Sep  3 23:01:55 2009
@@ -77,6 +77,10 @@
 
         String viewId = restoreViewSupport.calculateViewId(facesContext);
 
+        // TODO: JSF 2.0 spec section 2.5.5 if the browser issues a GET request with query parameters,
+        // no state is restored but full lifecycle minus invoke application should happen 
+        // if query parameters are set!
+        
         // Determine if this request is a postback or initial request
         if (restoreViewSupport.isPostback(facesContext))
         { // If the request is a postback