Hi Andy<br><br><div class="gmail_quote">2010/10/13 Andy Schwartz <span dir="ltr"><<a href="mailto:andy.schwartz@oracle.com">andy.schwartz@oracle.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hey Leonardo -<br>
<br>
I haven't had a chance to review your patches just yet, but one quick question about this:<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
javax.faces.application.ViewHandler (and ViewHandlerWrapper)<br>
<br>
// Resolve the ambiguity found on getViewDeclarationLanguage() method. Only called from Restore View Phase.<br>
public ViewDeclarationLanguage getViewDeclarationLanguageFromRawViewId(FacesContext context, String viewId)<br>
</blockquote>
<br></div>
Why not use ViewHandler.deriveViewId() and specify that ViewHandler.getViewDeclarationLanguage() takes a derived view id?<br><font color="#888888">
<br></font></blockquote><div><br>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.<br>
<br>Leonardo<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><font color="#888888">
Andy<br>
<br>
<br>
</font></blockquote></div><br>