rectorphp · TomasVotruba · Mar 9, 2019 · nschoellhorn · Mar 10, 2019 · nschoellhorn
diff --git a/src/Rector/Psr4/UniteFileAndClassNameRector.php b/src/Rector/Psr4/UniteFileAndClassNameRector.php
@@ -0,0 +1,111 @@
+<?php declare(strict_types=1);
+
+namespace Rector\Rector\Psr4;
+
+use PhpParser\Node;
+use PhpParser\Node\Stmt\ClassLike;
+use Rector\FileSystemRector\Contract\FileSystemRectorInterface;
+use Rector\PhpParser\Node\BetterNodeFinder;
+use Rector\PhpParser\Node\Resolver\NameResolver;
+use Rector\PhpParser\Parser\Parser;
+use Rector\RectorDefinition\CodeSample;
+use Rector\RectorDefinition\RectorDefinition;
+use Symfony\Component\Filesystem\Filesystem;
+use Symplify\PackageBuilder\FileSystem\SmartFileInfo;
+
+final class UniteFileAndClassNameRector implements FileSystemRectorInterface
+{
+    /**
+     * @var BetterNodeFinder
+     */
+    private $betterNodeFinder;
+
+    /**
+     * @var Filesystem
+     */
+    private $filesystem;
+
+    /**
+     * @var Parser
+     */
+    private $parser;
+
+    /**
+     * @var NameResolver
+     */
+    private $nameResolver;
+
+    public function __construct(
+        BetterNodeFinder $betterNodeFinder,
+        Filesystem $filesystem,
+        Parser $parser,
+        NameResolver $nameResolver
+    ) {
+        $this->betterNodeFinder = $betterNodeFinder;
+        $this->filesystem = $filesystem;
+        $this->parser = $parser;
+        $this->nameResolver = $nameResolver;
+    }
+
+    public function getDefinition(): RectorDefinition
+    {
+        return new RectorDefinition(
+            'Corrects file name to be in sync with class name in it and autoloadable by PSR-4 classes.',
+            [
+                new CodeSample(
+                    <<<'CODE_SAMPLE'
+// file: SomeClass.php
+final class SomeCllass 
+{
+
+}
+CODE_SAMPLE
+                    ,
+                    <<<'CODE_SAMPLE'
+// file: SomeClass.php
+final class SomeClass 
+{
+
+}
+CODE_SAMPLE
+                ),
+            ]
+        );
+    }
+
+    public function refactor(SmartFileInfo $smartFileInfo): void
+    {
+        $nodes = $this->parser->parseFile($smartFileInfo->getRealPath());
+
+        // collect named class nodes
+        /** @var ClassLike[] $classLikes */
+        $classLikes = $this->betterNodeFinder->find($nodes, function (Node $node) {
+            if (! $node instanceof ClassLike) {
+                return false;
+            }
+
+            return $node->name !== null;
+        });
+
+        // process files with exactly 1 non-anonymous class like nodes
+        if (count($classLikes) !== 1) {
+            return;
+        }
+
+        $classLikeShortName = $this->nameResolver->resolve($classLikes[0]);
+        if ($classLikeShortName === null) {
+            return;
+        }
+
+        // is in sync with file? skip
+        if ($classLikeShortName === $smartFileInfo->getBasenameWithoutSuffix()) {
+            return;
+        }
+
+        // rename!
+        $oldPath = $smartFileInfo->getRealPath();
+        $newPath = dirname($smartFileInfo->getRealPath()) . DIRECTORY_SEPARATOR . $classLikeShortName . '.php';
+
+        $this->filesystem->rename($oldPath, $newPath);
+    }
+}
diff --git a/tests/Rector/Psr4/UniteFileAndClassNameRector/Source/IncorrectClassNamee.php b/tests/Rector/Psr4/UniteFileAndClassNameRector/Source/IncorrectClassNamee.php
@@ -0,0 +1,8 @@
+<?php declare(strict_types=1);
+
+namespace Rector\Tests\Rector\Psr4\UniteFileAndClassNameRector\Fxitures;
+
+final class IncorrectClassName
+{
+
+}
diff --git a/tests/Rector/Psr4/UniteFileAndClassNameRector/Source/IncorrectInterfaceNamee.php b/tests/Rector/Psr4/UniteFileAndClassNameRector/Source/IncorrectInterfaceNamee.php
@@ -0,0 +1,8 @@
+<?php declare(strict_types=1);
+
+namespace Rector\Tests\Rector\Psr4\UniteFileAndClassNameRector\Fxitures;
+
+interface IncorrectInterfaceName
+{
+
+}
diff --git a/tests/Rector/Psr4/UniteFileAndClassNameRector/Source/IncorrectTraitNamee.php b/tests/Rector/Psr4/UniteFileAndClassNameRector/Source/IncorrectTraitNamee.php
@@ -0,0 +1,8 @@
+<?php declare(strict_types=1);
+
+namespace Rector\Tests\Rector\Psr4\UniteFileAndClassNameRector\Fxitures;
+
+trait IncorrectTraitName
+{
+
+}
diff --git a/tests/Rector/Psr4/UniteFileAndClassNameRector/UniteFileAndClassNameRectorTest.php b/tests/Rector/Psr4/UniteFileAndClassNameRector/UniteFileAndClassNameRectorTest.php
@@ -0,0 +1,58 @@
+<?php declare(strict_types=1);
+
+namespace Rector\Tests\Rector\Psr4\UniteFileAndClassNameRector;
+
+use Rector\FileSystemRector\FileSystemFileProcessor;
+use Rector\HttpKernel\RectorKernel;
+use Symfony\Component\Filesystem\Filesystem;
+use Symplify\PackageBuilder\FileSystem\SmartFileInfo;
+use Symplify\PackageBuilder\Tests\AbstractKernelTestCase;
+
+/**
+ * @covers \Rector\Rector\Psr4\UniteFileAndClassNameRector
+ */
+final class UniteFileAndClassNameRectorTest extends AbstractKernelTestCase
+{
+    /**
+     * @var FileSystemFileProcessor
+     */
+    private $fileSystemFileProcessor;
+
+    /**
+     * @var Filesystem
+     */
+    private $filesystem;
+
+    protected function setUp(): void
+    {
+        $this->bootKernelWithConfigs(RectorKernel::class, [__DIR__ . '/config.yaml']);
+        $this->fileSystemFileProcessor = self::$container->get(FileSystemFileProcessor::class);
+
+        // prepare fixture file
+        $this->filesystem = self::$container->get(Filesystem::class);
+        $this->filesystem->mirror(__DIR__ . '/Source/', __DIR__ . '/Fixtures/');
+    }
+
+    protected function tearDown(): void
+    {
+        // cleanup renamed file
+        $this->filesystem->remove(__DIR__ . '/Fixtures');
+    }
+
+    public function test(): void
+    {
+        $this->assertFileNotExists(__DIR__ . '/Fixtures/IncorrectClassName.php');
+
+        $file = __DIR__ . '/Fixtures/IncorrectClassNamee.php';
+        $this->fileSystemFileProcessor->processFileInfo(new SmartFileInfo($file));
+        $this->assertFileExists(__DIR__ . '/Fixtures/IncorrectClassName.php');
+
+        $file = __DIR__ . '/Fixtures/IncorrectInterfaceNamee.php';
+        $this->fileSystemFileProcessor->processFileInfo(new SmartFileInfo($file));
+        $this->assertFileExists(__DIR__ . '/Fixtures/IncorrectInterfaceName.php');
+
+        $file = __DIR__ . '/Fixtures/IncorrectTraitNamee.php';
+        $this->fileSystemFileProcessor->processFileInfo(new SmartFileInfo($file));
+        $this->assertFileExists(__DIR__ . '/Fixtures/IncorrectTraitName.php');
+    }
+}
diff --git a/tests/Rector/Psr4/UniteFileAndClassNameRector/config.yaml b/tests/Rector/Psr4/UniteFileAndClassNameRector/config.yaml
@@ -0,0 +1,2 @@
+services:
+    Rector\Rector\Psr4\UniteFileAndClassNameRector: ~
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		services:
		Rector\Rector\Psr4\UniteFileAndClassNameRector: ~