Merge pull request #74 from CraftLuna/feat/command-unregister

Unregister commands onDisable

src/main/java/xyz/geik/farmer/Main.java

@@ -16,6 +16,7 @@
 import xyz.geik.farmer.database.SQL;
 import xyz.geik.farmer.database.SQLite;
 import xyz.geik.farmer.helpers.CacheLoader;
+import xyz.geik.farmer.helpers.CommandHelper;
 import xyz.geik.farmer.helpers.WorldHelper;
 import xyz.geik.farmer.integrations.Integrations;
 import xyz.geik.farmer.listeners.ListenerRegister;
@@ -148,6 +149,7 @@ public void onEnable() {
      */
     public void onDisable() {
         getSql().updateAllFarmers();
+        CommandHelper.unregisterCommands();
     }
 
     /**

src/main/java/xyz/geik/farmer/helpers/CommandHelper.java

@@ -0,0 +1,70 @@
+package xyz.geik.farmer.helpers;
+
+import com.google.common.collect.Lists;
+import org.bukkit.Bukkit;
+import org.bukkit.Server;
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandMap;
+import org.bukkit.command.SimpleCommandMap;
+import org.jetbrains.annotations.NotNull;
+import xyz.geik.glib.shades.triumphteam.cmd.core.exceptions.CommandRegistrationException;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.Method;
+import java.util.Map;
+
+/**
+ * Utility class for unregistering commands
+ *
+ * @author Efe Kurban (hyperion), CraftLuna, TriumphTeam
+ */
+public class CommandHelper {
+
+    /**
+     * Unregisters every command related to Farmer v6
+     */
+    public static void unregisterCommands() {
+        Lists.newArrayList("çiftçi", "farmer", "farm", "fm", "ciftci").forEach(CommandHelper::unregisterCommand);
+    }
+
+    /**
+     * Unregisters a command
+     *
+     * @param name Command's name
+     */
+    private static void unregisterCommand(String name) {
+        getBukkitCommands(getCommandMap()).remove(name);
+    }
+
+    /**
+     * @return CommandMap object from Server
+     */
+    @NotNull
+    private static CommandMap getCommandMap() {
+        try {
+            final Server server = Bukkit.getServer();
+            final Method getCommandMap = server.getClass().getDeclaredMethod("getCommandMap");
+            getCommandMap.setAccessible(true);
+
+            return (CommandMap) getCommandMap.invoke(server);
+        } catch (final Exception ignored) {
+            throw new CommandRegistrationException("Unable get Command Map. Commands will not be registered!");
+        }
+    }
+
+    /**
+     * @return Map of Registered Commands, directly from Bukkit
+     */
+    @NotNull
+    private static Map<String, Command> getBukkitCommands(@NotNull final CommandMap commandMap) {
+        try {
+            final Field bukkitCommands = SimpleCommandMap.class.getDeclaredField("knownCommands");
+            bukkitCommands.setAccessible(true);
+            //noinspection unchecked
+            return (Map<String, Command>) bukkitCommands.get(commandMap);
+        } catch (NoSuchFieldException | IllegalAccessException e) {
+            throw new CommandRegistrationException("Unable get Bukkit commands. Commands might not be registered correctly!");
+        }
+    }
+
+}