Remove UpdateFileNameByClassNameFileSystemRector niche rule that was …

…added just for example sake (#3849)

build/target-repository/docs/rector_rules_overview.md

@@ -1,4 +1,4 @@
-# 409 Rules Overview
+# 408 Rules Overview
 
 <br>
 
@@ -58,7 +58,7 @@
 
 - [Renaming](#renaming) (11)
 
-- [Restoration](#restoration) (3)
+- [Restoration](#restoration) (2)
 
 - [Strict](#strict) (6)
 
@@ -7389,22 +7389,6 @@ Convert missing class reference to string
 
 <br>
 
-### UpdateFileNameByClassNameFileSystemRector
-
-Rename file to respect class name
-
-- class: [`Rector\Restoration\Rector\ClassLike\UpdateFileNameByClassNameFileSystemRector`](../rules/Restoration/Rector/ClassLike/UpdateFileNameByClassNameFileSystemRector.php)
-
-```diff
--// app/SomeClass.php
-+// app/AnotherClass.php
- class AnotherClass
- {
- }
-```
-
-<br>
-
 ## Strict
 
 ### AddConstructorParentCallRector

packages/Testing/PHPUnit/Behavior/MovingFilesTrait.php

@@ -41,13 +41,6 @@ private function resolveAddedFilePathsWithContents(): array
         $addedFilePathsWithContents = $this->removedAndAddedFilesCollector->getAddedFilesWithContent();
         $nodesWithFileDestinationPrinter = $this->getService(NodesWithFileDestinationPrinter::class);
 
-        $movedFiles = $this->removedAndAddedFilesCollector->getMovedFiles();
-
-        foreach ($movedFiles as $movedFile) {
-            $fileContent = $nodesWithFileDestinationPrinter->printNodesWithFileDestination($movedFile);
-            $addedFilePathsWithContents[] = new AddedFileWithContent($movedFile->getNewFilePath(), $fileContent);
-        }
-
         $addedFilesWithNodes = $this->removedAndAddedFilesCollector->getAddedFilesWithNodes();
         if ($addedFilesWithNodes === []) {
             return $addedFilePathsWithContents;

src/Application/FileSystem/RemovedAndAddedFilesCollector.php

@@ -4,8 +4,6 @@
 
 namespace Rector\Core\Application\FileSystem;
 
-use Rector\Core\ValueObject\Application\File;
-use Rector\Core\ValueObject\Application\MovedFile;
 use Rector\FileSystemRector\Contract\AddedFileInterface;
 use Rector\FileSystemRector\ValueObject\AddedFileWithContent;
 use Rector\FileSystemRector\ValueObject\AddedFileWithNodes;
@@ -22,11 +20,6 @@ final class RemovedAndAddedFilesCollector
      */
     private array $addedFiles = [];
 
-    /**
-     * @var MovedFile[]
-     */
-    private array $movedFiles = [];
-
     public function removeFile(string $filePath): void
     {
         $this->removedFilePaths[] = $filePath;
@@ -50,15 +43,6 @@ public function isFileRemoved(string $filePath): bool
             return true;
         }
 
-        foreach ($this->movedFiles as $movedFile) {
-            $file = $movedFile->getFile();
-            if ($movedFile->getFilePath() !== $file->getFilePath()) {
-                continue;
-            }
-
-            return true;
-        }
-
         return false;
     }
 
@@ -108,20 +92,6 @@ public function getRemovedFilesCount(): int
     public function reset(): void
     {
         $this->addedFiles = [];
-        $this->movedFiles = [];
         $this->removedFilePaths = [];
     }
-
-    public function addMovedFile(File $file, string $newPathName): void
-    {
-        $this->movedFiles[] = new MovedFile($file, $newPathName);
-    }
-
-    /**
-     * @return MovedFile[]
-     */
-    public function getMovedFiles(): array
-    {
-        return $this->movedFiles;
-    }
 }

src/Application/FileSystem/RemovedAndAddedFilesProcessor.php

@@ -28,7 +28,6 @@ public function run(Configuration $configuration): void
     {
         $this->processAddedFilesWithContent($configuration);
         $this->processAddedFilesWithNodes($configuration);
-        $this->processMovedFilesWithNodes($configuration);
 
         $this->processDeletedFiles($configuration);
     }
@@ -86,30 +85,4 @@ private function processAddedFilesWithNodes(Configuration $configuration): void
             }
         }
     }
-
-    private function processMovedFilesWithNodes(Configuration $configuration): void
-    {
-        foreach ($this->removedAndAddedFilesCollector->getMovedFiles() as $movedFile) {
-            $fileContent = $this->nodesWithFileDestinationPrinter->printNodesWithFileDestination($movedFile);
-
-            if ($configuration->isDryRun()) {
-                $message = sprintf(
-                    'File "%s" will be moved to "%s"',
-                    $movedFile->getFilePath(),
-                    $movedFile->getNewFilePath()
-                );
-                $this->rectorOutputStyle->note($message);
-            } else {
-                $this->filesystem->dumpFile($movedFile->getNewFilePath(), $fileContent);
-                $this->filesystem->remove($movedFile->getFilePath());
-
-                $message = sprintf(
-                    'File "%s" was moved to "%s"',
-                    $movedFile->getFilePath(),
-                    $movedFile->getNewFilePath()
-                );
-                $this->rectorOutputStyle->note($message);
-            }
-        }
-    }
 }

src/PhpParser/Printer/NodesWithFileDestinationPrinter.php

@@ -5,7 +5,7 @@
 namespace Rector\Core\PhpParser\Printer;
 
 use Rector\Core\Contract\PhpParser\NodePrinterInterface;
-use Rector\FileSystemRector\Contract\FileWithNodesInterface;
+use Rector\FileSystemRector\ValueObject\AddedFileWithNodes;
 use Rector\PostRector\Application\PostFileProcessor;
 
 final class NodesWithFileDestinationPrinter
@@ -16,9 +16,9 @@ public function __construct(
     ) {
     }
 
-    public function printNodesWithFileDestination(FileWithNodesInterface $fileWithNodes): string
+    public function printNodesWithFileDestination(AddedFileWithNodes $addedFileWithNodes): string
     {
-        $nodes = $this->postFileProcessor->traverse($fileWithNodes->getNodes());
+        $nodes = $this->postFileProcessor->traverse($addedFileWithNodes->getNodes());
         return $this->nodePrinter->prettyPrintFile($nodes);
     }
 }