[ http://issues.apache.org/jira/browse/VALIDATOR-209?page=all ]

Niall Pemberton resolved VALIDATOR-209.
---------------------------------------

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

Fixed thanks

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

> Additional constructor for ValidatorResources that takes URL[] instead of 
> String[]
> ----------------------------------------------------------------------------------
>
>                 Key: VALIDATOR-209
>                 URL: http://issues.apache.org/jira/browse/VALIDATOR-209
>             Project: Commons Validator
>          Issue Type: Improvement
>          Components: Framework
>    Affects Versions: 1.3.0 Release
>            Reporter: Craig McClanahan
>         Assigned To: Niall Pemberton
>            Priority: Minor
>             Fix For: 1.3.1
>
>         Attachments: validator-209-ValidatorResources.patch
>
>
> Currently, the constructor for ValidatorResources takes an InputStream, and 
> array of InputStream, a single String, or an array of Strings (in the latter 
> two cases, the strings are assumed to be URIs of either webapp resources or 
> classpath resources to be parsed).  In a web application environment, a 
> framework or application using Commons Validator will typically use either 
> ServletContext.getResource() or Class.getResource() to find URLs of the set 
> of resources to be configured.
> However, the CommonsValidator constructor cannot take URLs correctly.  
> Therefore, the caller will need to convert these URLs to external (String) 
> form in order to pass them in.  However, these Strings will ultimately need 
> to be turned back into URLs anyway (inside the Digester instance being used), 
> in order for relative references to work.
> Thus, the current implementation assumes that there is a lossless conversion 
> from a URL returned by ServletContext.getResource() or Class.getResource(), 
> to a String, and then back to a URL.  That assumption is *not* necessarily 
> guaranteed for the servlet context resources (although it is generally the 
> case in practice for most containers).  It is legal for the container to 
> embed information (such as a custom URLStreamHandler implementation) inside 
> the URLs it returns for webapp resources.
> It would be better defensive coding for ValidatorResources to accept an array 
> of URLs of the resources to be loaded (and pass them directly in to Digester 
> unchanged), in addition to the other constructors that are currently 
> supported.

-- 
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to