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

Feature: Added support for "literal wrapped". #100

Merged
merged 1 commit into from
Mar 9, 2023

Conversation

deivide
Copy link
Contributor

@deivide deivide commented Mar 7, 2023

I used the https://github.com/DimitriCodeurs/nusoap package for a few years, but this package has been unsupported for 6 years, forcing me to create my own fork and make the necessary adjustments.

With that, I decided to migrate to a package with more contributors, where I can also contribute with it, but to be able to use this package in my use case, I needed to add a few lines of code taken from the old package, which adapts the library to my needs.

Basically, it is an adaptation to the needs of some specific SOAP servers, where no nsPrefix is foreseen.
I don't know if I was able to explain this feature correctly, but the code can speak for itself:

// wrap document/literal wrapped calls with operation element
if ($usewrapped) {
  // (This code block was based on http://www.ibm.com/developerworks/webservices/library/ws-whichwsdl/
  // and tailored to the needs of one specific SOAP server, where no nsPrefix was seen...
  $this->debug("wrapping document request with literal method element");
  
  if ($namespace) {
    $payload = "<$operation xmlns=\"$namespace\">" .
	    $payload .
             "</$operation>";
  } else {
    $payload = "<$operation>" . $payload . "</$operation>";
  }
}

@f3l1x f3l1x merged commit a5fd9cb into contributte:master Mar 9, 2023
@f3l1x
Copy link
Member

f3l1x commented Mar 9, 2023

Thank you

@deivide deivide deleted the feature/literal-wrapped branch March 9, 2023 12:06
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