minor #62375 [PropertyInfo] ReflectionMethod::setAccessible() is no-op since PHP 8.1 (xabbuh)

nicolas-grekas · nicolas-grekas · commit 17c1070fbf07 · 2025-11-13T08:59:31.000+01:00
This PR was merged into the 6.4 branch. Discussion ---------- [PropertyInfo] ReflectionMethod::setAccessible() is no-op since PHP 8.1 | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | | License | MIT added in #62334, this allows to entirely remove it after merging into `7.3` like we did everywhere else Commits ------- ef9e803 ReflectionMethod::setAccessible() is no-op since PHP 8.1
diff --git a/src/Symfony/Component/PropertyInfo/Tests/Extractor/ReflectionExtractorTest.php b/src/Symfony/Component/PropertyInfo/Tests/Extractor/ReflectionExtractorTest.php
@@ -806,10 +806,6 @@ public function testCamelize(string $input, string $expected)
         $reflection = new \ReflectionClass($this->extractor);
         $method = $reflection->getMethod('camelize');
 
-        if (\PHP_VERSION_ID < 80500) {
-            $method->setAccessible(true);
-        }
-
         $this->assertSame($expected, $method->invoke($this->extractor, $input));
     }
 

Original file line number	Diff line number	Diff line change
`@@ -806,10 +806,6 @@ public function testCamelize(string $input, string $expected)`
`806`	`806`	`$reflection = new \ReflectionClass($this->extractor);`
`807`	`807`	`$method = $reflection->getMethod('camelize');`
`808`	`808`
`809`		`- if (\PHP_VERSION_ID < 80500) {`
`810`		`- $method->setAccessible(true);`
`811`		`- }`
`812`		`-`
`813`	`809`	`$this->assertSame($expected, $method->invoke($this->extractor, $input));`
`814`	`810`	`}`
`815`	`811`