Discard dangerous (potentially kick-triggering) clicks in MixinScreenHandler

Author: EvModder

src/main/java/net/evmodder/KeyBound/mixin/MixinScreenHandler.java

@@ -2,12 +2,14 @@
 
 import net.evmodder.KeyBound.Main;
 import net.evmodder.KeyBound.MapClickMoveNeighbors;
+import net.minecraft.client.MinecraftClient;
 import net.minecraft.client.gui.screen.Screen;
 import net.minecraft.entity.player.PlayerEntity;
 import net.minecraft.item.ItemStack;
 import net.minecraft.registry.Registries;
 import net.minecraft.screen.ScreenHandler;
 import net.minecraft.screen.slot.SlotActionType;
+import net.minecraft.text.Text;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.injection.At;
 import org.spongepowered.asm.mixin.injection.Inject;
@@ -18,7 +20,11 @@ public abstract class MixinScreenHandler{
 
 	@Inject(method = "internalOnSlotClick", at = @At("TAIL"))
 	private void add_logic_for_bulk_move_maparts(int slotIndex, int button, SlotActionType actionType, PlayerEntity player, CallbackInfo ci){
-		Main.inventoryUtils.addClick(actionType);
+		if(Main.inventoryUtils.addClick(actionType) > Main.inventoryUtils.MAX_CLICKS){
+			ci.cancel(); // Throw out clicks that exceed the limit!!
+			Main.LOGGER.error("Discarding click in internalOnSlotClick() due to exceeding MAX_CLICKS limit");
+			MinecraftClient.getInstance().player.sendMessage(Text.literal("Discarding unsafe clicks!! > LIMIT").copy().withColor(/*&c=*/16733525), false);
+		}
 		//Main.LOGGER.info("MapMoveClick: click slot "+slotIndex);
 		if(button != 0 || actionType != SlotActionType.PICKUP) return;
 		//Main.LOGGER.info("MapMoveClick: PICKUP");