refactor: setAccessible no longer effect since PHP 8.1

ddevsr · jnvsor · commit 33e535cd3ec8 · 2025-08-17T11:28:10.000+02:00
diff --git a/build/kint.phar b/build/kint.phar
diff --git a/src/Parser/ClassStaticsPlugin.php b/src/Parser/ClassStaticsPlugin.php
@@ -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
diff --git a/src/Parser/Parser.php b/src/Parser/Parser.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:
diff --git a/tests/Parser/ClassStringsPluginTest.php b/tests/Parser/ClassStringsPluginTest.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));
diff --git a/tests/Parser/DomPluginTest.php b/tests/Parser/DomPluginTest.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));
diff --git a/tests/Parser/SimpleXMLElementPluginTest.php b/tests/Parser/SimpleXMLElementPluginTest.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));

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,9 @@ private function buildStaticValue(ReflectionProperty $pr, int $depth): AbstractV`
`136`	`136`	`$context->access_path = '\\'.$context->owner_class.'::$'.$context->name;`
`137`	`137`	`}`
`138`	`138`
`139`		`- $pr->setAccessible(true);`
	`139`	`+ if (KINT_PHP81 === false) {`
	`140`	`+ $pr->setAccessible(true);`
	`141`	`+ }`
`140`	`142`
`141`	`143`	`/**`
`142`	`144`	`* @psalm-suppress TooFewArguments`