[utils] Add RemoveRefactorDuplicatedNodeInstanceCheckRector

composer.json

@@ -81,6 +81,7 @@
                 "rules-tests",
                 "tests"
             ],
+            "Rector\\Utils\\Tests\\": "utils-tests",
             "E2e\\Parallel\\Reflection\\Resolver\\": [
                 "e2e/parallel-reflection-resolver/src/",
                 "e2e/no-parallel-reflection-resolver/src"

phpstan.neon

@@ -352,3 +352,5 @@ parameters:
         -
             identifier: offsetAccess.invalidOffset
             path: src/CustomRules/SimpleNodeDumper.php
+
+        - '#Method Rector\\Utils\\Rector\\RemoveRefactorDuplicatedNodeInstanceCheckRector\:\:getInstanceofNodeClass\(\) should return class\-string<PhpParser\\Node>\|null but returns class\-string#'

phpunit.xml

@@ -13,6 +13,7 @@
         <testsuite name="main">
             <directory>tests</directory>
             <directory>rules-tests</directory>
+            <directory>utils-tests</directory>
         </testsuite>
     </testsuites>
 

rector.php

@@ -6,6 +6,7 @@
 use Rector\Config\RectorConfig;
 use Rector\DeadCode\Rector\ConstFetch\RemovePhpVersionIdCheckRector;
 use Rector\Php55\Rector\String_\StringClassNameToClassConstantRector;
+use Rector\Utils\Rector\RemoveRefactorDuplicatedNodeInstanceCheckRector;
 
 return RectorConfig::configure()
     ->withPreparedSets(
@@ -18,7 +19,6 @@
         naming: true,
         instanceOf: true,
         earlyReturn: true,
-        strictBooleans: true,
         rectorPreset: true,
         phpunitCodeQuality: true
     )
@@ -37,6 +37,7 @@
     ])
     ->withRootFiles()
     ->withImportNames(removeUnusedImports: true)
