Begun adding support for data/damage values to the blockbag.

src/main/java/com/sk89q/worldedit/EditSession.java

@@ -209,7 +209,7 @@ public class EditSession {
 
             if (existing > 0) {
                 try {
-                    blockBag.storeDroppedBlock(existing);
+                    blockBag.storeDroppedBlock(existing, world.getBlockData(pt));
                 } catch (BlockBagException e) {
                 }
             }

src/main/java/com/sk89q/worldedit/bags/BlockBag.java

@@ -32,13 +32,15 @@ public abstract class BlockBag {
      * Stores a block as if it was mined.
      * 
      * @param id
+     * @param data 
      * @throws BlockBagException
      */
-    public void storeDroppedBlock(int id) throws BlockBagException {
-        int dropped = BlockType.getDroppedBlock(id);
-        if (dropped > 0) {
-            storeBlock(dropped);
-        }
+    public void storeDroppedBlock(int id, int data) throws BlockBagException {
+        BaseItem dropped = BlockType.getDroppedBlock(id, data);
+        if (dropped == null) return;
+        if (dropped.getType() == BlockID.AIR) return;
+
+        storeItem(dropped);
     }
     
     /**
@@ -122,7 +124,19 @@ public abstract class BlockBag {
      * @param id
      * @throws BlockBagException 
      */
-    public abstract void storeBlock(int id) throws BlockBagException;
+    public void storeBlock(int id) throws BlockBagException {
+        storeItem(new BaseItem(id));
+    }
+    
+    /**
+     * Store a block.
+     * 
+     * @param item
+     * @throws BlockBagException 
+     */
+    public void storeItem(BaseItem item) throws BlockBagException {
+        storeBlock(item.getType());
+    }
     
     /**
      * Checks to see if a block exists without removing it.

src/main/java/com/sk89q/worldedit/bukkit/BukkitPlayerBlockBag.java

@@ -23,6 +23,8 @@ import org.bukkit.entity.Player;
 import org.bukkit.inventory.ItemStack;
 import com.sk89q.worldedit.Vector;
 import com.sk89q.worldedit.bags.*;
+import com.sk89q.worldedit.blocks.BaseItem;
+import com.sk89q.worldedit.blocks.BaseItemStack;
 import com.sk89q.worldedit.blocks.BlockID;
 
 public class BukkitPlayerBlockBag extends BlockBag {
@@ -114,54 +116,71 @@ public class BukkitPlayerBlockBag extends BlockBag {
      * @param id
      */
     @Override
-    public void storeBlock(int id) throws BlockBagException {
+    public void storeItem(BaseItem item) throws BlockBagException {
+        final int id = item.getType();
+        final int damage = item.getDamage();
+        int amount = (item instanceof BaseItemStack) ? ((BaseItemStack) item).getAmount() : 1;
+        assert(amount <= 64);
+        boolean usesDamageValue = false; // TODO: Use ItemType.usesDamageValue once it's fixed.
+
         if (id == BlockID.AIR) {
             throw new IllegalArgumentException("Can't store air block");
         }
 
         loadInventory();
 
-        boolean found = false;
         int freeSlot = -1;
 
         for (int slot = 0; slot < items.length; ++slot) {
-            ItemStack item = items[slot];
+            ItemStack bukkitItem = items[slot];
 
+            if (bukkitItem == null) {
                 // Delay using up a free slot until we know there are no stacks
                 // of this item to merge into
-            if (item == null) {
+                
                 if (freeSlot == -1) {
                     freeSlot = slot;
                 }
                 continue;
             }
 
-            if (item.getTypeId() == id) {
-                int amount = item.getAmount();
+            if (bukkitItem.getTypeId() != id) {
+                // Type id doesn't fit
+                continue;
+            }
 
+            if (usesDamageValue && bukkitItem.getDurability() != damage) {
+                // Damage value doesn't fit.
+                continue;
+            }
+
+            int currentAmount = bukkitItem.getAmount();
+            if (currentAmount < 0) {
                 // Unlimited
-                if (amount < 0) {
+                return;
+            }
+            if (currentAmount >= 64) {
+                // Full stack
+                continue;
+            }
+
+            int spaceLeft = 64 - currentAmount;
+            if (spaceLeft >= amount) {
+                bukkitItem.setAmount(currentAmount + amount);
                 return;
             }
 
-                if (amount < 64) {
-                    item.setAmount(amount + 1);
-                    found = true;
-                    break;
-                }
-            }
+            bukkitItem.setAmount(64);
+            amount -= spaceLeft;
         }
 
-        if (!found && freeSlot > -1) {
-            items[freeSlot] = new ItemStack(id, 1);
-            found = true;
+        if (freeSlot > -1) {
+            items[freeSlot] = new ItemStack(id, amount);
+            return;
         }
 
-        if (found) {
-        } else {
         throw new OutOfSpaceException(id);
     }
-    }
     
     /**
      * Flush any changes. This is called at the end.