From c30e339af6dd7594cca7e0c1ca31f0ea34a0afa9 Mon Sep 17 00:00:00 2001
From: feildmaster <admin@feildmaster.com>
Date: Sun, 25 Mar 2012 17:53:59 -0500
Subject: [Bleeding] Fixed item duping in certain occasions. Fixes BUKKIT-1310

---
 .../java/org/bukkit/craftbukkit/event/CraftEventFactory.java  | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

(limited to 'src/main/java/org')

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
     }
-- 
cgit v1.2.3