From 3275fb0169f82bb05ca8f738dc06ca76545baa57 Mon Sep 17 00:00:00 2001 From: SajmonOriginal Date: Tue, 21 Apr 2026 05:08:14 +0200 Subject: [PATCH] Fix: eating bug caused by incorrect InteractionResultHolder comparison Using != on InteractionResultHolder objects always returns true because each call to InteractionResultHolder.pass() creates a new instance. This caused the onUseItem event to always cancel item usage, preventing players from eating food even when authenticated. Changed to compare getResult() with InteractionResult.PASS instead. --- src/main/java/su/rico040/easyauth/EasyAuth.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/su/rico040/easyauth/EasyAuth.java b/src/main/java/su/rico040/easyauth/EasyAuth.java index 498491e..c6f6b07 100644 --- a/src/main/java/su/rico040/easyauth/EasyAuth.java +++ b/src/main/java/su/rico040/easyauth/EasyAuth.java @@ -124,7 +124,7 @@ public static void onRightClickBlock(PlayerInteractEvent.RightClickBlock event) @SubscribeEvent public static void onUseItem(PlayerInteractEvent.RightClickItem event) { - if (AuthEventHandler.onUseItem(event.getEntity()) != InteractionResultHolder.pass(ItemStack.EMPTY)) { + if (AuthEventHandler.onUseItem(event.getEntity()).getResult() != InteractionResult.PASS) { event.setCanceled(true); } }