Do not call Reflection*::setAccessible() #1726

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Open

W0rma wants to merge 1 commit into symfony:1.x from W0rma:reflection-set-accessible

W0rma commented Aug 11, 2025

As of PHP 8.1.0, calling this method has no effect; all methods are invokable by default.

(https://www.php.net/manual/en/reflectionmethod.setaccessible.php and https://www.php.net/manual/en/reflectionproperty.setaccessible.php).

There're even plans to deprecate the method in PHP 8.5: https://wiki.php.net/rfc/deprecations_php_8_5#extreflection_deprecations


          Remove noop setAccessible() call

c55d785

lyrixx approved these changes

View reviewed changes

Member

lyrixx left a comment

👍🏼 Thanks

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet