Fix ClickEvent firing at wrong times

Author: ZeroMemes

src/main/java/clientapi/load/mixin/MixinMinecraft.java

@@ -25,6 +25,7 @@
 import clientapi.event.handle.ClientHandler;
 import clientapi.load.ClientInitException;
 import clientapi.load.mixin.extension.IMinecraft;
+import clientapi.util.io.MouseKeyTracker;
 import clientapi.util.io.StreamReader;
 import clientapi.util.render.gl.GLUtils;
 import com.google.gson.GsonBuilder;
@@ -116,7 +117,10 @@ private void onMouseEvent(CallbackInfo info) {
         if (currentScreen != null)
             return;
 
-        ClientAPI.EVENT_BUS.post(new ClickEvent(Mouse.getEventButton()));
+        int button = Mouse.getEventButton();
+        if (button > 0 && MouseKeyTracker.INSTANCE.wasButtonPressed(button, Mouse.getEventButtonState())) {
+            ClientAPI.EVENT_BUS.post(new ClickEvent(button));
+        }
     }
 
     @Inject(method = "init", at = @At("RETURN"))

src/main/java/clientapi/util/io/MouseKeyTracker.java

@@ -0,0 +1,32 @@
+package clientapi.util.io;
+
+/**
+ * @author Brady
+ * @since 4/13/2018 4:59 PM
+ */
+public enum MouseKeyTracker {
+
+    INSTANCE;
+
+    /**
+     * Array that maps the mouse button id (the index in the array) to
+     * the state of that button. Used to track when the state of the
+     * button changes so that a ClickEvent is correctly dispatched.
+     */
+    private boolean[] mouseKeyStates = new boolean[0];
+
+    public final boolean wasButtonPressed(int button, boolean newState) {
+        // Check if the state array needs to be expanded
+        if (mouseKeyStates.length <= button) {
+            // Create a new array with the required size
+            boolean[] expandedArray = new boolean[button + 1];
+            // Copy the elements from the old array to the new one
+            System.arraycopy(mouseKeyStates, 0, expandedArray, 0, mouseKeyStates.length);
+            // Set the state array to the expanded one
+            mouseKeyStates = expandedArray;
+        }
+
+        // Return true if the mouse is pressed in the new state and wasn't in the old state
+        return newState && !mouseKeyStates[button];
+    }
+}