[AutoImport] Skip multiple namespaces on single file on auto imports (#…

…2940)

* [AutoImport] Skip multiple namespaces on auto imports

* more fixture

* more fixture

* skip multiple namespaces

packages/PostRector/Rector/NameImportingPostRector.php

@@ -7,7 +7,9 @@
 use PhpParser\Node;
 use PhpParser\Node\Name;
 use PhpParser\Node\Name\FullyQualified;
+use PhpParser\Node\Stmt;
 use PhpParser\Node\Stmt\GroupUse;
+use PhpParser\Node\Stmt\Namespace_;
 use PhpParser\Node\Stmt\Use_;
 use PHPStan\Reflection\ReflectionProvider;
 use Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfoFactory;
@@ -102,7 +104,16 @@ public function run(AnotherClass $anotherClass)
     private function processNodeName(Name $name, File $file): ?Node
     {
         if ($name->isSpecialClassName()) {
-            return $name;
+            return null;
+        }
+
+        $namespaces = array_filter(
+            $file->getNewStmts(),
+            static fn (Stmt $stmt): bool => $stmt instanceof Namespace_
+        );
+
+        if (count($namespaces) > 1) {
+            return null;
         }
 
         /** @var Use_[]|GroupUse[] $currentUses */

rules-tests/CodingStyle/Rector/Namespace_/ImportFullyQualifiedNamesRector/Fixture/skip_multiple_namespaces.php.inc

@@ -0,0 +1,18 @@
+<?php
+
+namespace {
+    function run_me_never()
+    {
+        // silent deprecations, since we test them
+        error_reporting(E_ALL ^ E_DEPRECATED);
+
+        // performance boost
+        \SomeNamespace\gc_disable();
+    }
+}
+
+namespace SomeNamespace
+{
+    function gc_disable()
+    {}
+}

tests/Issues/AutoImportGroupUse/Fixture/skip_multiple_namespaces_auto_import_in_groupuse.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace A {
+    use Rector\Core\Tests\Issues\AutoImportGroupUse\Source\{ SomeClass };
+
+    final class AutoImportInGroupUse extends \Rector\Core\Tests\Issues\AutoImportGroupUse\Source\SomeClass
+    {
+    }
+}
+
+namespace B {
+    final class AutoImportInGroupUse extends \Rector\Core\Tests\Issues\AutoImportGroupUse\Source\SomeClass
+    {
+    }
+}

tests/Issues/AutoImportInAlias/Fixture/skip_multiple_namespace.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace C {
+    use stdClass;
+
+    final class AutoImport extends \stdClass
+    {
+    }
+}
+
+namespace D {
+    final class AutoImport extends \stdClass
+    {
+    }
+}

tests/Issues/AutoImportInAlias/Fixture/skip_multiple_namespace_with_alias.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace A {
+    use stdClass as SomeObject;
+
+    final class AutoImportInAlias extends \stdClass
+    {
+    }
+}
+
+namespace B {
+    final class AutoImportInAlias extends \stdClass
+    {
+    }
+}