[MYFACES-1412] Restore view phase does not conforms specification

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.1.4
• Fix Version/s: 1.1.5
• Component/s: JSR-127
• Assignee: Martin Marinschek
• Reporter: Nikolay Petrov
• Created: 20/Sep/06 10:28
• Updated: 14/Feb/07 00:30
• Resolved: 20/Sep/06 22:44

Description

JSF 1.1 spec, section 2.2.1 says:
-Examine the FacesContext instance for the current request. If it already
contains a UIViewRoot:
--Set the locale on this UIViewRoot to the value returned by the
getRequestLocale() method on the ExternalContext for this request.
--For each component in the component tree, determine if a ValueBinding for
"binding" is present. If so, call the setValue() method on this
ValueBinding, passing the component instance on which it was found.
--Take no further action during this phase.

As I was looking at restore view phase implementation there was no such behaviour implemented. This should be implemented as:
if(facesContext.getViewRoot() != null)

{ facesContext.getViewRoot().setLocale(facesContext.getExternalContext().getRequestLocale()); RestoreStateUtils.recursivelyHandleComponentReferencesAndSetValid(facesContext, facesContext.getViewRoot()); return false; }

At least if understand the code properly.