diff --git a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/api/transfer/v1/fluid/FluidStorageUtil.java b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/api/transfer/v1/fluid/FluidStorageUtil.java index 393fd56c0..001a63835 100644 --- a/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/api/transfer/v1/fluid/FluidStorageUtil.java +++ b/fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/api/transfer/v1/fluid/FluidStorageUtil.java @@ -111,7 +111,7 @@ public final class FluidStorageUtil { if (!fill && handItem == Items.POTION) sound = SoundEvents.ITEM_BOTTLE_EMPTY; } - player.playSound(sound, SoundCategory.BLOCKS, 1, 1); + player.getWorld().playSound(player, player.getX(), player.getEyeY(), player.getZ(), sound, SoundCategory.PLAYERS, 1, 1); return true; } diff --git a/fabric-transfer-api-v1/src/testmod/java/net/fabricmc/fabric/test/transfer/ingame/FluidChuteBlock.java b/fabric-transfer-api-v1/src/testmod/java/net/fabricmc/fabric/test/transfer/ingame/FluidChuteBlock.java index 996b4e438..81fbf6ba9 100644 --- a/fabric-transfer-api-v1/src/testmod/java/net/fabricmc/fabric/test/transfer/ingame/FluidChuteBlock.java +++ b/fabric-transfer-api-v1/src/testmod/java/net/fabricmc/fabric/test/transfer/ingame/FluidChuteBlock.java @@ -65,14 +65,18 @@ public class FluidChuteBlock extends Block implements BlockEntityProvider { @Override public ActionResult onUse(BlockState state, World world, BlockPos pos, PlayerEntity player, Hand hand, BlockHitResult hit) { - if (!world.isClient() && world.getBlockEntity(pos) instanceof FluidChuteBlockEntity chute) { + if (world.getBlockEntity(pos) instanceof FluidChuteBlockEntity chute) { if (!FluidStorageUtil.interactWithFluidStorage(chute.storage, player, hand)) { - player.sendMessage( - Text.literal("Fluid: ") - .append(FluidVariantAttributes.getName(chute.storage.variant)) - .append(", amount: " + chute.storage.amount), - false - ); + if (!world.isClient()) { + player.sendMessage( + Text.literal("Fluid: ") + .append(FluidVariantAttributes.getName(chute.storage.variant)) + .append(", amount: " + chute.storage.amount), + false + ); + } + + return ActionResult.CONSUME; } }