From 4816438f9369965663ceedac39d36113e24e3c4f Mon Sep 17 00:00:00 2001 From: Christian Koop Date: Mon, 23 Oct 2023 19:03:41 +0200 Subject: [PATCH] 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 --- .../src/main/java/com/craftaro/core/database/DataManager.java | 4 ++-- 1 file changed, 2 insertions(+), 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 index f15bea51..e5b67e92 100644 --- 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 @@ public class DataManager { } 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(); } /**