From 1dbf37748f618c15364da2bb48f037048bd54b96 Mon Sep 17 00:00:00 2001
From: KyGuy2002 <kylanjaeger@gmail.com>
Date: Wed, 20 Apr 2022 10:46:39 -0700
Subject: [PATCH 1/6] Added tab complete

---
 .../java/network/palace/show/ShowPlugin.java  |  5 ++
 .../show/commands/show/ShowTabComplete.java   | 65 +++++++++++++++
 .../commands/showgen/ShowGenTabComplete.java  | 83 +++++++++++++++++++
 3 files changed, 153 insertions(+)
 create mode 100644 src/main/java/network/palace/show/commands/show/ShowTabComplete.java
 create mode 100644 src/main/java/network/palace/show/commands/showgen/ShowGenTabComplete.java

diff --git a/src/main/java/network/palace/show/ShowPlugin.java b/src/main/java/network/palace/show/ShowPlugin.java
index 46e7012..67ff8b6 100644
--- a/src/main/java/network/palace/show/ShowPlugin.java
+++ b/src/main/java/network/palace/show/ShowPlugin.java
@@ -6,6 +6,8 @@
 import com.craftmend.openaudiomc.spigot.OpenAudioMcSpigot;
 import lombok.Getter;
 import network.palace.show.commands.*;
+import network.palace.show.commands.show.ShowTabComplete;
+import network.palace.show.commands.showgen.ShowGenTabComplete;
 import network.palace.show.generator.ShowGenerator;
 import network.palace.show.listeners.ChunkListener;
 import network.palace.show.listeners.PlayerInteract;
