2010/10/13 Andy Schwartz <andy.schwartz@oracle.com>

Hey Leonardo -

I haven't had a chance to review your patches just yet, but one quick question about this:

javax.faces.application.ViewHandler (and ViewHandlerWrapper)

// Resolve the ambiguity found on getViewDeclarationLanguage() method. Only called from Restore View Phase.
public ViewDeclarationLanguage getViewDeclarationLanguageFromRawViewId(FacesContext context, String viewId)

Why not use ViewHandler.deriveViewId() and specify that ViewHandler.getViewDeclarationLanguage() takes a derived view id?

I tried to use it in MyFaces long time ago but unfortunately use it breaks TCK test, because deriveViewId() check at the end if the physical resource exists and if it does not, return null.

Leonardo

Andy