fix potential log spam

changelog.md

@@ -1,3 +1,4 @@
 - add support for 1.21.4 fabric
 - correctly deactivate buttons in the config screen
-- update config screen translations
+- update config screen translations
+- fix log spam when items affected by autodrop where moved into your inventory while having a container opened

gradle.properties

@@ -12,6 +12,6 @@ mod.id=autodrop
 mod.name=autodrop
 mod.description=Simply drops specific items after pickup
 mod.beta=0
-mod.major-version=2.1.0
+mod.major-version=2.2.0
 mod.mixins=autodrop.mixins.json
 mod.supported-loaders=fabric,quilt,neoforge

src/main/java/dev/nyon/autodrop/mixins/InventoryMixin.java

@@ -40,4 +40,18 @@ public void invokeOnTake(
             .getTriggerConfig()
             .getOnPickup()) AutoDrop.INSTANCE.invokeAutodrop();
     }
+
+    @Inject(
+        method = "addResource(ILnet/minecraft/world/item/ItemStack;)I",
+        at = @At("RETURN")
+    )
+    public void invokeOnResourceTake(
+        int slot,
+        ItemStack stack,
+        CallbackInfoReturnable<Integer> cir
+    ){
+        if (ConfigHandlerKt.getConfig()
+            .getTriggerConfig()
+            .getOnPickup()) AutoDrop.INSTANCE.invokeAutodrop();
+    }
 }

src/main/kotlin/dev/nyon/autodrop/AutoDrop.kt

@@ -49,7 +49,6 @@ object AutoDrop {
                     val amountValid = itemStack.count >= identifier.amount
 
                     /*? if >=1.21 {*/
-
                     val componentValid =
                         identifier.components.isEmpty || identifier.components.entrySet().all { (key, component) ->
                             val identifierComponent = component.get()
@@ -90,11 +89,7 @@ object AutoDrop {
                 }
 
 
-                if (!isValid) return@forEach
-
-                minecraft.gameMode?.handleInventoryMouseClick(
-                    screen.menu.containerId, slot.index, 1, ClickType.THROW, player
-                )
+                if (isValid) minecraft.gameMode?.handleInventoryMouseClick(player.containerMenu.containerId, slot.index, 1, ClickType.THROW, player)
             }
 
             jobWaiting = false