> I want a distinction between directories to be scanned, and URLs to be > deployed. This goes back to an earlier patch (that I never checked in) for > URLDeploymentScanner. If you specify a URL that is the base of an exploded > archive, then currently the scanner scans that directory. It should, > instead, deploy the directory.
Right, so like I said before I think this should go into another scanner. > The solution you presented at the time was to create an alternate scanner > for this purpose. I don't like that since it violates the concept of > exploded archives being treated just like their packaged counterparts. What? No it does not at all. It is just the same, it is just a matter of which scanner it came from. For example, if using a cache in between UDS and MD it now has to know how to handle a directory... blah, screw that. Make a seperate scanner, perhaps if you must bring DirectoryDeploymentScanner back to life and put in all of your directory specific stuff. It has no business in UDS, perhaps even the bit for scanning a directory url for files has no business in there. Please do not complicate UDS when you can create a simple alternative DS which can be used in replacement or in conjunction with the UDS. The system is designed to take more than one DS for just this reason... to allow for different scanning behavior. This is why I seperated the common scanning loop code into ADS. We do not want to get back into the monolithic component business again. Design with plugablility then make use of it when you need new functionality. If required refactor the existing components to better seperate the plugability... but don't go making simple components complicated when you could have easily created a replacment or peer to gain the same result. *sigh* --jason ------------------------------------------------- This mail sent through IMP: http://horde.org/imp/ _______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development