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

Fix Issue #26 Strict Typing #80

Merged
merged 49 commits into from Nov 24, 2017
Merged

Fix Issue #26 Strict Typing #80

merged 49 commits into from Nov 24, 2017

Conversation

tolbon
Copy link
Contributor

@tolbon tolbon commented Oct 4, 2017

Fix issue #26

@veewee
Copy link
Contributor

veewee commented Oct 10, 2017

Thanks for your work on this @tolbon!
I'll have to test it out in some of our implementations to make sure it doesn't break anything.
This will take me some time.

@janvernieuwe janvernieuwe added this to the 0.6.0 milestone Oct 27, 2017
@janvernieuwe
Copy link
Member

janvernieuwe commented Oct 27, 2017

Tested it on my implementation and looked over the changes LGTM!

@janvernieuwe
Copy link
Member

Fixed merge conflicts on this branch.

@veewee veewee mentioned this pull request Nov 24, 2017
…olbon-master

# Conflicts:
#	src/Phpro/SoapClient/CodeGenerator/Assembler/FluentSetterAssembler.php
#	src/Phpro/SoapClient/CodeGenerator/Assembler/GetterAssembler.php
#	src/Phpro/SoapClient/CodeGenerator/Config/Config.php
#	src/Phpro/SoapClient/Middleware/BasicAuthMiddleware.php
#	src/Phpro/SoapClient/Middleware/CollectLastRequestInfoMiddleware.php
#	src/Phpro/SoapClient/Middleware/Middleware.php
#	src/Phpro/SoapClient/Middleware/MiddlewareInterface.php
#	src/Phpro/SoapClient/Middleware/NtlmMiddleware.php
#	src/Phpro/SoapClient/Middleware/RemoveEmptyNodesMiddleware.php
#	src/Phpro/SoapClient/Middleware/WsaMiddleware.php
#	src/Phpro/SoapClient/Middleware/Wsdl/DisableExtensionsMiddleware.php
#	src/Phpro/SoapClient/Middleware/WsseMiddleware.php
#	src/Phpro/SoapClient/Wsdl/Provider/GuzzleWsdlProvider.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants