Don't run database migrations on plugin enable with issues connecting

Migrations rely on `this.databaseConnector` being set which might be true if something fails
craftaro · Oct 23, 2023 · 4816438 · 4816438
1 parent 0bbc5af
commit 4816438
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Core/src/main/java/com/craftaro/core/database/DataManager.java b/Core/src/main/java/com/craftaro/core/database/DataManager.java
@@ -106,13 +106,13 @@ private void load(DatabaseType forcedType) {
             }
             this.type = databaseConnector.getType();
             this.plugin.getLogger().info("Data handler connected using " + databaseConnector.getType().name() + ".");
+
+            runMigrations();
         } catch (Exception ex) {
             this.plugin.getLogger().severe("Fatal error trying to connect to database. Please make sure all your connection settings are correct and try again. Plugin has been disabled.");
             ex.printStackTrace();
             Bukkit.getPluginManager().disablePlugin(this.plugin);
         }
-
-        runMigrations();
     }
 
     /**