23.07.2001 06:48:20, "Remy Maucherat" <[EMAIL PROTECTED]> wrote:
>> So, I'd like to have the GetMethod class 'dispatch' GET requests on
>> collections back to the servlets doGet() method (the standard
>> implementation of doGet() would be to return a FORBIDDEN response
>> code, or generate the standard directory-index). Similarly, POST
>> requests on collections should also be handled by the servlets
>> doPost() method. Both doGet() and doPost() would not get called
>> when non-collection resources are requested.
>>
>> Does that make sense ?
>
>Yes, but a possible side effect is that it creates some small
>namespace conflicts. For example, if you have a /fancy/fancydir.jsp
>to display the collections, then these paths cannot exist in the
>Slide namespace. It's not that bad, though.
That's true. I'm using the approach of having some JSP pages in WEB-
INF/include that I include in from my WebdavServlet subclass via
RequestDispatcher. This is working very well so far, albeit I'm not
using the SlideDirContext (and I have no idea how that actually works,
never tested it).
So, the changes I've outlined above are pretty much done, but rather
extensive: I've moved all the HTML generation code into the
WebdavServlet, and needed to make a couple of methods in WebdavMethod
static and/or public (e.g. isCollection()). Everything is completely
backwards compatible though, and I've not had any problems so far
(which is pretty unusual for me - I seem to attract problems ;>).
I'll go through everything and clean up my changes, and commit a patch
later.
On a related note, I've started with a JSP taglib for Slide (because I
needed one). It is based on and uses JSPTL/EA1 (JSR 52,
http://jakarta.apache.org/taglibs/doc/jsptl-doc/intro.html), which of
course is in a fairly early state, but looks very promising and works
very well IMHO. Would there be interest in incorporating that into the
Slide-CVS at some point (meaning: even though it uses JSPTL) ?
-chris
________________________________________________________________
cmlenz at gmx.de