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
This commit is contained in:
Christian Koop 2023-10-23 19:03:41 +02:00
parent 0bbc5afd2f
commit 4816438f93
No known key found for this signature in database
GPG Key ID: 89A8181384E010A3

View File

@ -106,13 +106,13 @@ public class DataManager {
} }
this.type = databaseConnector.getType(); this.type = databaseConnector.getType();
this.plugin.getLogger().info("Data handler connected using " + databaseConnector.getType().name() + "."); this.plugin.getLogger().info("Data handler connected using " + databaseConnector.getType().name() + ".");
runMigrations();
} catch (Exception ex) { } 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."); 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(); ex.printStackTrace();
Bukkit.getPluginManager().disablePlugin(this.plugin); Bukkit.getPluginManager().disablePlugin(this.plugin);
} }
runMigrations();
} }
/** /**