Add support for "form" attribute

[GoogleCodeExporter]

Add support for element's "form" attribute.

Example:

<xs:element name="elem" form="qualified" type="xs:int" />

Original issue reported on code.google.com by loentar on 8 Jan 2013 at 12:01

[GoogleCodeExporter]

here goes a patch with a quick fix for this issue. on our test cases it worked 
quite well

Original comment by rafaelpi...@gmail.com on 14 Jan 2013 at 12:18

Attachments:

• serviceclient.patch

[GoogleCodeExporter]

I'm afraid this patch will make SOAP request invalid in case of attrubute 
form="qualified" is not set in wsdl. This is not a very correct solution.

Need to make some changes in WSDL plugin and codegen templates to add support 
for "form" attribute. Client's code doesn't need to be changed.

Original comment by loentar on 14 Jan 2013 at 1:06

[GoogleCodeExporter]

This issue was closed by revision r727.

Original comment by loentar on 19 Jan 2013 at 8:30

• Changed state: Fixed

[GoogleCodeExporter]

We still got some problems.

Tested with rev 759

The following class
class Filters {
public List<String> keys;
public List<String> values;
}

It is been serialized this way:

<soapenv:Envelope 
xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/"><soapenv:Body><ns0:Che
ck 
xmlns:ns0="http://test.org/"><ns0:Filters><ns0:keys>username</ns0:keys><ns0:keys
>level</ns0:keys><ns0:keys>computer_name</ns0:keys><ns0:values>tester@test.org</
ns0:values><ns0:values>31</ns0:values><ns0:values>test.local</ns0:values></ns0:F
ilters></ns0:Check></soapenv:Body></soapenv:Envelope>

While it should be more like this:

<soapenv:Envelope 
xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/"><soapenv:Body><ns0:Che
ck 
xmlns:ns0="http://test.org/"><ns0:Filters><keys>username</keys><keys>level</keys
><keys>computer_name</keys><values>tester@test.org</values><values>31</values><v
alues>test.local</values></ns0:Filters></ns0:Check></soapenv:Body></soapenv:Enve
lope>

The namespace preffix shold be appended only to types declared in that 
namespace.

Thanks for your quick reply and sorry for take me so long to test your patches. 
Keep up the good work!

Original comment by pedro.me...@gmail.com on 3 Apr 2013 at 7:24

[GoogleCodeExporter]

Original comment by loentar on 4 Apr 2013 at 6:20

• Changed state: Accepted

[GoogleCodeExporter]

This issue was closed by revision r767.

Original comment by loentar on 18 Apr 2013 at 8:36

• Changed state: Fixed

[GoogleCodeExporter]

Issue 202 has been merged into this issue.

Original comment by loentar on 2 May 2013 at 7:28

[GoogleCodeExporter]

Issue 202 has been merged into this issue.

Original comment by alexmantaut on 6 May 2013 at 1:47