Fixed issue where modules with modes would be on by default

Author: ZeroMemes

src/main/java/clientapi/module/Module.java

@@ -159,6 +159,9 @@ public final ModuleMode setMode(ModuleMode mode) {
         if (mode == null || mode.getParent() != this)
             return null;
 
+        if (mode == this.mode)
+            return mode;
+
         if (this.mode != null)
             this.mode.setState(false);
 

src/main/java/clientapi/module/ModuleMode.java

@@ -48,31 +48,35 @@ public class ModuleMode<T extends Module> extends ValueHolder implements IModule
      */
     private Keybind bind;
 
+    /**
+     * The state of the mode
+     */
+    private boolean state;
+
     public ModuleMode(T parent, String name, String description) {
         this.parent = parent;
         this.name = name;
         this.description = description;
 
         this.bind = new Keybind(Keybind.Type.TOGGLE, 0, type -> {
             if (type == Keybind.Action.CLICK) {
-                this.toggle();
+                parent.setState(parent.getMode() != this || !parent.getState());
+                parent.setMode(this);
             }
         });
     }
 
     @Override
     public final void setState(boolean state) {
-        parent.setState(state);
-
-        if (parent.getMode() != this)
-            parent.setMode(this);
-
+        this.state = state;
         if (state) {
-            onEnable();
-            ClientAPI.EVENT_BUS.subscribe(this);
+            if (parent.getState()) {
+                this.onEnable();
+                ClientAPI.EVENT_BUS.subscribe(this);
+            }
         } else {
             ClientAPI.EVENT_BUS.unsubscribe(this);
-            onDisable();
+            this.onDisable();
         }
     }
 
@@ -98,7 +102,7 @@ public final String getDescription() {
 
     @Override
     public final boolean getState() {
-        return this.parent.getState() && this.parent.getMode() == this;
+        return this.state;
     }
 
     @Override