refactor

Author: imanghafoori1

src/Features/CheckImports/Checks/CheckClassAtMethod.php

@@ -16,14 +16,7 @@ public static function check(PhpFileDescriptor $file)
     {
         $tokens = $file->getTokens();
 
-        $replaced = self::$handler::handle(
-            $file,
-            self::getAtSignTokens($tokens)
-        );
-
-        if ($replaced) {
-            return $file->getTokens(true);
-        }
+        self::$handler::handle($file, self::getAtSignTokens($tokens));
     }
 
     #[Pure]

src/Features/CheckImports/Checks/CheckClassReferencesAreValid.php

@@ -43,13 +43,12 @@ public static function check(PhpFileDescriptor $file)
         [$extraWrongImports] = ImportsAnalyzer::filterWrongClassRefs($extraImports, $absFilePath);
 
         $wrongClassRefs = array_merge($wrongClassRefs, $wrongDocblockRefs);
-        $tokens = null;
+
         if ($wrongClassRefs && self::$checkWrong && self::$wrongClassRefsHandler) {
-            [$tokens, $isFixed] = self::$wrongClassRefsHandler::handle(
+            $isFixed = self::$wrongClassRefsHandler::handle(
                 $wrongClassRefs,
                 $file,
                 $hostNamespace,
-                $file->getTokens(),
             );
 
             if ($isFixed) {
@@ -60,7 +59,5 @@ public static function check(PhpFileDescriptor $file)
         // Extra wrong imports:
         $handler = self::$extraWrongImportsHandler;
         $handler && $handler::handle($extraWrongImports, $file);
-
-        return $tokens;
     }
 }

src/Features/CheckImports/Handlers/FixWrongClassRefs.php

@@ -13,10 +13,8 @@
  */
 class FixWrongClassRefs
 {
-    public static function handle(array $wrongClassRefs, PhpFileDescriptor $file, $hostNamespace, array $tokens): array
+    public static function handle(array $wrongClassRefs, PhpFileDescriptor $file, $hostNamespace): bool
     {
-        $printer = ErrorPrinter::singleton();
-
         foreach ($wrongClassRefs as $classReference) {
             $wrongClassRef = $classReference['class'];
             $line = $classReference['line'];
@@ -39,15 +37,9 @@ public static function handle(array $wrongClassRefs, PhpFileDescriptor $file, $h
             } else {
                 WrongImportHandler::handle($wrongClassRef, $file, $line);
             }
-
-            if ($isFixed) {
-                $tokens = token_get_all($afterFix);
-
-                return [$tokens, true];
-            }
         }
 
-        return [$tokens, false];
+        return false;
     }
 
     private static function fixClassReference($file, $class, $line, $namespace)

src/Foundations/Analyzers/Fixer.php

@@ -3,7 +3,6 @@
 namespace Imanghafoori\LaravelMicroscope\Foundations\Analyzers;
 
 use ImanGhafoori\ComposerJson\NamespaceCalculator;
-use Imanghafoori\Filesystem\FileManipulator;
 use Imanghafoori\LaravelMicroscope\Foundations\ClassListProvider;
 use Imanghafoori\LaravelMicroscope\Foundations\Loop;
 use Imanghafoori\LaravelMicroscope\Foundations\PhpFileDescriptor;
@@ -93,7 +92,7 @@ public static function fixReference(PhpFileDescriptor $file, $inlinedClassRef, $
         }
 
         return [
-            FileManipulator::insertNewLine($file->getAbsolutePath(), "use $fullClassPath;", $lineNum),
+            $file->insertNewLine("use $fullClassPath;", $lineNum),
             $correct,
         ];
     }

src/Foundations/Iterators/CheckSet.php

@@ -125,8 +125,7 @@ private function performCheck($check, PhpFileDescriptor $file)
     private function applyCheck($check, PhpFileDescriptor $file): void
     {
         try {
-            $newTokens = $this->performCheck($check, $file);
-            $newTokens && $file->setTokens($newTokens);
+            $this->performCheck($check, $file);
         } catch (Throwable $exception) {
             $this->exceptions[] = $exception;
         }

src/Foundations/PhpFileDescriptor.php

@@ -174,6 +174,8 @@ public static function searchReplace($search, $replace, $tokens): array
 
     public function insertNewLine($newLine, $atLine)
     {
+        $this->tokens = [];
+
         return FileManipulator::insertNewLine($this->getAbsolutePath(), $newLine, $atLine);
     }
 

src/Foundations/Reports/RouteReport.php

@@ -10,16 +10,16 @@ class RouteReport
 {
     /**
      * @param  \Imanghafoori\LaravelMicroscope\Foundations\Iterators\DTO\FilesDto  $routeFiles
-     * @return string
+     * @return array
      */
     #[Pure]
     public static function getStats($routeFiles)
     {
         $lines = CheckImportReporter::formatFiles($routeFiles);
         $count = count($lines);
         $s = $count <= 1 ? '' : 's';
-        $count = Color::white("($count files)");
+        $countStr = Color::white("($count files)");
 
-        return [CheckImportReporter::hyphen()."route$s {$count}", $lines];
+        return [CheckImportReporter::hyphen()."route$s {$countStr}", $lines];
     }
 }