Re: svn commit: r532422 [1/2] - in /webservices/axis2/trunk/java/modules/kernel: src/org/apache/axis2/context/ src/org/apache/axis2/deployment/ src/org/apache/axis2/deployment/repository/util/ src/org/apache/axis2/deployment/util/ src/org/apache/axis

[Thilina Gunarathne <cs...@gmail.com>]

Hi Glen,
> * Refactor deployment a bit to clean things up and get custom deployers actually working.
I've been using an custom deployer successfully for an outside project
(WSO2 Mashup Server).. But this commit broke it:(..

The mapping I had to add was with the "." in the extension.. But now
you seems to have removed the need for the ".".. I'm fine with that..
But it would have been great if you sent a mail noticing this (may
have saved several debugging hours for me)..

Also the hotupdate feature, which was working before was gone..
Following hard coding of type as service makes it unusable with any
custom deployers...
>+    public static final int TYPE_SERVICE = 0;
>+    public static final int TYPE_MODULE = 1;
>+
>    /**
>     * To check whether the file is a module or a servise
>     */
>-    private String type;
>+    private int type = TYPE_SERVICE;

BTW +1 for adding a test case for the custom deployers..

Thanks,
Thilina

-- 
Thilina Gunarathne  -  http://www.wso2.com - http://thilinag.blogspot.com

---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org