From 3134f6b9f396d690f129934676b885b1cf34f729 Mon Sep 17 00:00:00 2001 From: Nicholas Ignoffo Date: Tue, 28 Mar 2017 18:40:24 -0700 Subject: [PATCH] Fix bound tools not checking if they're bound (#1114) --- src/main/java/WayofTime/bloodmagic/item/ItemBoundSword.java | 5 +++++ src/main/java/WayofTime/bloodmagic/item/ItemBoundTool.java | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/src/main/java/WayofTime/bloodmagic/item/ItemBoundSword.java b/src/main/java/WayofTime/bloodmagic/item/ItemBoundSword.java index 418f4d56..23efa140 100644 --- a/src/main/java/WayofTime/bloodmagic/item/ItemBoundSword.java +++ b/src/main/java/WayofTime/bloodmagic/item/ItemBoundSword.java @@ -75,6 +75,11 @@ public class ItemBoundSword extends ItemSword implements IBindable, IActivatable @Override public void onUpdate(ItemStack stack, World world, Entity entity, int itemSlot, boolean isSelected) { + if (Strings.isNullOrEmpty(getOwnerUUID(stack))) { + setActivatedState(stack, false); + return; + } + if (entity instanceof EntityPlayer && getActivated(stack) && world.getTotalWorldTime() % 80 == 0) NetworkHelper.getSoulNetwork(getOwnerUUID(stack)).syphonAndDamage((EntityPlayer) entity, 20); } diff --git a/src/main/java/WayofTime/bloodmagic/item/ItemBoundTool.java b/src/main/java/WayofTime/bloodmagic/item/ItemBoundTool.java index 4ecef2e2..77ff7710 100644 --- a/src/main/java/WayofTime/bloodmagic/item/ItemBoundTool.java +++ b/src/main/java/WayofTime/bloodmagic/item/ItemBoundTool.java @@ -90,6 +90,11 @@ public class ItemBoundTool extends ItemTool implements IBindable, IActivatable @Override public void onUpdate(ItemStack stack, World world, Entity entity, int itemSlot, boolean isSelected) { + if (Strings.isNullOrEmpty(getOwnerUUID(stack))) { + setActivatedState(stack, false); + return; + } + if (entity instanceof EntityPlayer && getActivated(stack) && isSelected && getBeingHeldDown(stack) && stack == ((EntityPlayer) entity).getActiveItemStack()) { EntityPlayer player = (EntityPlayer) entity;