You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by "Vijay Pandey (JIRA)" <ji...@apache.org> on 2006/06/28 16:36:30 UTC

[jira] Created: (VALIDATOR-195) Lodaing of Digester Rules for classes extending ValidatorResources does not work

Lodaing of Digester Rules for classes extending  ValidatorResources does not work
---------------------------------------------------------------------------------

         Key: VALIDATOR-195
         URL: http://issues.apache.org/jira/browse/VALIDATOR-195
     Project: Commons Validator
        Type: Bug

    Versions: 1.3.0 Release, 1.2.0 Release    
 Environment: Windows/Unix
    Reporter: Vijay Pandey


If we extend the class "ValidatorResources", then the loading of the digester rules does not work properly.

---------------------------------ValidatorResources---------------------------------------------------------
    private Digester initDigester() {
        URL rulesUrl = this.getClass().getResource("digester-rules.xml");
-----------------------------------------------------------------------------------------------------------------------
The above line tries to find the "digester-rules.xml" from the path of extending class and hence to make this work we need to
copy the digester rules xml file in the same pacakge as the extending class.

I think a better way might be is to accept a InputStream or URL for the digester rules in the constructor if someone wants to 
have their custom digester rules, and by default if it's not provided it should always take from the ValidatorResources path
by changing the code 

this.getClass().getResource("digester-rules.xml");

to 

ValidatorResources.class.getResource("digester-rules.xml");

Thanks
Vijay






 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


[jira] Updated: (VALIDATOR-195) Lodaing of Digester Rules for classes extending ValidatorResources does not work

Posted by "Niall Pemberton (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/VALIDATOR-195?page=all ]

Niall Pemberton updated VALIDATOR-195:
--------------------------------------

    Component/s: Framework

> Lodaing of Digester Rules for classes extending  ValidatorResources does not work
> ---------------------------------------------------------------------------------
>
>                 Key: VALIDATOR-195
>                 URL: http://issues.apache.org/jira/browse/VALIDATOR-195
>             Project: Commons Validator
>          Issue Type: Bug
>          Components: Framework
>    Affects Versions: 1.3.0 Release, 1.2.0 Release
>         Environment: Windows/Unix
>            Reporter: Vijay Pandey
>
> If we extend the class "ValidatorResources", then the loading of the digester rules does not work properly.
> ---------------------------------ValidatorResources---------------------------------------------------------
>     private Digester initDigester() {
>         URL rulesUrl = this.getClass().getResource("digester-rules.xml");
> -----------------------------------------------------------------------------------------------------------------------
> The above line tries to find the "digester-rules.xml" from the path of extending class and hence to make this work we need to
> copy the digester rules xml file in the same pacakge as the extending class.
> I think a better way might be is to accept a InputStream or URL for the digester rules in the constructor if someone wants to 
> have their custom digester rules, and by default if it's not provided it should always take from the ValidatorResources path
> by changing the code 
> this.getClass().getResource("digester-rules.xml");
> to 
> ValidatorResources.class.getResource("digester-rules.xml");
> Thanks
> Vijay
>  

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


[jira] Resolved: (VALIDATOR-195) Lodaing of Digester Rules for classes extending ValidatorResources does not work

Posted by "Niall Pemberton (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/VALIDATOR-195?page=all ]

Niall Pemberton resolved VALIDATOR-195.
---------------------------------------

    Fix Version/s: 1.3.1
       Resolution: Fixed
         Assignee: Niall Pemberton

Changing ValidatorResources to always load the digester rules from the "org.apache.commons.validator" package could break behaviour for someone relying on this (i.e. if they're plugging in their own custom Digester rules) - so I don't want to make the change you requested.

I have however modified ValidatorResources to use the digester rules in the "org.apache.commons.validator" package if no rules are found in the default package for the class - this should resolve your issue and retain backwards compatibility:

    http://svn.apache.org/viewvc?view=rev&revision=478392

Thanks for reporting this

Niall



> Lodaing of Digester Rules for classes extending  ValidatorResources does not work
> ---------------------------------------------------------------------------------
>
>                 Key: VALIDATOR-195
>                 URL: http://issues.apache.org/jira/browse/VALIDATOR-195
>             Project: Commons Validator
>          Issue Type: Bug
>          Components: Framework
>    Affects Versions: 1.3.0 Release, 1.2.0 Release
>         Environment: Windows/Unix
>            Reporter: Vijay Pandey
>         Assigned To: Niall Pemberton
>             Fix For: 1.3.1
>
>
> If we extend the class "ValidatorResources", then the loading of the digester rules does not work properly.
> ---------------------------------ValidatorResources---------------------------------------------------------
>     private Digester initDigester() {
>         URL rulesUrl = this.getClass().getResource("digester-rules.xml");
> -----------------------------------------------------------------------------------------------------------------------
> The above line tries to find the "digester-rules.xml" from the path of extending class and hence to make this work we need to
> copy the digester rules xml file in the same pacakge as the extending class.
> I think a better way might be is to accept a InputStream or URL for the digester rules in the constructor if someone wants to 
> have their custom digester rules, and by default if it's not provided it should always take from the ValidatorResources path
> by changing the code 
> this.getClass().getResource("digester-rules.xml");
> to 
> ValidatorResources.class.getResource("digester-rules.xml");
> Thanks
> Vijay
>  

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org