diff options
author | feildmaster <admin@feildmaster.com> | 2012-03-25 17:53:59 -0500 |
---|---|---|
committer | Warren Loo <evilseph@gmail.com> | 2012-03-25 18:57:27 -0400 |
commit | c30e339af6dd7594cca7e0c1ca31f0ea34a0afa9 (patch) | |
tree | 0542eb206109014946aa1f8b54afd2cceff7bb0f /src/main/java/org | |
parent | 0c9b59e071194dd737c69809d6f41b22014eeec7 (diff) | |
download | craftbukkit-c30e339af6dd7594cca7e0c1ca31f0ea34a0afa9.tar craftbukkit-c30e339af6dd7594cca7e0c1ca31f0ea34a0afa9.tar.gz craftbukkit-c30e339af6dd7594cca7e0c1ca31f0ea34a0afa9.tar.lz craftbukkit-c30e339af6dd7594cca7e0c1ca31f0ea34a0afa9.tar.xz craftbukkit-c30e339af6dd7594cca7e0c1ca31f0ea34a0afa9.zip |
[Bleeding] Fixed item duping in certain occasions. Fixes BUKKIT-1310
Diffstat (limited to 'src/main/java/org')
-rw-r--r-- | src/main/java/org/bukkit/craftbukkit/event/CraftEventFactory.java | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/main/java/org/bukkit/craftbukkit/event/CraftEventFactory.java b/src/main/java/org/bukkit/craftbukkit/event/CraftEventFactory.java index 39f01117..243c3339 100644 --- a/src/main/java/org/bukkit/craftbukkit/event/CraftEventFactory.java +++ b/src/main/java/org/bukkit/craftbukkit/event/CraftEventFactory.java @@ -528,9 +528,10 @@ public class CraftEventFactory { ((EntityPlayer) player).netServerHandler.sendPacket(packet); } - List<org.bukkit.inventory.ItemStack> drops = new ArrayList<org.bukkit.inventory.ItemStack>(); + List<ItemStack> calculatedDrops = blockType.calculateDrops(world, player, x, y, z, data); + List<org.bukkit.inventory.ItemStack> drops = new ArrayList<org.bukkit.inventory.ItemStack>(calculatedDrops.size()); if (!creative && player.b(blockType)) { - for (ItemStack stack : blockType.calculateDrops(world, player, x, y, z, data)) { + for (ItemStack stack : calculatedDrops) { drops.add(new CraftItemStack(stack)); } } @@ -539,17 +540,17 @@ public class CraftEventFactory { world.getServer().getPluginManager().callEvent(event); if (event.isCancelled()) { - blockType.setDrops(new ArrayList<ItemStack>()); + blockType.dropList.clear(); // Let the client know the block still exists ((EntityPlayer) player).netServerHandler.sendPacket(new Packet53BlockChange(x, y, z, world)); return true; } - ArrayList<ItemStack> toDrop = new ArrayList<ItemStack>(); + ArrayList<ItemStack> toDrop = new ArrayList<ItemStack>(drops.size()); for (org.bukkit.inventory.ItemStack stack : drops) { toDrop.add(CraftItemStack.createNMSItemStack(stack)); } - blockType.setDrops(toDrop); + blockType.dropList = toDrop; return false; // Event not cancelled } |