[ 
https://issues.apache.org/jira/browse/DIGESTER-165?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Simone Tripodi resolved DIGESTER-165.
-------------------------------------

       Resolution: Fixed
    Fix Version/s: 3.3

Thanks a lot for contributing Dirk, I applied your patches, see r1352055.

Just few minor recommendations for future patches you'll submit, also in other 
ASF projects:

 * have a look at [On Contributing Patches 
|http://commons.apache.org/patches.html] on main Apache Commons site

 * it would be helpful provide one single patch which is name with the issue 
key, i.e. {{{DIGESTER-165}}}

 * please fix your svn settings, I had to fight against the _Inconsistent line 
ending style_ issue with {{dos2unix}}, see suggested [Native 
EOL|http://www.apache.org/dev/svn-eol-style.txt] style.

Hope to apply more patches from you!
                
> BinderClassLoader does not override getResource
> -----------------------------------------------
>
>                 Key: DIGESTER-165
>                 URL: https://issues.apache.org/jira/browse/DIGESTER-165
>             Project: Commons Digester
>          Issue Type: Bug
>    Affects Versions: 3.2
>            Reporter: Dirk Schaube
>            Assignee: Simone Tripodi
>             Fix For: 3.3
>
>         Attachments: BinderClassLoader.java.2.patch, 
> BinderClassLoader.java.patch, BinderClassLoaderTestCase.java.patch
>
>
> The class org.apache.commons.digester3.binder.BinderClassLoader adapts the 
> the actual classloader but does not override getResource and call getResource 
> of the adapted classloader. This method is called when using an include 
> element with a classpath url in a xml rules file (e.g. <include 
> url="classpath:/com/something/foo.xml" />).
> As the constructor of BinderClassLoader does not call super(parent) the 
> system class loader is the parent of the BinderClassLoader which is a problem 
> if you are e.g. in a web application. As result the xml file is not found 
> although it is in the classpath (and could be found by the adapted 
> classloader).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to