[jboss-dev-forums] [Design of POJO Server] - Fixing the WarClassLoaderDeployer

[adrian@jboss.org]

The current way the WarClassLoaderDeployer works is all of the place. :-)

INCLUDE WEB-INF

I thought the requirement was to include the root of the war
since that is what the old UCL did?
Anyway this should be in the structure deployer not the classloader deployer.
Same goes for some of the other flags, is "acceptNonWarDirs" still used?

COPYING TO TEMP

Again this should be in the structure deployer. The VirtualFiles is adds
should be the temp ones.

ALT-DD

This doesn't look like it is handled properly in the new deployers?
Shouldn't this be handled in the metadata parsing?

PLAYING AROUND WITH METADATA

The merging and altering of metadata shouldn't be left to the "RealDeployer"
i.e. AbstractWarDeployer. Doing so means it cannot be overridden
by some other policy.

What's most annoying is that bad examples like this are being used
by the other people as the correct way to do things :-)

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4137137#4137137

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4137137