chore: As of PHP 8.1.0, calling this method has no effect; all properties are accessible by default. Symfony 7.3 requires PHP 8.2+.

NickSdot · NickSdot · commit e3fdd2b97939 · 2025-07-23T22:03:03.000+07:00
https://www.php.net/manual/en/reflectionproperty.setaccessible.php
diff --git a/src/Symfony/Component/DependencyInjection/Tests/ContainerTest.php b/src/Symfony/Component/DependencyInjection/Tests/ContainerTest.php
@@ -420,7 +420,6 @@ public function testGetEnvDoesNotAutoCastNullWithDefaultEnvVarProcessor()
         $container->compile();
 
         $r = new \ReflectionMethod($container, 'getEnv');
-        $r->setAccessible(true);
         $this->assertNull($r->invoke($container, 'FOO'));
     }
 
@@ -436,7 +435,6 @@ public function testGetEnvDoesNotAutoCastNullWithEnvVarProcessorsLocatorReturnin
         $container->compile();
 
         $r = new \ReflectionMethod($container, 'getEnv');
-        $r->setAccessible(true);
         $this->assertNull($r->invoke($container, 'FOO'));
     }
 }
diff --git a/src/Symfony/Component/ErrorHandler/Tests/ErrorHandlerTest.php b/src/Symfony/Component/ErrorHandler/Tests/ErrorHandlerTest.php
@@ -34,7 +34,6 @@ class ErrorHandlerTest extends TestCase
     protected function tearDown(): void
     {
         $r = new \ReflectionProperty(ErrorHandler::class, 'exitCode');
-        $r->setAccessible(true);
         $r->setValue(null, 0);
     }
 
diff --git a/src/Symfony/Component/Mailer/Tests/Transport/SendmailTransportTest.php b/src/Symfony/Component/Mailer/Tests/Transport/SendmailTransportTest.php
@@ -94,7 +94,6 @@ public function testThrowsTransportExceptionOnFailure()
         $sendmailTransport->send($mail, $envelope);
 
         $streamProperty = new \ReflectionProperty(SendmailTransport::class, 'stream');
-        $streamProperty->setAccessible(true);
         $stream = $streamProperty->getValue($sendmailTransport);
         $this->assertNull($stream->stream);
     }
@@ -112,10 +111,8 @@ public function testStreamIsClearedOnFailure()
         }
 
         $streamProperty = new \ReflectionProperty(SendmailTransport::class, 'stream');
-        $streamProperty->setAccessible(true);
         $stream = $streamProperty->getValue($sendmailTransport);
         $innerStreamProperty = new \ReflectionProperty(ProcessStream::class, 'stream');
-        $innerStreamProperty->setAccessible(true);
         $this->assertNull($innerStreamProperty->getValue($stream));
     }
 
@@ -127,7 +124,6 @@ public function testDoesNotThrowWhenInteractive()
 
         $sendmailTransport = new SendmailTransport(self::FAKE_INTERACTIVE_SENDMAIL);
         $transportProperty = new \ReflectionProperty(SendmailTransport::class, 'transport');
-        $transportProperty->setAccessible(true);
 
         // Replace the transport with an anonymous consumer that trigger the stream methods
         $transportProperty->setValue($sendmailTransport, new class($transportProperty->getValue($sendmailTransport)->getStream()) extends SmtpTransport {

Original file line number	Diff line number	Diff line change
`@@ -420,7 +420,6 @@ public function testGetEnvDoesNotAutoCastNullWithDefaultEnvVarProcessor()`
`420`	`420`	`$container->compile();`
`421`	`421`
`422`	`422`	`$r = new \ReflectionMethod($container, 'getEnv');`
`423`		`- $r->setAccessible(true);`
`424`	`423`	`$this->assertNull($r->invoke($container, 'FOO'));`
`425`	`424`	`}`
`426`	`425`
`@@ -436,7 +435,6 @@ public function testGetEnvDoesNotAutoCastNullWithEnvVarProcessorsLocatorReturnin`
`436`	`435`	`$container->compile();`
`437`	`436`
`438`	`437`	`$r = new \ReflectionMethod($container, 'getEnv');`
`439`		`- $r->setAccessible(true);`
`440`	`438`	`$this->assertNull($r->invoke($container, 'FOO'));`
`441`	`439`	`}`
`442`	`440`	`}`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,6 @@ class ErrorHandlerTest extends TestCase`
`34`	`34`	`protected function tearDown(): void`
`35`	`35`	`{`
`36`	`36`	`$r = new \ReflectionProperty(ErrorHandler::class, 'exitCode');`
`37`		`- $r->setAccessible(true);`
`38`	`37`	`$r->setValue(null, 0);`
`39`	`38`	`}`
`40`	`39`