[CodingStyle] Keep partially qualified names as written on import names

rules-tests/CodingStyle/Rector/Namespace_/ImportFullyQualifiedNamesRector/Fixture/instance_of.php.inc → rules-tests/CodingStyle/Rector/Namespace_/ImportFullyQualifiedNamesRector/Fixture/skip_instance_of.php.inc

@@ -25,31 +25,3 @@ final class ImportPartialInstanceOf
 }
 
 ?>
------
-<?php
-
-declare(strict_types=1);
-
-namespace Rector\Tests\CodingStyle\Rector\Namespace_\ImportFullyQualifiedNamesRector\Fixture;
-
-use PhpParser\Node;
-use PhpParser\Node\Expr;
-use PhpParser\Node\Expr\FuncCall;
-
-final class ImportPartialInstanceOf
-{
-    /**
-     * @return string[]
-     */
-    public function getNodeTypes(): array
-    {
-        return [FuncCall::class];
-    }
-
-    public function isIteratorToArrayFuncCall(Expr $expr): bool
-    {
-        return $expr instanceof FuncCall;
-    }
-}
-
-?>

rules-tests/CodingStyle/Rector/Namespace_/ImportFullyQualifiedNamesRector/Fixture/keep_static_method.php.inc → rules-tests/CodingStyle/Rector/Namespace_/ImportFullyQualifiedNamesRector/Fixture/skip_keep_static_method.php.inc

@@ -18,24 +18,3 @@ abstract class KeepStaticMethod
 }
 
 ?>
------
-<?php
-
-declare(strict_types=1);
-
-namespace Rector\Tests\CodingStyle\Rector\Namespace_\ImportFullyQualifiedNamesRector\Fixture;
-
-use Rector\Tests\CodingStyle\Rector\Namespace_\ImportFullyQualifiedNamesRector\Source\Contract\Token;
-use Rector\Tests\CodingStyle\Rector\Namespace_\ImportFullyQualifiedNamesRector\Source\Contract;
-
-abstract class KeepStaticMethod
-{
-    public static function decode(string $token): Token
-    {
-        $parts = explode('|', (string) base64_decode($token, true));
-
-        return Token::create($parts[0] ?? '', $parts[1] ?? '');
-    }
-}
-
-?>

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

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\Tests\CodingStyle\Rector\Namespace_\ImportFullyQualifiedNamesRector\Fixture;
+
+use PhpParser\Node;
+
+class PreventDuplication
+{
+    public function getNodeTypes(): array
+    {
+        return [Node\Stmt\Expression::class];
+    }
+}
+
+?>

rules-tests/Renaming/Rector/Name/RenameClassRector/FixtureAutoImportNames/skip_class_imported_as_alias.php.inc

@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\Renaming\Rector\Name\RenameClassRector\FixtureAutoImportNames;
+
+use Rector\Tests\Renaming\Rector\Name\RenameClassRector\Source\FirstNamespace\FirstOriginalClass as AliasedClass;
+use Rector\Tests\Renaming\Rector\Name\RenameClassRector\Source\SecondNamespace;
+
+$aliasedClass = new AliasedClass();
+$secondClass = new SecondNamespace\SecondOriginalClass();
+
+?>

rules-tests/Renaming/Rector/Name/RenameClassRector/FixtureAutoImportNames/skip_partial_expression.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\Tests\Renaming\Rector\Name\RenameClassRector\FixtureAutoImportNames;
+
+use PhpParser\Node;
+
+class PartialExpression
+{
+    public function getNodeTypes(): array
+    {
+        return [Node\Stmt\Expression::class];
+    }
+}
+
+?>

rules/CodingStyle/ClassNameImport/ClassNameImportSkipper.php

@@ -69,6 +69,10 @@ public function shouldSkipNameForFullyQualifiedObjectType(File $file, Node $node
      */
     public function shouldSkipName(FullyQualified $fullyQualified, array $uses): bool
     {
+        if ($this->isWrittenPartiallyQualified($fullyQualified)) {
+            return true;
+        }
+
         if (substr_count($fullyQualified->toCodeString(), '\\') === 1) {
             return $this->isFunctionOrConstantImport($fullyQualified);
         }
@@ -99,6 +103,19 @@ public function shouldSkipName(FullyQualified $fullyQualified, array $uses): boo
         return false;
     }
 
+    /**
+     * Keep names written as "Field\TextField" or "namespace\Sub\Foo" as they are
+     */
+    private function isWrittenPartiallyQualified(FullyQualified $fullyQualified): bool
+    {
+        $originalName = $fullyQualified->getAttribute(AttributeKey::ORIGINAL_NAME);
+        if (! $originalName instanceof Name) {
+            return false;
+        }
+
+        return $originalName->isQualified() || $originalName->isRelative();
+    }
+
     private function isFunctionOrConstantImport(FullyQualified $fullyQualified): bool
     {
         if ($fullyQualified->getAttribute(AttributeKey::IS_CONSTFETCH_NAME) === true) {

tests/Issues/AutoImport/Fixture/skip_do_not_rename_different_namespaced_function.php.inc

@@ -0,0 +1,11 @@
+<?php
+
+final class DontRenameDifferentNamespacedFunction
+{
+    public function run(): void
+    {
+        $addressParts = Psl\Str\split($addressLine, $houseNumber);
+    }
+}
+
+?>

tests/Issues/AutoImport/Fixture/skip_inner_with_subnamespace.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace helpers;
+
+/**
+ * @see https://3v4l.org/WMUjV0
+ * it read with "helpers" as prefix: helpers\Sub\VarDumper
+ */
+class InnerWithSubnamespace extends Sub\VarDumper
+{
+}
+
+?>

tests/Issues/AutoImport/Fixture/skip_inner_with_subnamespace2.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace helpers;
+
+/**
+ * @see https://3v4l.org/PsIsB
+ * it read as double "helpers" as sub sub: helpers\helpers\Sub\VarDumper
+ */
+class InnerWithSubnamespace extends helpers\Sub\VarDumper
+{
+}
+
+?>

tests/Issues/AutoImportAliasedNamespace/AutoImportAliasedNamespaceTest.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\Issues\AutoImportAliasedNamespace;
+
+use Iterator;
+use PHPUnit\Framework\Attributes\DataProvider;
+use Rector\Testing\PHPUnit\AbstractRectorTestCase;
+
+final class AutoImportAliasedNamespaceTest 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';
+    }
+}