Fix #1074: Rate limit barrel left clicking in survival mode (#1079)

Author: Technici4n

src/client/java/aztech/modern_industrialization/proxy/ClientProxy.java

@@ -86,6 +86,14 @@ public boolean shouldSteamDrillForceBreakReset() {
         return false;
     }
 
+    @Override
+    public void delayNextBlockAttack(Player player) {
+        if (player == Minecraft.getInstance().player) {
+            // Add a 5 tick delay like vanilla.
+            Minecraft.getInstance().gameMode.destroyDelay = 5;
+        }
+    }
+
     @Override
     public boolean hasShiftDown() {
         return Screen.hasShiftDown();

src/main/java/aztech/modern_industrialization/blocks/storage/barrel/BarrelBlock.java

@@ -25,6 +25,7 @@
 
 import aztech.modern_industrialization.blocks.storage.AbstractStorageBlock;
 import aztech.modern_industrialization.blocks.storage.StorageBehaviour;
+import aztech.modern_industrialization.proxy.CommonProxy;
 import aztech.modern_industrialization.thirdparty.fabrictransfer.api.item.ItemVariant;
 import aztech.modern_industrialization.thirdparty.fabrictransfer.api.transaction.Transaction;
 import aztech.modern_industrialization.util.MobSpawning;
@@ -152,6 +153,9 @@ public static void setupBarrelEvents() {
             }
 
             if (attackBlock(event.getPos(), event.getFace(), event.getHand(), event.getEntity(), event.getLevel())) {
+                // NeoForge injects in such a way that the attack will be delayed in creative already,
+                // but we want to delay the attack in survival too to prevent the player from ending up with 5 stacks after 0.25s.
+                CommonProxy.INSTANCE.delayNextBlockAttack(event.getEntity());
                 event.setCanceled(true);
             }
         });

src/main/java/aztech/modern_industrialization/proxy/CommonProxy.java

@@ -97,6 +97,9 @@ public boolean shouldSteamDrillForceBreakReset() {
         return false;
     }
 
+    public void delayNextBlockAttack(Player player) {
+    }
+
     public boolean hasShiftDown() {
         return false;
     }

src/main/resources/META-INF/accesstransformer.cfg

@@ -29,3 +29,6 @@ public net.minecraft.world.entity.npc.VillagerTrades$ItemsForEmeralds
 
 # Used for jetpack
 public net.minecraft.server.network.ServerGamePacketListenerImpl aboveGroundTickCount
+
+# Used to delay attacks after a barrel interaction
+public net.minecraft.client.multiplayer.MultiPlayerGameMode destroyDelay