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

BeSimple\SoapClient\WsdlDownloader deprecated curly braces #116

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

orediggerco
Copy link

Line 217 in the WsdlDownloader class has curly braces to access a string offset. This was deprecated (https://wiki.php.net/rfc/deprecate_curly_braces_array_access). This PR changes the curly braces to [].

…used for array elements and string offsets were deprecated in php 7.4. This fixes the error and is backwards compatible.
@wollanup
Copy link

+1
But I think nobody will merge this. This project seems DEAD.

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.

2 participants