diff options
author | Erik Broes <erikbroes@grum.nl> | 2011-03-24 12:11:28 +0100 |
---|---|---|
committer | Erik Broes <erikbroes@grum.nl> | 2011-03-24 12:11:28 +0100 |
commit | d6c3fe23e57068ae3e562a2b4a60f5542a4492fe (patch) | |
tree | a471a773539f4c6a73c2e43d81b5f5d7578fd20a /src/main/java/net/minecraft/server/ItemInWorldManager.java | |
parent | 748b71b3ccb242c762cb937e6943fe7ea3bdb917 (diff) | |
download | craftbukkit-d6c3fe23e57068ae3e562a2b4a60f5542a4492fe.tar craftbukkit-d6c3fe23e57068ae3e562a2b4a60f5542a4492fe.tar.gz craftbukkit-d6c3fe23e57068ae3e562a2b4a60f5542a4492fe.tar.lz craftbukkit-d6c3fe23e57068ae3e562a2b4a60f5542a4492fe.tar.xz craftbukkit-d6c3fe23e57068ae3e562a2b4a60f5542a4492fe.zip |
Minor logic change and small cleanups
Diffstat (limited to 'src/main/java/net/minecraft/server/ItemInWorldManager.java')
-rw-r--r-- | src/main/java/net/minecraft/server/ItemInWorldManager.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/main/java/net/minecraft/server/ItemInWorldManager.java b/src/main/java/net/minecraft/server/ItemInWorldManager.java index d21130f8..1c2dc575 100644 --- a/src/main/java/net/minecraft/server/ItemInWorldManager.java +++ b/src/main/java/net/minecraft/server/ItemInWorldManager.java @@ -195,16 +195,17 @@ public class ItemInWorldManager { boolean result = false; if (i1 > 0) { PlayerInteractEvent event = CraftEventFactory.callPlayerInteractEvent(entityhuman, Action.RIGHT_CLICK_BLOCK, i, j, k, l, itemstack); - if (event.useInteractedBlock() == Event.Result.DENY) {
+ if (event.useInteractedBlock() == Event.Result.DENY) { // If we denied a door from opening, we need to send a correcting update to the client, as it already opened the door. if (i1 == Block.WOODEN_DOOR.id) { boolean bottom = (world.getData(i, j, k) & 8) == 0; - ((EntityPlayer) entityhuman).a.b((Packet) (new Packet53BlockChange(i, j + (bottom ? 1 : -1), k, world)));
+ ((EntityPlayer) entityhuman).a.b((Packet) (new Packet53BlockChange(i, j + (bottom ? 1 : -1), k, world))); } } else { result = Block.byId[i1].a(world, i, j, k, entityhuman); } - if (itemstack != null && event.useItemInHand() != Event.Result.DENY && (!result || event.useItemInHand() == Event.Result.ALLOW)) { + // If we have 'true' and no explicit deny *or* an explicit allow -- run the item part of the hook + if (itemstack != null && ((!result && event.useItemInHand() != Event.Result.DENY) || event.useItemInHand() == Event.Result.ALLOW)) { result = itemstack.a(entityhuman, world, i, j, k, l); } } |