Merge pull request #114 from TheDragonCode/4.x-1

Database table name changed from `migration_actions` to `actions`

Author: andrey-helldar

config/actions.php

@@ -24,7 +24,7 @@
     |
     */
 
-    'table'      => 'deploy_actions',
+    'table'      => 'actions',
 
     /*
     |--------------------------------------------------------------------------

database/migrations/anonymous/2022_08_18_180137_change_migration_actions_table.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 use DragonCode\LaravelActions\Database\BaseChangeColumn;
 
 return new class () extends BaseChangeColumn

database/migrations/anonymous/2023_01_21_172923_rename_migrations_actions_table_to_actions.php

@@ -0,0 +1,9 @@
+<?php
+
+declare(strict_types=1);
+
+use DragonCode\LaravelActions\Database\BaseRenameMigrationsActionsTableToActions;
+
+return new class () extends BaseRenameMigrationsActionsTableToActions
+{
+};

database/migrations/named/2022_08_18_180137_change_migration_actions_table.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 use DragonCode\LaravelActions\Database\BaseChangeColumn;
 
 class ChangeMigrationActionsTable extends BaseChangeColumn

database/migrations/named/2023_01_21_172923_rename_migrations_actions_table_to_actions.php

@@ -0,0 +1,9 @@
+<?php
+
+declare(strict_types=1);
+
+use DragonCode\LaravelActions\Database\BaseRenameMigrationsActionsTableToActions;
+
+class RenameMigrationsActionsTableToActions extends BaseRenameMigrationsActionsTableToActions
+{
+}

src/Database/BaseRenameMigrationsActionsTableToActions.php

@@ -0,0 +1,55 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DragonCode\LaravelActions\Database;
+
+use DragonCode\LaravelActions\Action;
+use DragonCode\LaravelActions\Helpers\Config;
+use Illuminate\Support\Facades\Schema;
+use RuntimeException;
+
+class BaseRenameMigrationsActionsTableToActions extends Action
+{
+    protected Config $config;
+
+    public function __construct()
+    {
+        $this->config = app(Config::class);
+    }
+
+    public function up(): void
+    {
+        if (Schema::hasTable('migration_actions') && $this->doesntSame('migration_actions', $this->table())) {
+            $this->validateTable($this->table());
+
+            Schema::rename('migration_actions', $this->table());
+        }
+    }
+
+    public function down(): void
+    {
+        if (Schema::hasTable($this->table()) && $this->doesntSame('migration_actions', $this->table())) {
+            $this->validateTable('migration_actions');
+
+            Schema::rename($this->table(), 'migration_actions');
+        }
+    }
+
+    protected function validateTable(string $name): void
+    {
+        if (Schema::hasTable($name)) {
+            throw new RuntimeException(sprintf('A table named [%s] already exists. Change the table name settings in the [%s] configuration file.', $name, 'config/actions.php'));
+        }
+    }
+
+    protected function doesntSame(string $first, string $second): bool
+    {
+        return $first !== $second;
+    }
+
+    protected function table(): string
+    {
+        return $this->config->table();
+    }
+}

src/ServiceProvider.php

@@ -22,8 +22,8 @@ public function boot(): void
         if ($this->app->runningInConsole()) {
             $this->publishConfig();
 
-            $this->registerCommands();
             $this->registerAbout();
+            $this->registerCommands();
             $this->registerMigrations();
             $this->registerNotifications();
         }

tests/Migrations/MigrationTest.php

@@ -34,7 +34,8 @@ public function testRunMigrationAfterInstall(): void
 
         $this->artisan('migrate')->run();
 
-        $this->assertDatabaseCount('migrations', 1);
+        $this->assertDatabaseCount('migrations', 2);
         $this->assertDatabaseHas('migrations', ['migration' => '2022_08_18_180137_change_migration_actions_table']);
+        $this->assertDatabaseHas('migrations', ['migration' => '2023_01_21_172923_rename_migrations_actions_table_to_actions']);
     }
 }