commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Craig McClanahan (JIRA)" <j...@apache.org>
Subject [jira] Commented: (VALIDATOR-209) Additional constructor for ValidatorResources that takes URL[] instead of String[]
Date Thu, 23 Nov 2006 03:56:03 GMT
    [ http://issues.apache.org/jira/browse/VALIDATOR-209?page=comments#action_12452107 ] 
            
Craig McClanahan commented on VALIDATOR-209:
--------------------------------------------

> I just had a look at the Digester javadoc but I don't see a parse() method
> that takes a URL - just a String uri. I guess I must be missing something,
> can you point me in the right direction?

Grumble ... you're right, of course.  I thought this had been taken care of earlier in Digester.

I'll file an issue against Digester and make this one dependent on it.  I suspect it won't
be able to get in to Digester 1.8, but there's always 1.9 ...




> 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
>            Priority: Minor
>
> 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: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message