Skip the hopper doing the pulling. (#2186)

wizjany · web-flow · commit a949b106c665 · 2025-03-02T11:37:17.000-05:00
Closes #2185.
diff --git a/worldguard-bukkit/src/main/java/com/sk89q/worldguard/bukkit/listener/EventAbstractionListener.java b/worldguard-bukkit/src/main/java/com/sk89q/worldguard/bukkit/listener/EventAbstractionListener.java
@@ -1031,7 +1031,9 @@ public void onInventoryMoveItem(InventoryMoveItemEvent event) {
                 handleInventoryHolderUse(event, cause, sourceHolder);
             }
 
-            handleInventoryHolderUse(event, cause, targetHolder);
+            if (causeHolder != null && !causeHolder.equals(targetHolder)) {
+                handleInventoryHolderUse(event, cause, targetHolder);
+            }
 
             if (event.isCancelled() && causeHolder instanceof Hopper && wcfg.breakDeniedHoppers) {
                 Bukkit.getScheduler().scheduleSyncDelayedTask(getPlugin(),

Original file line number	Diff line number	Diff line change
`@@ -1031,7 +1031,9 @@ public void onInventoryMoveItem(InventoryMoveItemEvent event) {`
`1031`	`1031`	`handleInventoryHolderUse(event, cause, sourceHolder);`
`1032`	`1032`	`}`
`1033`	`1033`
`1034`		`- handleInventoryHolderUse(event, cause, targetHolder);`
	`1034`	`+ if (causeHolder != null && !causeHolder.equals(targetHolder)) {`
	`1035`	`+ handleInventoryHolderUse(event, cause, targetHolder);`
	`1036`	`+ }`
`1035`	`1037`
`1036`	`1038`	`if (event.isCancelled() && causeHolder instanceof Hopper && wcfg.breakDeniedHoppers) {`
`1037`	`1039`	`Bukkit.getScheduler().scheduleSyncDelayedTask(getPlugin(),`