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

PHP 8.3 | HasCapabilitiesTest: fix deprecation notices for ReflectionProperty::setValue() #818

Merged
merged 1 commit into from
Jul 24, 2023

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Jul 20, 2023

The ReflectionProperty::setValue() method supports three method signatures, two of which are deprecated as of PHP 8.3.

This adjusts the call to ReflectionProperty::setValue() in the HasCapabilitiesTest to pass null as the "object" for setting the value of a static property to make the method call cross-version compatible.

Ref: https://wiki.php.net/rfc/deprecate_functions_with_overloaded_signatures#reflectionpropertysetvalue

…Property::setValue()

The `ReflectionProperty::setValue()` method supports three method signatures, two of which are deprecated as of PHP 8.3.

This adjusts the call to `ReflectionProperty::setValue()` in the `HasCapabilitiesTest` to pass `null` as the "object" for setting the value of a static property to make the method call cross-version compatible.

Ref: https://wiki.php.net/rfc/deprecate_functions_with_overloaded_signatures#reflectionpropertysetvalue
@jrfnl jrfnl force-pushed the feature/php-8.3-fix-test-compat branch from 12713d1 to fcc5625 Compare July 24, 2023 09:04
@jrfnl jrfnl mentioned this pull request Jul 24, 2023
6 tasks
@schlessera schlessera merged commit 6117828 into develop Jul 24, 2023
29 checks passed
@schlessera schlessera deleted the feature/php-8.3-fix-test-compat branch July 24, 2023 09:31
@jrfnl jrfnl modified the milestones: 2.0.x Next, 2.1.0 Sep 11, 2023
@jrfnl jrfnl modified the milestones: 2.1.0, 2.0.10 Jan 8, 2024
@jrfnl jrfnl mentioned this pull request Jan 8, 2024
5 tasks
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.

2 participants