feat: added RemoveDeprecatedReflectionSetAccessibleCallsRector

Author: NickSdot

config/set/php85.php

@@ -4,7 +4,13 @@
 
 use Rector\Config\RectorConfig;
 use Rector\Php85\Rector\ArrayDimFetch\ArrayFirstLastRector;
+use Rector\Php85\Rector\MethodCall\RemoveReflectionSetAccessibleCallsRector;
 
 return static function (RectorConfig $rectorConfig): void {
-    $rectorConfig->rules([ArrayFirstLastRector::class]);
+    $rectorConfig->rules(
+        [
+            ArrayFirstLastRector::class,
+            RemoveReflectionSetAccessibleCallsRector::class,
+        ]
+    );
 };

rules-tests/Php85/Rector/MethodCall/RemoveReflectionSetAccessibleCallsRector/Fixture/fixture.php.inc

@@ -0,0 +1,43 @@
+<?php
+
+namespace Rector\Tests\Php85\Rector\MethodCall\RemoveDeprecatedReflectionSetAccessibleCalls\Fixture;
+
+use ReflectionMethod;
+use ReflectionProperty;
+
+final class Fixture
+{
+    public function run(): void
+    {
+        $reflectionProperty = new ReflectionProperty($this, 'privateProperty');
+        $reflectionProperty->setAccessible(true);
+        $value = $reflectionProperty->getValue($this);
+
+        $reflectionMethod = new ReflectionMethod($this, 'privateMethod');
+        $reflectionMethod->setAccessible(false);
+        $reflectionMethod->invoke($this);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php85\Rector\MethodCall\RemoveDeprecatedReflectionSetAccessibleCalls\Fixture;
+
+use ReflectionMethod;
+use ReflectionProperty;
+
+final class Fixture
+{
+    public function run(): void
+    {
+        $reflectionProperty = new ReflectionProperty($this, 'privateProperty');
+        $value = $reflectionProperty->getValue($this);
+
+        $reflectionMethod = new ReflectionMethod($this, 'privateMethod');
+        $reflectionMethod->invoke($this);
+    }
+}
+
+?>

rules-tests/Php85/Rector/MethodCall/RemoveReflectionSetAccessibleCallsRector/RemoveReflectionSetAccessibleCallsRectorTest.php

@@ -0,0 +1,30 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\Php85\Rector\MethodCall\RemoveReflectionSetAccessibleCallsRector;
+
+use PHPUnit\Framework\Attributes\DataProvider;
+use Rector\Testing\PHPUnit\AbstractRectorTestCase;
+
+final class RemoveReflectionSetAccessibleCallsRectorTest extends AbstractRectorTestCase
+{
+    #[DataProvider('provideData')]
+    public function testRule(string $filePath): void
+    {
+        $this->doTestFile($filePath);
+    }
+
+    /**
+     * @return \Iterator<array<string>>
+     */
+    public static function provideData(): \Iterator
+    {
+        return self::yieldFilesFromDirectory(__DIR__ . '/Fixture');
+    }
+
+    public function provideConfigFilePath(): string
+    {
+        return __DIR__ . '/config/configured_rule.php';
+    }
+}

rules-tests/Php85/Rector/MethodCall/RemoveReflectionSetAccessibleCallsRector/config/configured_rule.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\Php85\Rector\MethodCall\RemoveReflectionSetAccessibleCallsRector;
+use Rector\ValueObject\PhpVersion;
+
+return static function (RectorConfig $rectorConfig): void {
+    $rectorConfig->rule(RemoveReflectionSetAccessibleCallsRector::class);
+
+    $rectorConfig->phpVersion(PhpVersion::PHP_85);
+};

rules/Php85/Rector/MethodCall/RemoveReflectionSetAccessibleCallsRector.php

@@ -0,0 +1,88 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Php85\Rector\MethodCall;
+
+use PhpParser\Node;
+use PhpParser\Node\Expr\MethodCall;
+use PhpParser\Node\Stmt\Expression;
+use PhpParser\NodeVisitor;
+use PHPStan\Type\ObjectType;
+use Rector\Rector\AbstractRector;
+use Rector\ValueObject\PhpVersion;
+use Rector\VersionBonding\Contract\MinPhpVersionInterface;
+use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
+use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
+
+/**
+ * As of PHP 8.1.0, calling `Reflection*::setAccessible()` has no effect.
+ *
+ * @see https://www.php.net/manual/en/reflectionmethod.setaccessible.php
+ * @see https://www.php.net/manual/en/reflectionproperty.setaccessible.php
+ * @see \Rector\Tests\Php85\Rector\MethodCall\RemoveReflectionSetAccessibleCallsRector\RemoveReflectionSetAccessibleCallsRectorTest
+ */
+final class RemoveReflectionSetAccessibleCallsRector extends AbstractRector implements MinPhpVersionInterface
+{
+    /**
+     * @return array<class-string<Node>>
+     */
+    public function getNodeTypes(): array
+    {
+        return [Expression::class];
+    }
+
+    /**
+     * @param Expression $node
+     */
+    public function refactor(Node $node): ?int
+    {
+        if ($node->expr instanceof MethodCall === false) {
+            return null;
+        }
+
+        $methodCall = $node->expr;
+
+        if ($this->isName($methodCall->name, 'setAccessible') === false) {
+            return null;
+        }
+
+        if ($this->isObjectType($methodCall->var, new ObjectType('ReflectionProperty')) === true
+            || $this->isObjectType($methodCall->var, new ObjectType('ReflectionMethod')) === true
+        ) {
+            return NodeVisitor::REMOVE_NODE;
+        }
+
+        return null;
+    }
+
+    public function getRuleDefinition(): RuleDefinition
+    {
+        return new RuleDefinition('Remove Reflection::setAccessible() calls', [
+            new CodeSample(
+                <<<'CODE_SAMPLE'
+$reflectionProperty = new ReflectionProperty($object, 'property');
+$reflectionProperty->setAccessible(true);
+$value = $reflectionProperty->getValue($object);
+
+$reflectionMethod = new ReflectionMethod($object, 'method');
+$reflectionMethod->setAccessible(false);
+$reflectionMethod->invoke($object);
+CODE_SAMPLE
+                ,
+                <<<'CODE_SAMPLE'
+$reflectionProperty = new ReflectionProperty($object, 'property');
+$value = $reflectionProperty->getValue($object);
+
+$reflectionMethod = new ReflectionMethod($object, 'method');
+$reflectionMethod->invoke($object);
+CODE_SAMPLE
+            ),
+        ]);
+    }
+
+    public function provideMinPhpVersion(): int
+    {
+        return PhpVersion::PHP_85;
+    }
+}