fix (most of) the bugs

1 files changed, 30 insertions, 11 deletions

diff --git a/src/main/java/com/MylesAndMore/tumble/commands/SetAutoStart.java b/src/main/java/com/MylesAndMore/tumble/commands/SetAutoStart.java
index d64573c..0dcb4d9 100644
--- a/src/main/java/com/MylesAndMore/tumble/commands/SetAutoStart.java
+++ b/src/main/java/com/MylesAndMore/tumble/commands/SetAutoStart.java
@@ -1,6 +1,7 @@
 package com.MylesAndMore.tumble.commands;
 
 import com.MylesAndMore.tumble.TumbleManager;
+import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandExecutor;
@@ -16,30 +17,36 @@ public class SetAutoStart implements CommandExecutor{
             // Check if game and lobby worlds are null
             if (TumbleManager.getGameWorld() != null) {
                 if (TumbleManager.getLobbyWorld() != null) {
-                    // Check the player # argument and parse it into an int
-                    int args0 = 0;
-                    try {
-                        args0 = Integer.parseInt(args[0]);
-                    } catch (NumberFormatException nfe){
-                        sender.sendMessage(ChatColor.RED + "Player amount must be a valid number.");
-                    } catch (Exception e){
-                        sender.sendMessage(ChatColor.RED + "Invalid player amount.");
-                    }
                     // Check the amount of args entered
                     if (args.length == 2) {
+                        // Check the player # argument and parse it into an int
+                        int args0;
+                        try {
+                            args0 = Integer.parseInt(args[0]);
+                        } catch (NumberFormatException nfe){
+                            sender.sendMessage(ChatColor.RED + "Player amount must be a valid number.");
+                            return true;
+                        } catch (Exception e){
+                            sender.sendMessage(ChatColor.RED + "Invalid player amount.");
+                            return true;
+                        }
                         // PlayerAmount & enable/disable were entered
                         // Check if a playerAmount between 2-8 was entered
                         if ((args0 >= 2) && (args0 <= 8)) {
                             if (Objects.equals(args[1], "enable")) {
                                 // Write values to the config
                                 TumbleManager.getPlugin().getConfig().set("autoStart.players", args0);
-                                TumbleManager.getPlugin().getConfig().set("autoStart.enabled", args[1]);
+                                TumbleManager.getPlugin().getConfig().set("autoStart.enabled", true);
                                 TumbleManager.getPlugin().saveConfig();
+                                sender.sendMessage(ChatColor.GREEN + "Configuration saved!");
+                                sender.sendMessage(ChatColor.GREEN + "Run " + ChatColor.GRAY +  "/tumble:reload " + ChatColor.GREEN + "the changes to take effect.");
                             }
                             else if (Objects.equals(args[1], "disable")) {
                                 TumbleManager.getPlugin().getConfig().set("autoStart.players", args0);
-                                TumbleManager.getPlugin().getConfig().set("autoStart.enabled", args[1]);
+                                TumbleManager.getPlugin().getConfig().set("autoStart.enabled", false);
                                 TumbleManager.getPlugin().saveConfig();
+                                sender.sendMessage(ChatColor.GREEN + "Configuration saved!");
+                                sender.sendMessage(ChatColor.GREEN + "Run " + ChatColor.GRAY +  "/tumble:reload " + ChatColor.GREEN + "the changes to take effect.");
                             }
                             else {
                                 return false;
@@ -51,9 +58,21 @@ public class SetAutoStart implements CommandExecutor{
                     }
                     else if (args.length == 1) {
                         // Only PlayerAmount was entered
+                        int args0;
+                        try {
+                            args0 = Integer.parseInt(args[0]);
+                        } catch (NumberFormatException nfe){
+                            sender.sendMessage(ChatColor.RED + "Player amount must be a valid number.");
+                            return true;
+                        } catch (Exception e){
+                            sender.sendMessage(ChatColor.RED + "Invalid player amount.");
+                            return true;
+                        }
                         if ((args0 >= 2) && (args0 <= 8)) {
                             TumbleManager.getPlugin().getConfig().set("autoStart.players", args0);
                             TumbleManager.getPlugin().saveConfig();
+                            sender.sendMessage(ChatColor.GREEN + "Configuration saved!");
+                            sender.sendMessage(ChatColor.GREEN + "Run " + ChatColor.GRAY +  "/tumble:reload " + ChatColor.GREEN + "the changes to take effect.");
                         }
                         else {
                             sender.sendMessage(ChatColor.RED + "Please enter a player amount between two and eight!");