[ 
https://issues.apache.org/jira/browse/WODEN-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12572544#action_12572544
 ] 

John Kaputin commented on WODEN-65:
-----------------------------------

r631272
Committed Sagara's latest patches to woden65 branch.

Minor change - changed the printType method back to printTypes to reflect the 
element name <types>.  I have not finished reviewing the code yet, so will add 
feedback later. 

The new WSDLWriter testcase looks good. The failure is due to a problem in 
Woden, not the testcase. Woden does not yet have a completed implementation of 
the equals() method throughout the WSDL model. This might not be resolved 
unless/until we merge the Element and Component APIs, as per the API Review 
issue on the wiki.  I'll look at the testcase and consider alternative tests 
for the time being.

> Woden elements must have serialize methods
> ------------------------------------------
>
>                 Key: WODEN-65
>                 URL: https://issues.apache.org/jira/browse/WODEN-65
>             Project: Woden
>          Issue Type: Improvement
>          Components: Parser
>            Reporter: Eran Chinthaka
>            Assignee: Sagara Gunathunga 
>         Attachments: BaseWSDLWriter.java.txt, DOMWSDLWriter.java.txt, 
> Schema-error-out-put.wsdl, Schema-error.wsdl, WSDLWriterTest.java.txt
>
>
> When we load a WSDL in to Woden object model, there is no way to serialize 
> the object model in to an output stream. This is very much useful when some 
> one wants to make sure Woden can properly spit out what it has read. 
> And there is a requirement from Axis2 also to make Woden element serializable 
> to cater for ?wsdl2 requests. 

-- 
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