Return the valid max value for piston direction. Fixes BUKKIT-2191

By returning the following value (7) we remove the need to special
case pistons in any way (other than the original purpose of this
check, which is to ensure pistons have valid data)

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

@@ -217,7 +217,7 @@ public class BlockPiston extends Block {
     }
 
     public static int e(int i) {
-        if ((i & 7) >= Facing.OPPOSITE_FACING.length) return 0; // CraftBukkit - check for AIOOB on piston data
+        if ((i & 7) >= Facing.OPPOSITE_FACING.length) return 7; // CraftBukkit - check for AIOOB on piston data
         return i & 7;
     }
 

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

@@ -429,8 +429,8 @@ public class Chunk {
 
                 if (l != 0) {
                     if (!this.world.isStatic) {
-                        // CraftBukkit start - Don't extend piston until data is set, don't "place" if we're processing the event
-                        if (!this.world.suppressPhysics && (!(Block.byId[l] instanceof BlockPiston) || i2 != 0)) {
+                        // CraftBukkit start - Don't "place" if we're processing the event
+                        if (!this.world.suppressPhysics) {
                             Block.byId[l].onPlace(this.world, j2, j, k2);
                         }
                         // CraftBukkit end