refactor: Reflection*::setAccessible() is now no-op in PHP 8.1 (#9331)

Author: paulbalandan

system/BaseModel.php

@@ -1799,8 +1799,6 @@ protected function objectToRawArray($object, bool $onlyChanged = true, bool $rec
             // Loop over each property,
             // saving the name/value in a new array we can return.
             foreach ($props as $prop) {
-                // Must make protected values accessible.
-                $prop->setAccessible(true);
                 $properties[$prop->getName()] = $prop->getValue($object);
             }
         }

system/Test/ReflectionHelper.php

@@ -39,8 +39,7 @@ trait ReflectionHelper
     public static function getPrivateMethodInvoker($obj, $method)
     {
         $refMethod = new ReflectionMethod($obj, $method);
-        $refMethod->setAccessible(true);
-        $obj = (gettype($obj) === 'object') ? $obj : null;
+        $obj       = (gettype($obj) === 'object') ? $obj : null;
 
         return static fn (...$args) => $refMethod->invokeArgs($obj, $args);
     }
@@ -59,10 +58,7 @@ private static function getAccessibleRefProperty($obj, $property)
     {
         $refClass = is_object($obj) ? new ReflectionObject($obj) : new ReflectionClass($obj);
 
-        $refProperty = $refClass->getProperty($property);
-        $refProperty->setAccessible(true);
-
-        return $refProperty;
+        return $refClass->getProperty($property);
     }
 
     /**

system/Traits/PropertiesTrait.php

@@ -71,7 +71,6 @@ final public function getNonPublicProperties(): array
                 continue;
             }
 
-            $property->setAccessible(true);
             $properties[] = $property;
         }
 

tests/system/CLI/CLITest.php

@@ -549,13 +549,11 @@ public function testParseCommandMultipleOptions(): void
     public function testWindow(): void
     {
         $height = new ReflectionProperty(CLI::class, 'height');
-        $height->setAccessible(true);
         $height->setValue(null, null);
 
         $this->assertIsInt(CLI::getHeight());
 
         $width = new ReflectionProperty(CLI::class, 'width');
-        $width->setAccessible(true);
         $width->setValue(null, null);
 
         $this->assertIsInt(CLI::getWidth());