Maintain old setType method in WorldGenerator. Fixes BUKKIT-4915

WorldGenerator setType and setTypeAndData have their arguments changed to
add in support for CraftBlockChangeDelegate, which changes the method
signature. This change in the method signature breaks any WorldGenerators
that aren't modified to use CraftBlockChangeDelegate.

This commit fixes the issue by readding the old method and maintaining the
CraftBlockChangeDelegate method.  This makes it so that there is a
compatible method for both  CraftBlockChangeDelegate WorldGenerators and
unmodified WorldGenerators.

Additionally, this commit reduces and corrects the diffs in
WorldGenerator, moving the fix for layering violations to
CraftBlockChangeDelegate.

1 files changed, 8 insertions, 2 deletions

diff --git a/src/main/java/org/bukkit/craftbukkit/CraftBlockChangeDelegate.java b/src/main/java/org/bukkit/craftbukkit/CraftBlockChangeDelegate.java
index 470828a2..5946e7d0 100644
--- a/src/main/java/org/bukkit/craftbukkit/CraftBlockChangeDelegate.java
+++ b/src/main/java/org/bukkit/craftbukkit/CraftBlockChangeDelegate.java
@@ -1,6 +1,7 @@
 package org.bukkit.craftbukkit;
 
 import net.minecraft.server.Block;
+import net.minecraft.server.World;
 
 import org.bukkit.BlockChangeDelegate;
 
@@ -19,8 +20,13 @@ public class CraftBlockChangeDelegate {
         return Block.e(this.delegate.getTypeId(x, y, z));
     }
 
-    public void setTypeAndData(int x, int y, int z, Block block, int data, int light) {
-        delegate.setRawTypeIdAndData(x, y, z, Block.b(block), data);
+    public void setTypeAndData(int x, int y, int z, Block block, int data, int updateFlag) {
+        // Layering violation :(
+        if (delegate instanceof World) {
+            ((World) delegate).setTypeAndData(x, y, z, block, data, 2);
+        } else {
+            delegate.setRawTypeIdAndData(x, y, z, Block.b(block), data);
+        }
     }
 
     public boolean isEmpty(int x, int y, int z) {