Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Workaround PHP bug #63206 in Stream::setStream method #314

Merged
merged 1 commit into from
Jul 19, 2018
Merged

Workaround PHP bug #63206 in Stream::setStream method #314

merged 1 commit into from
Jul 19, 2018

Conversation

ste93cry
Copy link
Contributor

@ste93cry ste93cry commented Jul 9, 2018

Q A
Branch? master
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes

This PR fixes #298 by removing the usage of the second argument of the set_error_handler function and moves the check of the level of the raised error inside the error handler callback.

@weierophinney
Copy link
Member

@ste93cry Umm... that's not the PR template we use... and the package is licensed BSD-3-Clause; we're not in a position to dual-license. Can you clarify the license before we merge, please?

@ste93cry
Copy link
Contributor Author

Sorry I copy-pasted a template from another repository I used to contribute to. My code is released under any license you use, so feel free to do whatever you want.

@weierophinney weierophinney merged commit ff219ca into zendframework:master Jul 19, 2018
weierophinney added a commit that referenced this pull request Jul 19, 2018
weierophinney added a commit that referenced this pull request Jul 19, 2018
weierophinney added a commit that referenced this pull request Jul 19, 2018
@weierophinney
Copy link
Member

Thanks, @ste93cry!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stream::setStream method causes PHP bug #63206
2 participants