Fixed bonemeal not being consumed on failed growth attempts. This fixes BUKKIT-337. Thanks to md-5 for the pull request!

src/main/java/net/minecraft/server/BlockMushroom.java

@@ -108,10 +108,13 @@ public class BlockMushroom extends BlockFlower {
             event = new StructureGrowEvent(location, TreeType.RED_MUSHROOM, bonemeal, player, new ArrayList<BlockState>());
             worldgenhugemushroom = new WorldGenHugeMushroom(1);
         }
-        if (worldgenhugemushroom != null && event != null){
+        if (worldgenhugemushroom != null && event != null) {
             grown = worldgenhugemushroom.grow(world, random, i, j, k, event, itemstack, world.getWorld());
+            if (event.isFromBonemeal() && itemstack != null) {
+                --itemstack.count;
+            }
         }
-        if (!grown || event.isCancelled()){
+        if (!grown || event.isCancelled()) {
             world.setRawTypeIdAndData(i, j, k, this.id, l);
             return false;
         }

src/main/java/net/minecraft/server/BlockSapling.java

@@ -70,6 +70,9 @@ public class BlockSapling extends BlockFlower {
         if (event == null) {
             return;
         }
+        if (event.isFromBonemeal() && itemstack != null) {
+            --itemstack.count;
+        }
         if (!grownTree || event.isCancelled()) {
             world.setRawTypeIdAndData(i, j, k, this.id, l);
         }
@@ -82,6 +85,7 @@ public class BlockSapling extends BlockFlower {
 
     // CraftBukkit start
     private class BlockChangeWithNotify implements BlockChangeDelegate {
+
         World world;
 
         BlockChangeWithNotify(World world) {

src/main/java/net/minecraft/server/WorldGenBigTree.java

@@ -401,9 +401,6 @@ public class WorldGenBigTree extends WorldGenerator {
                     for (BlockState state : event.getBlocks()) {
                         state.update();
                     }
-                    if (event.isFromBonemeal() && itemstack != null) {
-                        --itemstack.count;
-                    }
                 }
             }
             // Craftbukkit end

src/main/java/net/minecraft/server/WorldGenForest.java

@@ -126,9 +126,6 @@ public class WorldGenForest extends WorldGenerator {
                             for (BlockState state : event.getBlocks()) {
                                 state.update(true);
                             }
-                            if (event.isFromBonemeal() && itemstack != null) {
-                                --itemstack.count;
-                            }
                         }
                     }
                     // Craftbukkit end

src/main/java/net/minecraft/server/WorldGenHugeMushroom.java

@@ -196,9 +196,6 @@ public class WorldGenHugeMushroom extends WorldGenerator {
                             for (BlockState state : event.getBlocks()) {
                                 state.update(true);
                             }
-                            if (event.isFromBonemeal() && itemstack != null) {
-                                --itemstack.count;
-                            }
                         }
                     }
                     // Craftbukkit end

src/main/java/net/minecraft/server/WorldGenTaiga2.java

@@ -144,9 +144,6 @@ public class WorldGenTaiga2 extends WorldGenerator {
                             for (BlockState state : event.getBlocks()) {
                                 state.update(true);
                             }
-                            if (event.isFromBonemeal() && itemstack != null) {
-                                --itemstack.count;
-                            }
                         }
                     }
                     // Craftbukkit end

src/main/java/net/minecraft/server/WorldGenTrees.java

@@ -124,9 +124,6 @@ public class WorldGenTrees extends WorldGenerator {
                             for (BlockState state : event.getBlocks()) {
                                 state.update(true);
                             }
-                            if (event.isFromBonemeal() && itemstack != null) {
-                                --itemstack.count;
-                            }
                         }
                     }
                     // Craftbukkit end