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

Add context property #270

Merged
merged 1 commit into from
Feb 23, 2022
Merged

Add context property #270

merged 1 commit into from
Feb 23, 2022

Conversation

neclimdul
Copy link
Contributor

Streams are expected to have a context property to be populated with the
current context. Starting with PHP 8.2, not including this property with emit a
deprecation warning.

Streams are expected to have a context property to be populated with the
current context. Starting with PHP 8.2, not including this property with emit a
deprecation warning.
@neclimdul
Copy link
Contributor Author

A bit early since PHP 8.2 isn't released but the dynamic property changes where already merged and I ran into this doing some early testing of some different projects.

@neclimdul
Copy link
Contributor Author

also PHP documentation of this property https://www.php.net/manual/en/class.streamwrapper.php#streamwrapper.props.context

@allejo allejo self-assigned this Feb 17, 2022
@allejo allejo merged commit 20e11ff into bovigo:master Feb 23, 2022
@allejo
Copy link
Member

allejo commented Feb 23, 2022

Backported to v1.6-dev: 17b0f39
Backported to v1.7-dev: 7a20ce2

@andypost
Copy link

Please create new release with the fix included

@allejo
Copy link
Member

allejo commented Jun 27, 2022

Please create new release with the fix included

Tagged v1.6.11-alpha.0 😄

@neclimdul neclimdul deleted the context-property branch June 27, 2022 04:25
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.

3 participants