fix: project bench shift-click dupe issues

MrTJP · May 1, 2024 · 5f123e9 · 5f123e9
1 parent 3b5ccd1
commit 5f123e9
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/...n/src/main/java/mrtjp/projectred/expansion/inventory/container/ProjectBenchContainer.java b/...n/src/main/java/mrtjp/projectred/expansion/inventory/container/ProjectBenchContainer.java
@@ -118,6 +118,10 @@ public ItemStack quickMoveStack(Player player, int slotIndex) {
             if (!moveToStorage(stack, true) && !moveToEntireInventory(stack, false)) return ItemStack.EMPTY;
 
         } else if (isResultSlot(slotIndex)) {
+            // Usualy, mayPickup is only queried once, and then this method is called repeatedly until EMPTY is returned.
+            // Since this slot is allowed to remain non-empty even after pickup is not possible, we must check this every time.
+            // See AbstractContainerMenu#doClick
+            if (!slot.mayPickup(player)) return ItemStack.EMPTY;
             if (!moveToEntireInventory(stack, true) && !moveToStorage(stack, true)) return ItemStack.EMPTY;
 
         } else if (isPlayerInventory(slotIndex) || isHotbar(slotIndex)) {