Fixed paths

Author: andrey-helldar

src/Processors/Install.php

@@ -35,7 +35,7 @@ protected function create(): void
     protected function ensureDirectory(): void
     {
         Directory::ensureDirectory(
-            $this->getActionsPath()
+            $this->getActionsPath(realpath: $this->options->realpath)
         );
     }
 }

src/Processors/Make.php

@@ -22,7 +22,7 @@ public function handle(): void
     protected function run(): void
     {
         $name = $this->getName();
-        $path = $this->getActionsPath($name);
+        $path = $this->getActionsPath($name, realpath: $this->options->realpath);
 
         $this->create($path);
     }

src/Processors/Processor.php

@@ -41,16 +41,16 @@ public function __construct(
         $this->migrator->setConnection($this->options->connection)->setOutput($this->output);
     }
 
-    protected function getFiles(?Closure $filter = null, ?string $path = null): array
+    protected function getFiles(?Closure $filter = null, ?string $path = null, bool $realpath = false): array
     {
-        $path = $this->getActionsPath($path);
+        $path = $this->getActionsPath($path, $realpath);
 
-        return $this->file->exists($path) ? [$path] : $this->file->allPaths($path, $filter, true);
+        return $this->file->exists($path) ? [$path] : $this->file->names($path, $filter, true);
     }
 
-    protected function getActionsPath(?string $path = null): string
+    protected function getActionsPath(?string $path = null, bool $realpath = false): string
     {
-        $path = $this->options->realpath ? $path : $this->config->path($path);
+        $path = $realpath ? $path : $this->config->path($path);
 
         if (! is_dir($path) && ! Str::endsWith($path, '.php')) {
             return $this->file->exists($path . '.php') ? $path . '.php' : $path;

src/Processors/Rollback.php

@@ -51,7 +51,7 @@ protected function getActions(?int $step): array
     protected function rollbackAction(string $action): void
     {
         $this->migrator->runDown(
-            $this->getActionsPath($action)
+            $this->getActionsPath($action, realpath: $this->options->realpath)
         );
     }
 

src/Processors/Upgrade.php

@@ -45,9 +45,9 @@ protected function move(string $filename): void
         $content = $this->replaceClassName($content);
         $content = $this->replaceDeclareStrictType($content);
         $content = $this->replaceWithInvoke($content);
+        $content = $this->replaceProperties($content);
 
         $this->store($filename, $content);
-        $this->delete($filename);
     }
 
     protected function clean(): void
@@ -57,19 +57,19 @@ protected function clean(): void
         ));
     }
 
-    protected function open(string $path): string
+    protected function open(string $filename): string
     {
-        return file_get_contents(base_path('database/actions/' . $path));
+        return file_get_contents(base_path('database/actions/' . $filename));
     }
 
-    protected function store(string $path, string $content): void
+    protected function store(string $filename, string $content): void
     {
-        file_put_contents($this->config->path($path), $content);
+        File::store($this->config->path($filename), $content);
     }
 
-    protected function delete(string $path): void
+    protected function delete(string $filename): void
     {
-        File::ensureDelete($this->config->path($path));
+        File::ensureDelete($filename);
     }
 
     protected function replaceNamespace(string $content): string
@@ -83,8 +83,9 @@ protected function replaceNamespace(string $content): string
     protected function replaceClassName(string $content): string
     {
         return Str::of($content)
-            ->pregReplace('/^([final\s|class]+.+extends\sAction)$/', 'return new class () extends Action')
+            ->pregReplace('/((?:return\s+new\s+class\s*\(?\s*\)?|final\s+class|class)\s*.+extends\s+Action)/', 'return new class () extends Action')
             ->trim()
+            ->trim(';')
             ->append(';')
             ->append(PHP_EOL)
             ->toString();
@@ -106,6 +107,18 @@ protected function replaceWithInvoke(string $content): string
             })->toString();
     }
 
+    protected function replaceProperties(string $content): string
+    {
+        return Str::of($content)
+            ->pregReplace('/protected\s+\$once/', 'protected bool $once')
+            ->pregReplace('/protected\s+\$transactions/', 'protected bool $transactions')
+            ->pregReplace('/protected\s+\$transaction_attempts/', 'protected int $transactionAttempts')
+            ->pregReplace('/protected\s+\$environment/', 'protected string|array|null $environment')
+            ->pregReplace('/protected\s+\$except_environment/', 'protected string|array|null $exceptEnvironment')
+            ->pregReplace('/protected\s+\$before/', 'protected bool $before')
+            ->toString();
+    }
+
     protected function moveConfig(): void
     {
         $this->notification->task('Moving config file', function () {
@@ -126,7 +139,7 @@ protected function moveConfig(): void
 
     protected function getOldFiles(): array
     {
-        return $this->getFiles(path: database_path('actions'));
+        return $this->getFiles(path: database_path('actions'), realpath: true);
     }
 
     protected function alreadyUpgraded(): bool