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.

Sign up for GitHub

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

Jump to bottom

support recursive complex types #86

Closed
wants to merge 1 commit into from
Closed

support recursive complex types #86

wants to merge 1 commit into from

Conversation

sastan
Copy link
Contributor

@sastan sastan commented Sep 17, 2012

This patch allows to use complex types which are recursive (ie have a property or child which points to itself).

@chusi
Copy link

chusi commented Nov 7, 2013

i need this feature as well. unfortunately it's quite a while ago since you wrote the patch. i tried to apply it to the latest version, however just with medium success. it works partially for my case, however it might also be due to some quite broken wdsl....
any body else interested in this feature or any body know a lib for nodejs that can handle this well?

@sastan
Copy link
Contributor Author

sastan commented Nov 7, 2013

Sorry. Still using that old branch as it works for us.

@chusi
Copy link

chusi commented Nov 7, 2013

thanks, i tried your branch and i found that my problem lies deeper. it's not just a problem with recursive complex types but in my case they are even from different targetNamespaces. and it looks like this is generally not really supported (afaik)

@aheckmann
Copy link
Contributor

Closing due to it being unmergeable and missing tests. Please reopen after a rebase and tests are added. Thank you.

@aheckmann aheckmann closed this Jan 23, 2014
christiaanwesterbeek pushed a commit to christiaanwesterbeek/node-soap that referenced this pull request Jun 30, 2014
@christiaanwesterbeek
Fix and test for #257
07838f5 
* Fix for running mocha tests in Windows.
* Add a wsdl that fails during describe.

Bug vpulim#86 also mentions recursive complex types whitch this is all about.
diarmaidm pushed a commit to diarmaidm/node-soap that referenced this pull request Feb 3, 2016
Fix and test for #257
65e47ce 
* Fix for running mocha tests in Windows.
* Add a wsdl that fails during describe.

Bug vpulim#86 also mentions recursive complex types whitch this is all about.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sastan @chusi @aheckmann