[jsr-314-open-mirror] [jsr-314-open] 490-XmlViews Processing JSPX files as Facelets

[Leonardo Uribe]

Hi

Yes, the changes looks good. In the patch I proposed:

https://javaserverfaces-spec-public.dev.java.net/nonav/issues/showattachment.cgi/308/fixResourceResolver-2.patch

It was proposed this method:

+    /**
+     * <p class="changed_added_2_1">Return an Array with the configured
file
+     * extensions that can be processed by any VDL returned by this
factory</p>
+     */
+    public String[] getConfiguredExtensions()
+    {
+        return null;
+    }

The idea is just allow VDLs to define which file extensions should process,
so the algorithm on ViewHandler.deriveView() could consume it. In this way
section 7.5.2 could be fixed, and open the way other VDL to define its own
mechanism to define file extensions, usually through web config params.
Maybe this point was missing from the minimal points proposed to solve.

Now, if each VDL has its own id, it should be provided a way to retrieve the
VDL from the factory using that id and it should be provided a way to
"iterate" in a ordered way through VDL implementations, in the order they
are processed.

regards,

Leonardo Uribe
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jsr-314-open-mirror/attachments/20101021/7e302a6f/attachment-0002.html