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