+    ->withRules([RemoveRefactorDuplicatedNodeInstanceCheckRector::class])
     ->withSkip([
         StringClassNameToClassConstantRector::class,
         // tests

rules/CodeQuality/Rector/BooleanNot/ReplaceConstantBooleanNotRector.php

@@ -69,10 +69,6 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?Node
     {
-        if (! $node instanceof BooleanNot) {
-            return null;
-        }
-
         if ($this->valueResolver->isFalse($node->expr)) {
             return new ConstFetch(new Name('true'));
         }

rules/CodingStyle/Rector/FuncCall/FunctionFirstClassCallableRector.php

@@ -61,12 +61,11 @@ public function getNodeTypes(): array
         return [FuncCall::class];
     }
 
+    /**
+     * @param FuncCall $node
+     */
     public function refactor(Node $node): ?FuncCall
     {
-        if (! $node instanceof FuncCall) {
-            return null;
-        }
-
         if (! $node->name instanceof Name) {
             return null;
         }

rules/DeadCode/Rector/ClassMethod/RemoveUnusedPublicMethodParameterRector.php

@@ -13,6 +13,7 @@
 use Rector\NodeAnalyzer\MagicClassMethodAnalyzer;
 use Rector\Php80\NodeAnalyzer\PhpAttributeAnalyzer;
 use Rector\Rector\AbstractRector;
+use Rector\ValueObject\MethodName;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 
@@ -115,7 +116,7 @@ private function shouldSkipClassMethod(ClassMethod $classMethod, Class_ $class):
         }
 
         // parameter is required for contract coupling
-        if ($this->isName($classMethod->name, '__invoke') && $this->phpAttributeAnalyzer->hasPhpAttribute(
+        if ($this->isName($classMethod->name, MethodName::INVOKE) && $this->phpAttributeAnalyzer->hasPhpAttribute(
             $class,
             'Symfony\Component\Messenger\Attribute\AsMessageHandler'
         )) {

src/VendorLocker/NodeVendorLocker/ClassMethodParamVendorLockResolver.php

@@ -6,7 +6,6 @@
 
 use PhpParser\Node\Stmt\ClassMethod;
 use PHPStan\Reflection\ClassReflection;
-use Rector\FileSystem\FilePathHelper;
 use Rector\NodeNameResolver\NodeNameResolver;
 use Rector\Reflection\ReflectionResolver;
 
@@ -15,7 +14,6 @@
     public function __construct(
         private NodeNameResolver $nodeNameResolver,
         private ReflectionResolver $reflectionResolver,
-        private FilePathHelper $filePathHelper
     ) {
     }
 
@@ -38,11 +36,7 @@ public function isVendorLocked(ClassMethod $classMethod): bool
         $methodName = $this->nodeNameResolver->getName($classMethod);
 
         // has interface vendor lock? → better skip it, as PHPStan has access only to just analyzed classes
-        if ($this->hasParentInterfaceMethod($classReflection, $methodName)) {
-            return true;
-        }
-
-        return $this->hasClassMethodLockMatchingFileName($classReflection, $methodName, '/vendor/');
+        return $this->hasParentInterfaceMethod($classReflection, $methodName);
     }
 
     /**
@@ -60,37 +54,4 @@ private function hasParentInterfaceMethod(ClassReflection $classReflection, stri
 
         return false;
     }
-
-    private function hasClassMethodLockMatchingFileName(
-        ClassReflection $classReflection,
-        string $methodName,
-        string $filePathPartName
-    ): bool {
-        $ancestorClassReflections = [...$classReflection->getParents(), ...$classReflection->getInterfaces()];
-        foreach ($ancestorClassReflections as $ancestorClassReflection) {
-            // parent type
-            if (! $ancestorClassReflection->hasNativeMethod($methodName)) {
-                continue;
-            }
-
-            // is file in vendor?
-            $fileName = $ancestorClassReflection->getFileName();
-            // probably internal class
-            if ($fileName === null) {
-                continue;
-            }
-
-            // not conditions? its a match
-            if ($filePathPartName === '') {
-                return true;
-            }
-
-            $normalizedFileName = $this->filePathHelper->normalizePathAndSchema($fileName);
-            if (str_contains($normalizedFileName, $filePathPartName)) {
-                return true;
-            }
-        }
-
-        return false;
-    }
 }

utils-tests/Rector/RemoveRefactorDuplicatedNodeInstanceCheckRector/Fixture/cover_bare_get_node_types.php.inc

@@ -0,0 +1,54 @@
+<?php
+
+namespace Rector\Utils\Tests\Rector\RemoveRefactorDuplicatedNodeInstanceCheckRector\Fixture;
+
+use PhpParser\Node;
+use Rector\Rector\AbstractRector;
+use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
+
+final class CoverBareGetNodeTypes extends AbstractRector
+{
+    public function getRuleDefinition(): RuleDefinition
+    {
+    }
+
+    public function getNodeTypes(): array
+    {
+        return [Node\Stmt\ClassMethod::class];
+    }
+
+    public function refactor(Node $node)
+    {
+        if (! $node instanceof Node\Stmt\ClassMethod) {
+            return null;
+        }
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Utils\Tests\Rector\RemoveRefactorDuplicatedNodeInstanceCheckRector\Fixture;
+
+use PhpParser\Node;
+use Rector\Rector\AbstractRector;
+use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
+
+final class CoverBareGetNodeTypes extends AbstractRector
+{
+    public function getRuleDefinition(): RuleDefinition
+    {
+    }
+
+    public function getNodeTypes(): array
+    {
+        return [Node\Stmt\ClassMethod::class];
+    }
+
+    public function refactor(Node $node)
+    {
+    }
+}
+
+?>

utils-tests/Rector/RemoveRefactorDuplicatedNodeInstanceCheckRector/Fixture/some_class.php.inc

@@ -0,0 +1,60 @@
+<?php
+
+namespace Rector\Utils\Tests\Rector\RemoveRefactorDuplicatedNodeInstanceCheckRector\Fixture;
+
+use PhpParser\Node;
+use Rector\Rector\AbstractRector;
+use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
+
+final class SomeClass extends AbstractRector
+{
+    public function getRuleDefinition(): RuleDefinition
+    {
+    }
+
+    public function getNodeTypes(): array
+    {
+        return [Node\Stmt\ClassMethod::class];
+    }
+
+    /**
+     * @param Node\Stmt\ClassMethod $node
+     */
+    public function refactor(Node $node)
+    {
+        if (! $node instanceof Node\Stmt\ClassMethod) {
+            return null;
+        }
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Utils\Tests\Rector\RemoveRefactorDuplicatedNodeInstanceCheckRector\Fixture;
+
+use PhpParser\Node;
+use Rector\Rector\AbstractRector;
+use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
+
+final class SomeClass extends AbstractRector
+{
+    public function getRuleDefinition(): RuleDefinition
+    {
+    }
+
+    public function getNodeTypes(): array
+    {
+        return [Node\Stmt\ClassMethod::class];
+    }
+
+    /**
+     * @param Node\Stmt\ClassMethod $node
+     */
+    public function refactor(Node $node)
+    {
+    }
+}
+
+?>

utils-tests/Rector/RemoveRefactorDuplicatedNodeInstanceCheckRector/RemoveRefactorDuplicatedNodeInstanceCheckRectorTest.php

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

utils-tests/Rector/RemoveRefactorDuplicatedNodeInstanceCheckRector/config/configured_rule.php

@@ -0,0 +1,11 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\Utils\Rector\RemoveRefactorDuplicatedNodeInstanceCheckRector;
+
+return RectorConfig::configure()
+    ->withRules([
+        RemoveRefactorDuplicatedNodeInstanceCheckRector::class,
+    ]);