refactor: setAccessible no longer effect since PHP 8.1

Author: ddevsr

build/kint.phar

@@ -136,7 +136,9 @@ private function buildStaticValue(ReflectionProperty $pr, int $depth): AbstractV
             $context->access_path = '\\'.$context->owner_class.'::$'.$context->name;
         }
 
-        $pr->setAccessible(true);
+        if (KINT_PHP81 === false) {
+            $pr->setAccessible(true);
+        }
 
         /**
          * @psalm-suppress TooFewArguments

src/Parser/ClassStaticsPlugin.php

@@ -401,7 +401,10 @@ private function parseObject(object &$var, ContextInterface $c): AbstractValue
             $properties = [];
 
             foreach ($props as $rprop) {
-                $rprop->setAccessible(true);
+                if (KINT_PHP81 === false) {
+                    $rprop->setAccessible(true);
+                }
+
                 $name = $rprop->getName();
 
                 // Casting object to array:

src/Parser/Parser.php

@@ -54,17 +54,28 @@ public function testConstruct()
 
         $reflector = new ReflectionClass($d);
         $mprop = $reflector->getProperty('methods_plugin');
-        $mprop->setAccessible(true);
+
+        if (KINT_PHP81 === false) {
+            $mprop->setAccessible(true);
+        }
+
         $m = $mprop->getValue($d);
         $this->assertInstanceOf(ClassMethodsPlugin::class, $m);
         $sprop = $reflector->getProperty('statics_plugin');
-        $sprop->setAccessible(true);
+
+        if (KINT_PHP81 === false) {
+            $sprop->setAccessible(true);
+        }
+
         $s = $sprop->getValue($d);
         $this->assertInstanceOf(ClassStaticsPlugin::class, $s);
 
         $reflector = new ReflectionClass(AbstractPlugin::class);
         $aparser = $reflector->getProperty('parser');
-        $aparser->setAccessible(true);
+
+        if (KINT_PHP81 === false) {
+            $aparser->setAccessible(true);
+        }
 
         $p = new Parser();
         $this->assertNotSame($p, $aparser->getValue($m));

tests/Parser/ClassStringsPluginTest.php

@@ -115,17 +115,28 @@ public function testConstruct()
 
         $reflector = new ReflectionClass($d);
         $mprop = $reflector->getProperty('methods_plugin');
-        $mprop->setAccessible(true);
+
+        if (KINT_PHP81 === false) {
+            $mprop->setAccessible(true);
+        }
+
         $m = $mprop->getValue($d);
         $this->assertInstanceOf(ClassMethodsPlugin::class, $m);
         $sprop = $reflector->getProperty('statics_plugin');
-        $sprop->setAccessible(true);
+
+        if (KINT_PHP81 === false) {
+            $sprop->setAccessible(true);
+        }
+
         $s = $sprop->getValue($d);
         $this->assertInstanceOf(ClassStaticsPlugin::class, $s);
 
         $reflector = new ReflectionClass(AbstractPlugin::class);
         $aparser = $reflector->getProperty('parser');
-        $aparser->setAccessible(true);
+
+        if (KINT_PHP81 === false) {
+            $aparser->setAccessible(true);
+        }
 
         $p = new Parser();
         $this->assertNotSame($p, $aparser->getValue($m));

tests/Parser/DomPluginTest.php

@@ -87,13 +87,20 @@ public function testConstruct()
 
         $reflector = new ReflectionClass($s);
         $mprop = $reflector->getProperty('methods_plugin');
-        $mprop->setAccessible(true);
+
+        if (KINT_PHP81 === false) {
+            $mprop->setAccessible(true);
+        }
+
         $m = $mprop->getValue($s);
         $this->assertInstanceOf(ClassMethodsPlugin::class, $m);
 
         $reflector = new ReflectionClass(AbstractPlugin::class);
         $mparser = $reflector->getProperty('parser');
-        $mparser->setAccessible(true);
+
+        if (KINT_PHP81 === false) {
+            $mparser->setAccessible(true);
+        }
 
         $p = new Parser();
         $this->assertNotSame($p, $mparser->getValue($m));