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 00:51:17 UTC

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

Author: mmarinschek
Date: Wed Sep 20 15:51:17 2006
New Revision: 448391

URL: http://svn.apache.org/viewvc?view=rev&rev=448391
Log:
fix for [MYFACES-1412] : Restore view phase does not conform to specification. Thanks to Nikolay Petrov for fixing this spec-compatibility bug.

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?view=diff&rev=448391&r1=448390&r2=448391
==============================================================================
--- 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 Wed Sep 20 15:51:17 2006
@@ -44,6 +44,12 @@
 	private static final Log log = LogFactory.getLog(LifecycleImpl.class);
 
 	public boolean execute(FacesContext facesContext) {
+		if(facesContext.getViewRoot() != null) {
+			facesContext.getViewRoot().setLocale(facesContext.getExternalContext().getRequestLocale());
+			RestoreStateUtils.recursivelyHandleComponentReferencesAndSetValid(facesContext, facesContext.getViewRoot());
+			return false;
+		}
+
 		// Derive view identifier
 		String viewId = deriveViewId(facesContext);