fix(install): Enabled option for removing previous migration files

anibalealvarezs · Feb 11, 2022 · dd21137 · dd21137
1 parent 6b05f71
commit dd21137
Show file tree

Hide file tree

Showing 7 changed files with 22 additions and 133 deletions.
diff --git a/src/Commands/PbAltInstallCommand.php b/src/Commands/PbAltInstallCommand.php
@@ -19,6 +19,7 @@ class PbAltInstallCommand extends Command
                             {--update : pachage will be updated before installation}
                             {--publish : Resources will be published to the application}
                             {--migrate : Migrations will be run}
+                            {--reset : Migrations will be removed from migrations directory}
                             {--seed : Tables will be seeded}
                             {--config : Application wil be configured}
                             {--link : Links will be created}

diff --git a/src/Commands/PbInstallCommand.php b/src/Commands/PbInstallCommand.php
@@ -20,6 +20,7 @@ class PbInstallCommand extends Command
                             {--update : pachage will be updated before installation}
                             {--publish : Resources will be published to the application}
                             {--migrate : Migrations will be run}
+                            {--reset : Migrations will be removed from migrations directory}
                             {--seed : Tables will be seeded}
                             {--config : Application wil be configured}
                             {--link : Links will be created}
@@ -59,11 +60,6 @@ public function installInertia(): bool
                 echo "------ [[ ERROR: Jetstream installation failed failed ]]\n";
                 return false;
             }
-            echo "---- Installing providers...\n";
-            if (!$this->installProviders()) {
-                echo "------ [[ ERROR: proviers couldn't be installed ]]\n";
-                return false;
-            }
         } catch (Exception $e) {
             echo "------ [[ ERROR: ".$e->getMessage()." ]]\n";
             return false;

diff --git a/src/Database/Migrations/make_configkey_column_unique.php.stub b/src/Database/Migrations/make_configkey_column_unique.php.stub
diff --git a/src/Database/Migrations/make_modulekey_column_unique.php.stub b/src/Database/Migrations/make_modulekey_column_unique.php.stub
diff --git a/src/Database/Migrations/make_name_column_unique.php.stub b/src/Database/Migrations/make_name_column_unique.php.stub
diff --git a/src/Database/Migrations/rename_logger_and_config_tables.php.stub b/src/Database/Migrations/rename_logger_and_config_tables.php.stub
diff --git a/src/Traits/PbInstallTrait.php b/src/Traits/PbInstallTrait.php
@@ -88,6 +88,13 @@ public function installProjectBuilder(): bool
                 }
             }
             if ($this->option('migrate') || $this->option('seed') || $this->option('all') || (str_starts_with($this->signature,  'pbuilder:update')) || (str_starts_with($this->signature,  'pbuilder:altupdate'))) {
+                if ($this->confirm("'Do you want to remove previous migration files?")) {
+                    echo "------ Removing previous migration files\n";
+                    if (!$this->removePreviousMigrations()) {
+                        echo "-------- [[ ERROR: Error removing migrations files ]]\n";
+                        return false;
+                    }
+                }
                 echo "---- Configurating database...\n";
                 if (!$this->migrateAndSeed()) {
                     return false;
@@ -407,6 +414,19 @@ protected function addPubPath(): bool
         return true;
     }
 
+    /**
+     * Scope a query to only include popular users.
+     *
+     * @return bool
+     */
+    protected function removePreviousMigrations(): bool
+    {
+        if (!shell_exec("rm ".database_path('migrations' . DIRECTORY_SEPARATOR)."*")) {
+            return false;
+        }
+        return true;
+    }
+
     /**
      * Scope a query to only include popular users.
      *