Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for rpc encoded wsdl #744

Merged
merged 1 commit into from
Mar 4, 2016
Merged

Add support for rpc encoded wsdl #744

merged 1 commit into from
Mar 4, 2016

Conversation

fernandes
Copy link
Contributor

After Wasabi add support for multi part on messages, now it's returning a hash
with parts, and Savon needs to handle correctly.

So on builder, instead of just add a single message to body, now handle if it's
a hash and build the body containing the respective parts, with its type and
arguments passed via locals variable.

This way we add support for builder create a message that contains multi parts.

check: savonrb/wasabi#63 for wasabi support on
rpc/encoded wsdl.

disclaimer: I'm totally new to SOAP, sorry if I misunderstood any concept.
I tried to implement touch as few places as possible on current code base.

After Wasabi add support for multi part on messages, now it's returning a hash
with parts, and Savon needs to handle correctly.

So on builder, instead of just add a single message to body, now handle if it's
a hash and build the body containing the respective parts, with its type and
arguments passed via `locals` variable.

This way we add support for builder create a message that contains multi parts.
@fernandes
Copy link
Contributor Author

build is broken because it's using wasabi latest release that do not include rcp/encoded support

tjarratt added a commit that referenced this pull request Mar 4, 2016
@tjarratt tjarratt merged commit 34edc4c into savonrb:master Mar 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants