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

John Kaputin closed WODEN-119.
------------------------------

    Resolution: Fixed

Close with no further action. Discussed on the weekly call 23 Jan 07 and agreed.

> readWSDL method args should be java.net.URI instead of String
> -------------------------------------------------------------
>
>                 Key: WODEN-119
>                 URL: https://issues.apache.org/jira/browse/WODEN-119
>             Project: Woden
>          Issue Type: Improvement
>          Components: Parser
>            Reporter: John Kaputin
>         Assigned To: John Kaputin
>            Priority: Minor
>             Fix For: M7
>
>
> I propose changing the WSDLReader API so that the readWSDL methods that take 
> a wsdl location uri argument use java.net.URI instead of String.
> So the WSDLReader methods:
>     readWSDL(String wsdlURI)
>     readWSDL(String wsdlURI, ErrorHandler errorHandler)
> become
>     readWSDL(URI wsdlURI)
>     readWSDL(URI wsdlURI, ErrorHandler errorHandler)
> One of our design goals for the Woden API was to consistently use 
> java.net.URI instead of java.lang.String to represent URIs, so as to achieve 
> a measure of type safety at the API boundary. Elsewhere the Woden API uses 
> java.net.URI - the two readWSDL methods that take a String as the wsdl uri 
> are the only exceptions and these are a left over from my early prototyping 
> that was based on WSDL4J. Changing these two methods to take a java.net.URI 
> argument will make the Woden API consistent in this regard and will make it 
> more robust by ensuring the WSDL uri passed to the reader is correctly 
> formatted as a URI (because the Woden client must instantiate the 
> java.net.URI object).
> As discussed previously, we are delivering Milestone releases to get code out 
> quickly to early adopters, but we have not yet declared the Woden API 
> 'stable'. This will happen with a 1.0 release. However, to minimize 
> disruption to existing Woden users, these two readWSDL methods could be 
> changed via a deprecation strategy for M7 - that is, add the new method 
> signatures to WSDLReader and deprecate the existing ones, then remove those 
> deprecated methods for the release that follows M7.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to