@@ -83,6 +85,8 @@ public void onEnable() {
         openAudioMcSpigot = OpenAudioMcSpigot.getInstance();
         FileUtil.setupFiles();
         this.getCommand("show").setExecutor(new ShowCommand());
+        this.getCommand("show").setTabCompleter(new ShowTabComplete());
+
         this.getCommand("showdebug").setExecutor(new ShowDebugCommand());
 
         FileConfiguration config = this.getConfig();
@@ -90,6 +94,7 @@ public void onEnable() {
         if (config.getString("github.token") != null) {
             githubToken = config.getString("github.token");
             this.getCommand("showgen").setExecutor(new ShowgenCommand());
+            this.getCommand("showgen").setTabCompleter(new ShowGenTabComplete());
             Bukkit.getConsoleSender().sendMessage(ChatColor.GREEN + "[Show] Showgen has been enabled in show!");
         } else {
             Bukkit.getConsoleSender().sendMessage(ChatColor.RED + "[Show] Showgen will not be running in Show! To enable it, add a github token to the config!");
diff --git a/src/main/java/network/palace/show/commands/show/ShowTabComplete.java b/src/main/java/network/palace/show/commands/show/ShowTabComplete.java
new file mode 100644
index 0000000..8ab027e
--- /dev/null
+++ b/src/main/java/network/palace/show/commands/show/ShowTabComplete.java
@@ -0,0 +1,65 @@
+package network.palace.show.commands.show;
+
+import network.palace.show.ShowPlugin;
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandSender;
+import org.bukkit.command.TabCompleter;
+import org.bukkit.util.StringUtil;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.List;
+
+public class ShowTabComplete implements TabCompleter {
+
+    /*
+    Commands:
+
+    /show start <name>
+    /show stop <name>
+    /show reload
+    /show list
+
+    /showgen generate <type> <time>
+    /showgen setinitialscene
+    /showgen setcorner <x,y,z>
+
+    /showdebug
+     */
+
+    private static final String[] baseShowCmds = { "start", "stop", "list", "reload"};
+
+    @Override
+    public List<String> onTabComplete(CommandSender sender, Command command, String alias, String[] args) {
+        final List<String> completions = new ArrayList<>();
+
+        // Return all subcommands if nothing has been typed
+        if (args.length == 0) {
+            completions.addAll(Arrays.asList(baseShowCmds));
+            return completions;
+        }
+
+        // Return applicable commands if started typing
+        if (args.length == 1) {
+            StringUtil.copyPartialMatches(args[0], Arrays.asList(baseShowCmds), completions);
+            return completions;
+        }
+
+        // Handle subcommand args
+        switch (args[0]) {
+            case "start": {
+                // TODO add all not running shows
+                completions.add("Unsupported");
+                return completions;
+            }
+            case "stop": {
+                completions.addAll(ShowPlugin.getShows().keySet());
+                return completions;
+            }
+        }
+
+        // Everything else without args
+        return completions;
+    }
+
+}
diff --git a/src/main/java/network/palace/show/commands/showgen/ShowGenTabComplete.java b/src/main/java/network/palace/show/commands/showgen/ShowGenTabComplete.java
new file mode 100644
index 0000000..d57daf6
--- /dev/null
+++ b/src/main/java/network/palace/show/commands/showgen/ShowGenTabComplete.java
@@ -0,0 +1,83 @@
+package network.palace.show.commands.showgen;
+
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandSender;
+import org.bukkit.command.TabCompleter;
+import org.bukkit.entity.Player;
+import org.bukkit.util.StringUtil;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.List;
+
+public class ShowGenTabComplete implements TabCompleter {
+
+    /*
+    Commands:
+
+    /show start <name>
+    /show stop <name>
+    /show reload
+    /show list
+
+    /showgen generate <type> [top/bottom] <layer-delay> <timestamp>
+    /showgen setinitialscene
+    /showgen setcorner <x,y,z>
+
+    /showdebug
+     */
+
+    private static final String[] baseShowGenCmds = { "generate", "setinitialscene", "setcorner" };
+    private static final String[] actionTypes = { "fakeblock" };
+    private static final String[] topOrBottom = { "top", "bottom" };
+
+    @Override
+    public List<String> onTabComplete(CommandSender sender, Command command, String alias, String[] args) {
+        final List<String> completions = new ArrayList<>();
+
+        // Return all subcommands if nothing has been typed
+        if (args.length == 0) {
+            completions.addAll(Arrays.asList(baseShowGenCmds));
+            return completions;
+        }
+
+        // Return applicable commands if started typing
+        if (args.length == 1) {
+            StringUtil.copyPartialMatches(args[0], Arrays.asList(baseShowGenCmds), completions);
+            return completions;
+        }
+
+        // Handle subcommand args
+        switch (args[0]) {
+            case "generate": {
+
+                // Params
+                if (args.length == 2) {
+                    completions.addAll(Arrays.asList(actionTypes));
+                } else if (args.length == 3) {
+                    completions.addAll(Arrays.asList(topOrBottom));
+                }
+
+                return completions;
+            }
+            case "setcorner": {
+                Player player = (Player) sender;
+
+                // Return players coords
+                if (args.length == 2) { // X
+                    completions.add(player.getLocation().getX()+",");
+                } else if (args.length == 3) { // Y
+                    completions.add(player.getLocation().getY()+",");
+                } else if (args.length == 4) { // Z
+                    completions.add(player.getLocation().getZ()+",");
+                }
+
+                return completions;
+            }
+        }
+
+        // Everything else without args
+        return completions;
+    }
+
+}

From c51aa551c8ac76d5e9fc0666a19918ea8224909e Mon Sep 17 00:00:00 2001
From: KyGuy2002 <kylanjaeger@gmail.com>
Date: Wed, 20 Apr 2022 11:47:12 -0700
Subject: [PATCH 2/6] Finished tab completions, added default help messages

---
 .../palace/show/commands/ShowCommand.java     | 34 +++++++++++++++---
 .../palace/show/commands/ShowgenCommand.java  | 21 ++++++-----
 .../show/commands/show/ShowTabComplete.java   | 36 ++++++++++++++++---
 .../show/commands/show/StartCommand.java      |  2 +-
 .../show/commands/show/StopCommand.java       |  4 ---
 .../commands/showgen/SetCornerCommand.java    | 32 ++++++++++++-----
 .../commands/showgen/ShowGenTabComplete.java  | 12 +++----
 7 files changed, 104 insertions(+), 37 deletions(-)

diff --git a/src/main/java/network/palace/show/commands/ShowCommand.java b/src/main/java/network/palace/show/commands/ShowCommand.java
index 06b94a1..61854a4 100644
--- a/src/main/java/network/palace/show/commands/ShowCommand.java
+++ b/src/main/java/network/palace/show/commands/ShowCommand.java
@@ -21,10 +21,8 @@ public class ShowCommand implements CommandExecutor {
     @Override
     public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
         if (args.length == 0) {
-            sender.sendMessage(ChatColor.GREEN + "Show Commands:");
-            sender.sendMessage(ChatColor.AQUA + "/show list " + ChatColor.GREEN + "- List all running shows");
-            sender.sendMessage(ChatColor.AQUA + "/show start [Show Name] " + ChatColor.GREEN + "- Start a show");
-            sender.sendMessage(ChatColor.AQUA + "/show stop [Show Name] " + ChatColor.GREEN + "- Stop a show");
+            sendHelpMsg(sender);
+            return true;
         } else {
             switch (args[0]) {
                 case "list":
@@ -32,8 +30,16 @@ public boolean onCommand(CommandSender sender, Command command, String label, St
                     break;
                 case "start":
                     if (sender instanceof Player) {
+                        if (args.length < 2) {
+                            sender.sendMessage(ChatColor.RED + "/show start <name>");
+                            return true;
+                        }
                         new StartCommand().handle(sender, args[1], ((Player) sender).getWorld());
                     } else if (sender instanceof BlockCommandSender) {
+                        if (args.length < 2) {
+                            sender.sendMessage(ChatColor.RED + "/show start <name>");
+                            return true;
+                        }
                         new StartCommand().handle(sender, args[1], ((BlockCommandSender) sender).getBlock().getWorld());
                     } else {
                         sender.sendMessage(ChatColor.RED + "You cannot run this from the console!");
@@ -41,18 +47,38 @@ public boolean onCommand(CommandSender sender, Command command, String label, St
                     break;
                 case "stop":
                     if (sender instanceof Player) {
+                        if (args.length < 2) {
+                            sender.sendMessage(ChatColor.RED + "/show stop <name>");
+                            return true;
+                        }
                         new StopCommand().handle(sender, args[1]);
                     } else if (sender instanceof CommandBlock) {
+                        if (args.length < 2) {
+                            sender.sendMessage(ChatColor.RED + "/show stop <name>");
+                            return true;
+                        }
                         new StopCommand().handle(sender, args[1]);
                     } else {
                         sender.sendMessage(ChatColor.RED + "You cannot run this from the console!");
                     }
                     break;
                 default:
+                    sendHelpMsg(sender);
                     break;
             }
         }
 
         return true;
     }
+
+    /**
+     * Sends a help message to the player.
+     * @param sender Who to send it to.
+     */
+    private void sendHelpMsg(CommandSender sender) {
+        sender.sendMessage(ChatColor.GREEN + "Show Commands:");
+        sender.sendMessage(ChatColor.AQUA + "/show list " + ChatColor.GREEN + "- List all running shows");
+        sender.sendMessage(ChatColor.AQUA + "/show start [Show Name] " + ChatColor.GREEN + "- Start a show");
+        sender.sendMessage(ChatColor.AQUA + "/show stop [Show Name] " + ChatColor.GREEN + "- Stop a show");
+    }
 }
diff --git a/src/main/java/network/palace/show/commands/ShowgenCommand.java b/src/main/java/network/palace/show/commands/ShowgenCommand.java
index 852797a..26d3a08 100644
--- a/src/main/java/network/palace/show/commands/ShowgenCommand.java
+++ b/src/main/java/network/palace/show/commands/ShowgenCommand.java
@@ -17,10 +17,7 @@ public class ShowgenCommand implements CommandExecutor {
     @Override
     public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, @NotNull String[] args) {
         if (args.length == 0) {
-            sender.sendMessage(ChatColor.GREEN + "ShowGen Commands:");
-            sender.sendMessage(ChatColor.AQUA + "/showgen generate [action] [bottom/top] [delay per layer] [timestamp]" + ChatColor.GREEN + "- Generate blocks of show actions with one command");
-            sender.sendMessage(ChatColor.AQUA + "/showgen setcorner x,y,z" + ChatColor.GREEN + "- Set the location of the final north-west-bottom corner to help give real coordinate values");
-            sender.sendMessage(ChatColor.AQUA + "/showgen setinitialscene " + ChatColor.GREEN + "- Set the initial scene for a generator session");
+            sendHelpMsg(sender);
             return true;
         }
 
@@ -40,12 +37,20 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
                 new SetInitialCommand().handle(sender);
                 break;
             default:
-                sender.sendMessage(ChatColor.GREEN + "ShowGen Commands:");
-                sender.sendMessage(ChatColor.AQUA + "/showgen generate [action] [bottom/top] [delay per layer] [timestamp]" + ChatColor.GREEN + "- Generate blocks of show actions with one command");
-                sender.sendMessage(ChatColor.AQUA + "/showgen setcorner x,y,z" + ChatColor.GREEN + "- Set the location of the final north-west-bottom corner to help give real coordinate values");
-                sender.sendMessage(ChatColor.AQUA + "/showgen setinitialscene " + ChatColor.GREEN + "- Set the initial scene for a generator session");
+                sendHelpMsg(sender);
                 break;
         }
         return true;
     }
+
+    /**
+     * Sends a help message to the player.
+     * @param sender Who to send it to.
+     */
+    private void sendHelpMsg(CommandSender sender) {
+        sender.sendMessage(ChatColor.GREEN + "ShowGen Commands:");
+        sender.sendMessage(ChatColor.AQUA + "/showgen generate [action] [bottom/top] [delay per layer] [timestamp]" + ChatColor.GREEN + "- Generate blocks of show actions with one command");
+        sender.sendMessage(ChatColor.AQUA + "/showgen setcorner x,y,z" + ChatColor.GREEN + "- Set the location of the final north-west-bottom corner to help give real coordinate values");
+        sender.sendMessage(ChatColor.AQUA + "/showgen setinitialscene " + ChatColor.GREEN + "- Set the initial scene for a generator session");
+    }
 }
diff --git a/src/main/java/network/palace/show/commands/show/ShowTabComplete.java b/src/main/java/network/palace/show/commands/show/ShowTabComplete.java
index 8ab027e..991a2bd 100644
--- a/src/main/java/network/palace/show/commands/show/ShowTabComplete.java
+++ b/src/main/java/network/palace/show/commands/show/ShowTabComplete.java
@@ -1,11 +1,14 @@
 package network.palace.show.commands.show;
 
 import network.palace.show.ShowPlugin;
+import org.bukkit.World;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
 import org.bukkit.command.TabCompleter;
+import org.bukkit.entity.Player;
 import org.bukkit.util.StringUtil;
 
+import java.io.File;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
@@ -17,7 +20,6 @@ public class ShowTabComplete implements TabCompleter {
 
     /show start <name>
     /show stop <name>
-    /show reload
     /show list
 
     /showgen generate <type> <time>
@@ -27,7 +29,7 @@ public class ShowTabComplete implements TabCompleter {
     /showdebug
      */
 
-    private static final String[] baseShowCmds = { "start", "stop", "list", "reload"};
+    private static final String[] baseShowCmds = { "start", "stop", "list" };
 
     @Override
     public List<String> onTabComplete(CommandSender sender, Command command, String alias, String[] args) {
@@ -48,12 +50,11 @@ public List<String> onTabComplete(CommandSender sender, Command command, String
         // Handle subcommand args
         switch (args[0]) {
             case "start": {
-                // TODO add all not running shows
-                completions.add("Unsupported");
+                StringUtil.copyPartialMatches(args[1], Arrays.asList(getStoppedShows(((Player)sender).getWorld())), completions);
                 return completions;
             }
             case "stop": {
-                completions.addAll(ShowPlugin.getShows().keySet());
+                StringUtil.copyPartialMatches(args[1], ShowPlugin.getShows().keySet(), completions);
                 return completions;
             }
         }
@@ -62,4 +63,29 @@ public List<String> onTabComplete(CommandSender sender, Command command, String
         return completions;
     }
 
+    /**
+     * Gets all shows that arent currently running in the world.
+     * @param world What world
+     * @return The show names (minus .show)
+     */
+    private String[] getStoppedShows(World world) {
+        File f = new File("plugins/Show/shows/" + world.getName());
+        String[] fileNames = f.list();
+        if (fileNames == null) return new String[]{};
+
+        ArrayList<String> names = new ArrayList<>(List.of(fileNames));
+
+        // Strip ".show"
+        ArrayList<String> tempNames = new ArrayList<>();
+        for (String name : names) {
+            tempNames.add(name.replaceAll(".show", ""));
+        }
+        names = tempNames;
+
+        // Remove running shows
+        for (String name : ShowPlugin.getShows().keySet()) names.remove(name);
+
+        return names.toArray(new String[0]);
+    }
+
 }
diff --git a/src/main/java/network/palace/show/commands/show/StartCommand.java b/src/main/java/network/palace/show/commands/show/StartCommand.java
index b750413..caf1ada 100644
--- a/src/main/java/network/palace/show/commands/show/StartCommand.java
+++ b/src/main/java/network/palace/show/commands/show/StartCommand.java
@@ -21,7 +21,7 @@ public void handle(CommandSender sender, String filename, World world) {
             return;
         }
         if (filename == null | filename.equals("")) {
-            sender.sendMessage(ChatColor.RED + "/show start [Show Name]");
+            sender.sendMessage(ChatColor.RED + "/show start <name>");
             return;
         }
         if (ShowPlugin.getShows().containsKey(filename)) {
diff --git a/src/main/java/network/palace/show/commands/show/StopCommand.java b/src/main/java/network/palace/show/commands/show/StopCommand.java
index 1c20d6f..cbb32fc 100644
--- a/src/main/java/network/palace/show/commands/show/StopCommand.java
+++ b/src/main/java/network/palace/show/commands/show/StopCommand.java
@@ -12,10 +12,6 @@
 public class StopCommand {
 
     public void handle(CommandSender sender, String filename)  {
-        if (filename == null | filename.equals("")) {
-            sender.sendMessage(ChatColor.RED + "/show stop [Show Name]");
-            return;
-        }
         if (!ShowPlugin.getShows().containsKey(filename)) {
             sender.sendMessage(ChatColor.RED + "----------------------------------------------");
             sender.sendMessage(ChatColor.GOLD + filename + ChatColor.AQUA + " is not running!");
diff --git a/src/main/java/network/palace/show/commands/showgen/SetCornerCommand.java b/src/main/java/network/palace/show/commands/showgen/SetCornerCommand.java
index 2733637..a94fdb8 100644
--- a/src/main/java/network/palace/show/commands/showgen/SetCornerCommand.java
+++ b/src/main/java/network/palace/show/commands/showgen/SetCornerCommand.java
@@ -13,22 +13,36 @@ public class SetCornerCommand {
     public void handle(CommandSender sender, String[] args) {
         Player player = (Player) sender;
         if (args.length < 2) {
-            player.sendMessage(ChatColor.RED + "/showgen setcorner x,y,z");
+            player.sendMessage(ChatColor.RED + "/showgen setcorner <x> <y> <z>");
             return;
         }
-        String[] locData = args[1].split(",");
-        if (locData.length < 3) {
-            player.sendMessage(ChatColor.RED + "/showgen setcorner x,y,z");
-            return;
+
+        String[] locData;
+
+        if (args[1].contains(",")) { // Legacy coordinate format - showgen setcorner x,y,z
+            locData = args[1].split(",");
+            if (locData.length < 3) {
+                player.sendMessage(ChatColor.RED + "/showgen setcorner <x> <y> <z>");
+                return;
+            }
+
+        } else { // New format - showgen setcorner x y z
+            if (args.length != 4) {
+                player.sendMessage(ChatColor.RED + "/showgen setcorner <x> <y> <z>");
+                return;
+            }
+
+            locData = new String[]{args[1], args[2], args[3]};
         }
+
         try {
-            int x = Integer.parseInt(locData[0]);
-            int y = Integer.parseInt(locData[1]);
-            int z = Integer.parseInt(locData[2]);
+            int x = (int) Double.parseDouble(locData[0]);
+            int y = (int) Double.parseDouble(locData[1]);
+            int z = (int) Double.parseDouble(locData[2]);
             World w = player.getWorld();
             GeneratorSession session = ShowPlugin.getShowGenerator().getOrCreateSession(player.getUniqueId());
             session.setCorner(new Location(w, x, y, z));
-            player.sendMessage(ChatColor.GREEN + "Set north-west-bottom corner to " + x + "," + y + "," + z + "!");
+            player.sendMessage(ChatColor.GREEN + "Set north-west-bottom corner to " + x + ", " + y + ", " + z + "!");
         } catch (NumberFormatException e) {
             player.sendMessage(ChatColor.RED + "Are you sure those are all numbers?");
         }
diff --git a/src/main/java/network/palace/show/commands/showgen/ShowGenTabComplete.java b/src/main/java/network/palace/show/commands/showgen/ShowGenTabComplete.java
index d57daf6..a3a348a 100644
--- a/src/main/java/network/palace/show/commands/showgen/ShowGenTabComplete.java
+++ b/src/main/java/network/palace/show/commands/showgen/ShowGenTabComplete.java
@@ -53,9 +53,9 @@ public List<String> onTabComplete(CommandSender sender, Command command, String
 
                 // Params
                 if (args.length == 2) {
-                    completions.addAll(Arrays.asList(actionTypes));
-                } else if (args.length == 3) {
-                    completions.addAll(Arrays.asList(topOrBottom));
+                    StringUtil.copyPartialMatches(args[1], Arrays.asList(actionTypes), completions);
+                } else if (args.length == 3 && !args[2].equals("")) { // Tab complete when start typing, but not if empty
+                    StringUtil.copyPartialMatches(args[2], Arrays.asList(topOrBottom), completions);
                 }
 
                 return completions;
@@ -65,11 +65,11 @@ public List<String> onTabComplete(CommandSender sender, Command command, String
 
                 // Return players coords
                 if (args.length == 2) { // X
-                    completions.add(player.getLocation().getX()+",");
+                    completions.add(player.getLocation().getBlockX()+"");
                 } else if (args.length == 3) { // Y
-                    completions.add(player.getLocation().getY()+",");
+                    completions.add(player.getLocation().getBlockY()+"");
                 } else if (args.length == 4) { // Z
-                    completions.add(player.getLocation().getZ()+",");
+                    completions.add(player.getLocation().getBlockZ()+"");
                 }
 
                 return completions;

From bd84c9530a9527f19b4b1ff2104de4c8f6771cfe Mon Sep 17 00:00:00 2001
From: KyGuy2002 <kylanjaeger@gmail.com>
Date: Wed, 20 Apr 2022 12:40:28 -0700
Subject: [PATCH 3/6] changed return value so usage isnt sent to player every
 time.

---
 .../show/commands/ShowDebugCommand.java       | 23 +++++++++++--------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/src/main/java/network/palace/show/commands/ShowDebugCommand.java b/src/main/java/network/palace/show/commands/ShowDebugCommand.java
index 4b72db8..94e862c 100644
--- a/src/main/java/network/palace/show/commands/ShowDebugCommand.java
+++ b/src/main/java/network/palace/show/commands/ShowDebugCommand.java
@@ -11,17 +11,20 @@ public class ShowDebugCommand implements CommandExecutor {
 
     @Override
     public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
-        if (sender instanceof Player) {
-            Player player = (Player) sender;
+        if (!(sender instanceof Player)) {
+            sender.sendMessage(ChatColor.RED + "Debug is always enabled for console!");
+            return true;
+        }
+
+        Player player = (Player) sender;
 
-            if (!ShowPlugin.debugMap.containsKey(player.getDisplayName())) {
-                ShowPlugin.debugMap.put(player.getDisplayName(), true);
-                player.sendMessage(ChatColor.AQUA + "[ShowDebug] - " + ChatColor.GREEN + "Enabled");
-            } else {
-                ShowPlugin.debugMap.remove(player.getDisplayName());
-                player.sendMessage(ChatColor.AQUA + "[ShowDebug] - " + ChatColor.RED + "Disabled");
-            }
+        if (!ShowPlugin.debugMap.containsKey(player.getDisplayName())) {
+            ShowPlugin.debugMap.put(player.getDisplayName(), true);
+            player.sendMessage(ChatColor.AQUA + "[ShowDebug] - " + ChatColor.GREEN + "Enabled");
+        } else {
+            ShowPlugin.debugMap.remove(player.getDisplayName());
+            player.sendMessage(ChatColor.AQUA + "[ShowDebug] - " + ChatColor.RED + "Disabled");
         }
-        return false;
+        return true;
     }
 }

From 090107322e0300e9ef2262e64b4ebcab786f11a8 Mon Sep 17 00:00:00 2001
From: KyGuy2002 <kylanjaeger@gmail.com>
Date: Fri, 22 Apr 2022 10:33:15 -0700
Subject: [PATCH 4/6] Revert "changed return value so usage isnt sent to player
 every time."

This reverts commit bd84c9530a9527f19b4b1ff2104de4c8f6771cfe.
---
 .../show/commands/ShowDebugCommand.java       | 23 ++++++++-----------
 1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/src/main/java/network/palace/show/commands/ShowDebugCommand.java b/src/main/java/network/palace/show/commands/ShowDebugCommand.java
index 94e862c..4b72db8 100644
--- a/src/main/java/network/palace/show/commands/ShowDebugCommand.java
+++ b/src/main/java/network/palace/show/commands/ShowDebugCommand.java
@@ -11,20 +11,17 @@ public class ShowDebugCommand implements CommandExecutor {
 
     @Override
     public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
-        if (!(sender instanceof Player)) {
-            sender.sendMessage(ChatColor.RED + "Debug is always enabled for console!");
-            return true;
-        }
-
-        Player player = (Player) sender;
+        if (sender instanceof Player) {
+            Player player = (Player) sender;
 
-        if (!ShowPlugin.debugMap.containsKey(player.getDisplayName())) {
-            ShowPlugin.debugMap.put(player.getDisplayName(), true);
-            player.sendMessage(ChatColor.AQUA + "[ShowDebug] - " + ChatColor.GREEN + "Enabled");
-        } else {
-            ShowPlugin.debugMap.remove(player.getDisplayName());
-            player.sendMessage(ChatColor.AQUA + "[ShowDebug] - " + ChatColor.RED + "Disabled");
+            if (!ShowPlugin.debugMap.containsKey(player.getDisplayName())) {
+                ShowPlugin.debugMap.put(player.getDisplayName(), true);
+                player.sendMessage(ChatColor.AQUA + "[ShowDebug] - " + ChatColor.GREEN + "Enabled");
+            } else {
+                ShowPlugin.debugMap.remove(player.getDisplayName());
+                player.sendMessage(ChatColor.AQUA + "[ShowDebug] - " + ChatColor.RED + "Disabled");
+            }
         }
-        return true;
+        return false;
     }
 }

From 6400888f3da57f81672423444d543531a6409554 Mon Sep 17 00:00:00 2001
From: KyGuy2002 <kylanjaeger@gmail.com>
Date: Fri, 22 Apr 2022 10:44:17 -0700
Subject: [PATCH 5/6] Removed duplicate filename check

---
 .../java/network/palace/show/commands/show/StartCommand.java  | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/main/java/network/palace/show/commands/show/StartCommand.java b/src/main/java/network/palace/show/commands/show/StartCommand.java
index caf1ada..6fd2d4c 100644
--- a/src/main/java/network/palace/show/commands/show/StartCommand.java
+++ b/src/main/java/network/palace/show/commands/show/StartCommand.java
@@ -20,10 +20,6 @@ public void handle(CommandSender sender, String filename, World world) {
             sender.sendMessage(ChatColor.RED + "Invalid world!");
             return;
         }
-        if (filename == null | filename.equals("")) {
-            sender.sendMessage(ChatColor.RED + "/show start <name>");
-            return;
-        }
         if (ShowPlugin.getShows().containsKey(filename)) {
             sender.sendMessage(ChatColor.RED + "----------------------------------------------");
             sender.sendMessage(ChatColor.RED + "That show is already running!");

From 1d5b6f130d7bd3ebed1b58bc2cf2e448e57dec4c Mon Sep 17 00:00:00 2001
From: KyGuy2002 <kylanjaeger@gmail.com>
Date: Thu, 28 Apr 2022 16:41:17 -0700
Subject: [PATCH 6/6] Only loads show names for tab completion on startup.

---
 .../show/commands/show/ShowTabComplete.java   | 51 +++++++++++++------
 1 file changed, 35 insertions(+), 16 deletions(-)

diff --git a/src/main/java/network/palace/show/commands/show/ShowTabComplete.java b/src/main/java/network/palace/show/commands/show/ShowTabComplete.java
index 991a2bd..835a08c 100644
--- a/src/main/java/network/palace/show/commands/show/ShowTabComplete.java
+++ b/src/main/java/network/palace/show/commands/show/ShowTabComplete.java
@@ -1,11 +1,9 @@
 package network.palace.show.commands.show;
 
 import network.palace.show.ShowPlugin;
-import org.bukkit.World;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
 import org.bukkit.command.TabCompleter;
-import org.bukkit.entity.Player;
 import org.bukkit.util.StringUtil;
 
 import java.io.File;
@@ -15,6 +13,11 @@
 
 public class ShowTabComplete implements TabCompleter {
 
+    List<String> shows;
+    public ShowTabComplete() {
+        shows = getShows();
+    }
+
     /*
     Commands:
 
@@ -50,7 +53,7 @@ public List<String> onTabComplete(CommandSender sender, Command command, String
         // Handle subcommand args
         switch (args[0]) {
             case "start": {
-                StringUtil.copyPartialMatches(args[1], Arrays.asList(getStoppedShows(((Player)sender).getWorld())), completions);
+                StringUtil.copyPartialMatches(args[1], getStoppedShows(), completions);
                 return completions;
             }
             case "stop": {
@@ -64,28 +67,44 @@ public List<String> onTabComplete(CommandSender sender, Command command, String
     }
 
     /**
-     * Gets all shows that arent currently running in the world.
-     * @param world What world
-     * @return The show names (minus .show)
+     * Gets stopped shows, not from disk.
+     * @return Stopped show names (minus .show)
      */
-    private String[] getStoppedShows(World world) {
-        File f = new File("plugins/Show/shows/" + world.getName());
-        String[] fileNames = f.list();
-        if (fileNames == null) return new String[]{};
+    private List<String> getStoppedShows() {
+        List<String> stoppedShows = shows;
+        for (String name : ShowPlugin.getShows().keySet()) stoppedShows.remove(name);
+        return stoppedShows;
+    }
 
-        ArrayList<String> names = new ArrayList<>(List.of(fileNames));
+    /**
+     * Gets all shows directly from disk.
+     * @return The show names (minus .show)
+     */
+    private List<String> getShows() {
+        List<String> names = new ArrayList<>();
+        listFiles("plugins/Show/shows/", names);
 
         // Strip ".show"
         ArrayList<String> tempNames = new ArrayList<>();
         for (String name : names) {
             tempNames.add(name.replaceAll(".show", ""));
         }
-        names = tempNames;
-
-        // Remove running shows
-        for (String name : ShowPlugin.getShows().keySet()) names.remove(name);
+        return tempNames;
+    }
 
-        return names.toArray(new String[0]);
+    private void listFiles(String directoryName, List<String> files) {
+        File root = new File(directoryName);
+
+        // Get all files from a directory.
+        File[] fList = root.listFiles();
+        if(fList != null)
+            for (File file : fList) {
+                if (file.isFile()) {
+                    files.add(file.getName());
+                } else if (file.isDirectory()) {
+                    listFiles(file.getAbsolutePath(), files);
+                }
+            }
     }
 
 }