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

See Issue #2153 #2154

Closed
wants to merge 3 commits into from
Closed

See Issue #2153 #2154

wants to merge 3 commits into from

Conversation

nickhagen
Copy link
Contributor

@nickhagen nickhagen commented Nov 6, 2020

Remove resetting the parameters and options after executing the soap request.

fixes #2153

Remove resetting the parameters and options after executing the soap request.
@nickhagen nickhagen changed the title Issue #2153 See Issue #2153 Nov 6, 2020
Nicholas Hagen added 2 commits November 6, 2020 10:47
only reset the defaults if trace option is NOT set!
@nickhagen
Copy link
Contributor Author

Thank you for the clarification, for future reference:

It is indeed to use the same soap object for multiple requests, instead of forging a new one for each request.

If you pass "trace" as one of the options in the forge() request, it is considered a default option for the request, while set_option() is a runtime option, and only active for that request (execute call).

So you need to use

$soap = Request::forge('https://api.cvent.com/soap/V200611.asmx?WSDL', array('driver'=>'soap', 'trace' => true));

@nickhagen nickhagen closed this Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Soap Trace Options
1 participant