From e6d3d5fdec379ade31b856aa5531e7f4c4d24c2f Mon Sep 17 00:00:00 2001
From: Bryson Steck <steck.bryson@gmail.com>
Date: Mon, 13 Sep 2021 15:45:51 -0600
Subject: plugin continues to run after disabling"

---
 out/artifacts/Resurrection_jar/Resurrection.jar    | Bin 1600786 -> 1600890 bytes
 src/net/brysonsteck/Resurrection/Resurrection.java |  33 +++++++++++++--------
 .../Resurrection/startup/ParseSettings.java        |   2 ++
 3 files changed, 22 insertions(+), 13 deletions(-)

diff --git a/out/artifacts/Resurrection_jar/Resurrection.jar b/out/artifacts/Resurrection_jar/Resurrection.jar
index e0f4134..3252640 100644
Binary files a/out/artifacts/Resurrection_jar/Resurrection.jar and b/out/artifacts/Resurrection_jar/Resurrection.jar differ
diff --git a/src/net/brysonsteck/Resurrection/Resurrection.java b/src/net/brysonsteck/Resurrection/Resurrection.java
index a4a70a4..3db60ed 100644
--- a/src/net/brysonsteck/Resurrection/Resurrection.java
+++ b/src/net/brysonsteck/Resurrection/Resurrection.java
@@ -4,6 +4,7 @@ import net.brysonsteck.Resurrection.commands.*;
 import net.brysonsteck.Resurrection.player.PlayerListener;
 import net.brysonsteck.Resurrection.startup.CheckForUpdate;
 import net.brysonsteck.Resurrection.startup.ParseSettings;
+import org.bukkit.Bukkit;
 import org.bukkit.event.Listener;
 import org.bukkit.plugin.PluginDescriptionFile;
 import org.bukkit.plugin.java.JavaPlugin;
@@ -71,6 +72,8 @@ public class Resurrection extends JavaPlugin implements Listener {
         // check if playerData.resurrection exists
         File playerFile = new File("plugins/playerData.resurrection");
         File settingsFile = new File("plugins/settings.resurrection");
+
+        boolean fileFail = false;
         if (!playerFile.exists()) {
             System.out.println("[Resurrection] Player data file does not exist. Creating now in the \"plugins\" directory...");
             try {
@@ -80,12 +83,14 @@ public class Resurrection extends JavaPlugin implements Listener {
                 System.out.println("[Resurrection] An error has occurred creating the player data file!");
                 e.printStackTrace();
                 System.out.println("[Resurrection] This file is crucial to Resurrection. Since the file could not be created, the plugin will now stop.");
+                Bukkit.getPluginManager().disablePlugin(this);
+                fileFail = true;
             }
         } else {
             System.out.println("[Resurrection] The player data file has been found!");
         }
         if (!settingsFile.exists()) {
-            System.out.println("[Resurrection] Settings file does not exist. (This file is new with the 0.2 beta if you upgraded.) Creating now in the \"plugins\" directory...");
+            System.out.println("[Resurrection] Settings file does not exist. Creating now in the \"plugins\" directory...");
             parseSettings = new ParseSettings();
             System.out.println("[Resurrection] Settings file created successfully.");
         } else {
@@ -94,20 +99,22 @@ public class Resurrection extends JavaPlugin implements Listener {
 
         System.out.println("[Resurrection] ---------------------------------------------------------");
 
-        System.out.println("[Resurrection] Registering listeners and adding commands...");
-        // register listener
-        this.getServer().getPluginManager().registerEvents(new PlayerListener(), this);
+        if (parseSettings.isSettingsComplete() && !fileFail) {
+            System.out.println("[Resurrection] Registering listeners and adding commands...");
+            // register listener
+            this.getServer().getPluginManager().registerEvents(new PlayerListener(), this);
 
-        // register commands
-        this.getCommand("about").setExecutor(new CommandAbout(parseSettings.getSetting("debug"), pluginInfo.getVersion(), outdated));
-        this.getCommand("bug").setExecutor(new CommandBug(parseSettings.getSetting("debug")));
-        this.getCommand("resurrect").setExecutor(new CommandResurrect(parseSettings.getSetting("debug")));
-        this.getCommand("howlong").setExecutor(new CommandHowLong(parseSettings.getSetting("debug")));
-        this.getCommand("source").setExecutor(new CommandSource(parseSettings.getSetting("debug")));
+            // register commands
+            this.getCommand("about").setExecutor(new CommandAbout(parseSettings.getSetting("debug"), pluginInfo.getVersion(), outdated));
+            this.getCommand("bug").setExecutor(new CommandBug(parseSettings.getSetting("debug")));
+            this.getCommand("resurrect").setExecutor(new CommandResurrect(parseSettings.getSetting("debug")));
+            this.getCommand("howlong").setExecutor(new CommandHowLong(parseSettings.getSetting("debug")));
+            this.getCommand("source").setExecutor(new CommandSource(parseSettings.getSetting("debug")));
 
-        System.out.println("[Resurrection] ---------------------------------------------------------");
-        System.out.println("[Resurrection] Successfully Started!");
-        System.out.println("[Resurrection] ---------------------------------------------------------");
+            System.out.println("[Resurrection] ---------------------------------------------------------");
+            System.out.println("[Resurrection] Successfully Started!");
+            System.out.println("[Resurrection] ---------------------------------------------------------");
+        }
 
     }
 
diff --git a/src/net/brysonsteck/Resurrection/startup/ParseSettings.java b/src/net/brysonsteck/Resurrection/startup/ParseSettings.java
index 450f750..6dc941e 100644
--- a/src/net/brysonsteck/Resurrection/startup/ParseSettings.java
+++ b/src/net/brysonsteck/Resurrection/startup/ParseSettings.java
@@ -104,4 +104,6 @@ public class ParseSettings {
     public String getSetting(String setting) {
         return settings.get(setting);
     }
+
+    public boolean isSettingsComplete() { return settingsComplete; }
 }
-- 
cgit v1.2.3