Forge direction (#1895)

* ForgeDirection

Also refactor the clusterfuck that was `getCoordinateScan`

Co-authored by: Jason Mitchell <mitchej@gmail.com>

* Fix rendering of Frame Boxes

Frame boxes needed their own implementation of getTexture with int connexion mask,
which is returning an error texture for the MetaTileEntity, because pipes (FrameBox
**is** a pipe) do use this method to return different textures based on connexion
status.

---------

Co-authored-by: Léa Gris <lea.gris@noiraude.net>

19 files changed, 107 insertions, 97 deletions

diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Cover_Tool.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Cover_Tool.java
index d0f5582549..305993c0d0 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Cover_Tool.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Cover_Tool.java
@@ -31,23 +31,23 @@ public class Behaviour_Cover_Tool extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if (aWorld.isRemote) {
             return false;
         }
-        NBTTagCompound tNBT = aStack.getTagCompound();
-        TileEntity tTileEntity = aWorld.getTileEntity(aX, aY, aZ);
-        boolean isCopyMode = aPlayer.isSneaking();
+        final NBTTagCompound tNBT = aStack.getTagCompound();
+        final TileEntity tTileEntity = aWorld.getTileEntity(aX, aY, aZ);
+        final boolean isCopyMode = aPlayer.isSneaking();
         initDataFromNBT(tNBT);
         if (((aPlayer instanceof EntityPlayerMP)) && (aItem.canUse(aStack, 100.0D))) {
             if (isCopyMode) {
                 ArrayList<String> tList = new ArrayList<>();
-                doCopy(tTileEntity, aWorld, aX, aY, aZ, aSide, hitX, hitY, hitZ, tList);
+                doCopy(tTileEntity, aWorld, aX, aY, aZ, side, hitX, hitY, hitZ, tList);
                 aItem.discharge(aStack, 100.0D, Integer.MAX_VALUE, true, false, false);
                 writeListToNBT(tList, tNBT, aPlayer);
                 saveDataToNBT(tNBT);
             } else {
-                doPaste(tTileEntity, aSide, hitX, hitY, hitZ, aPlayer);
+                doPaste(tTileEntity, side, hitX, hitY, hitZ, aPlayer);
                 aItem.discharge(aStack, 25.0D, Integer.MAX_VALUE, true, false, false);
             }
         }
@@ -84,8 +84,8 @@ public class Behaviour_Cover_Tool extends Behaviour_None {
     }
 
     @SuppressWarnings({ "unchecked", "rawtypes" })
-    private void doCopy(TileEntity aTileEntity, World aWorld, int aX, int aY, int aZ, int aSide, float hitX, float hitY,
-        float hitZ, List aList) {
+    private void doCopy(TileEntity aTileEntity, World aWorld, int aX, int aY, int aZ, ForgeDirection side, float hitX,
+        float hitY, float hitZ, List aList) {
         aList.add(
             "----- X: " + EnumChatFormatting.AQUA
                 + GT_Utility.formatNumbers(aX)
@@ -104,20 +104,17 @@ public class Behaviour_Cover_Tool extends Behaviour_None {
                 + EnumChatFormatting.RESET
                 + " -----");
         if (aTileEntity instanceof ICoverable tCoverable) {
-            int tSide = tCoverable.getCoverItemAtSide((byte) aSide) != null ? aSide
-                : tCoverable.getCoverItemAtSide(GT_Utility.determineWrenchingSide((byte) aSide, hitX, hitY, hitZ))
-                    != null ? GT_Utility.determineWrenchingSide((byte) aSide, hitX, hitY, hitZ) : -1;
-            if (tSide != -1) {
-                mStoredData = tCoverable.getComplexCoverDataAtSide((byte) tSide);
-                mCoverType = tCoverable.getCoverIDAtSide((byte) tSide);
-                aList.add(
-                    "Block Side: " + EnumChatFormatting.AQUA
-                        + ForgeDirection.getOrientation(tSide)
-                            .name()
-                        + EnumChatFormatting.RESET);
+            final ForgeDirection tSide = tCoverable.getCoverItemAtSide(side) != null ? side
+                : tCoverable.getCoverItemAtSide(GT_Utility.determineWrenchingSide(side, hitX, hitY, hitZ)) != null
+                    ? GT_Utility.determineWrenchingSide(side, hitX, hitY, hitZ)
+                    : ForgeDirection.UNKNOWN;
+            if (tSide != ForgeDirection.UNKNOWN) {
+                mStoredData = tCoverable.getComplexCoverDataAtSide(tSide);
+                mCoverType = tCoverable.getCoverIDAtSide(tSide);
+                aList.add("Block Side: " + EnumChatFormatting.AQUA + tSide.name() + EnumChatFormatting.RESET);
                 aList.add(
                     "Cover Type: " + EnumChatFormatting.GREEN
-                        + tCoverable.getCoverItemAtSide((byte) tSide)
+                        + tCoverable.getCoverItemAtSide(tSide)
                             .getDisplayName()
                         + EnumChatFormatting.RESET);
             } else {
@@ -132,19 +129,21 @@ public class Behaviour_Cover_Tool extends Behaviour_None {
         }
     }
 
-    private void doPaste(TileEntity aTileEntity, int aSide, float hitX, float hitY, float hitZ, EntityPlayer aPlayer) {
+    private void doPaste(TileEntity aTileEntity, ForgeDirection side, float hitX, float hitY, float hitZ,
+        EntityPlayer aPlayer) {
         if (aTileEntity instanceof ICoverable tCoverable) {
             if (mCoverType == 0 || mStoredData == null) {
                 GT_Utility.sendChatToPlayer(aPlayer, "Please Copy a Valid Cover First.");
                 return;
             }
-            int tSide = tCoverable.getCoverItemAtSide((byte) aSide) != null ? aSide
-                : tCoverable.getCoverItemAtSide(GT_Utility.determineWrenchingSide((byte) aSide, hitX, hitY, hitZ))
-                    != null ? GT_Utility.determineWrenchingSide((byte) aSide, hitX, hitY, hitZ) : -1;
-            if (tSide != -1) {
-                int tCoverID = tCoverable.getCoverIDAtSide((byte) tSide);
+            final ForgeDirection tSide = tCoverable.getCoverItemAtSide(side) != null ? side
+                : tCoverable.getCoverItemAtSide(GT_Utility.determineWrenchingSide(side, hitX, hitY, hitZ)) != null
+                    ? GT_Utility.determineWrenchingSide(side, hitX, hitY, hitZ)
+                    : ForgeDirection.UNKNOWN;
+            if (tSide != ForgeDirection.UNKNOWN) {
+                int tCoverID = tCoverable.getCoverIDAtSide(tSide);
                 if (tCoverID == mCoverType) {
-                    tCoverable.setCoverDataAtSide((byte) tSide, mStoredData);
+                    tCoverable.setCoverDataAtSide(tSide, mStoredData);
                     GT_Utility.sendChatToPlayer(aPlayer, "Cover Data Pasted.");
                 } else {
                     GT_Utility.sendChatToPlayer(aPlayer, "Not Matched Cover.");
@@ -158,8 +157,8 @@ public class Behaviour_Cover_Tool extends Behaviour_None {
     @Override
     public List<String> getAdditionalToolTips(GT_MetaBase_Item aItem, List<String> aList, ItemStack aStack) {
         try {
-            NBTTagCompound tNBT = aStack.getTagCompound();
-            int tSize = tNBT.getInteger("dataLinesCount");
+            final NBTTagCompound tNBT = aStack.getTagCompound();
+            final int tSize = tNBT.getInteger("dataLinesCount");
             if (tSize < 1) throw new Exception();
             aList.add(EnumChatFormatting.BLUE + "Stored Cover Data:");
             for (int i = 0; i < tSize; i++) {
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Crowbar.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Crowbar.java
index 76bc0615c1..97d3da4ca3 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Crowbar.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Crowbar.java
@@ -7,6 +7,7 @@ import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.init.Blocks;
 import net.minecraft.item.ItemStack;
 import net.minecraft.world.World;
+import net.minecraftforge.common.util.ForgeDirection;
 
 import gregtech.api.enums.SoundResource;
 import gregtech.api.items.GT_MetaBase_Item;
@@ -25,7 +26,7 @@ public class Behaviour_Crowbar extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if (aWorld.isRemote) {
             return false;
         }
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Hoe.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Hoe.java
index 0d8e26ac66..167eccdf49 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Hoe.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Hoe.java
@@ -28,8 +28,8 @@ public class Behaviour_Hoe extends Behaviour_None {
 
     @Override
     public boolean onItemUse(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
-        if (!aPlayer.canPlayerEdit(aX, aY, aZ, aSide, aStack)) {
+        int aY, int aZ, int ordinalSide, float hitX, float hitY, float hitZ) {
+        if (!aPlayer.canPlayerEdit(aX, aY, aZ, ordinalSide, aStack)) {
             return false;
         }
         UseHoeEvent event = new UseHoeEvent(aPlayer, aStack, aWorld, aX, aY, aZ);
@@ -43,7 +43,7 @@ public class Behaviour_Hoe extends Behaviour_None {
             return true;
         }
         Block aBlock = aWorld.getBlock(aX, aY, aZ);
-        if ((aSide != 0) && (GT_Utility.isBlockAir(aWorld, aX, aY + 1, aZ))
+        if ((ordinalSide != 0) && (GT_Utility.isBlockAir(aWorld, aX, aY + 1, aZ))
             && ((aBlock == Blocks.grass) || (aBlock == Blocks.dirt))) {
             new WorldSpawnedEventBuilder.SoundEventBuilder()
                 .setVolume((Blocks.farmland.stepSound.getVolume() + 1.0F) / 2.0F)
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Lighter.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Lighter.java
index d348f3eed4..a4c26e2254 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Lighter.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Lighter.java
@@ -73,23 +73,23 @@ public class Behaviour_Lighter extends Behaviour_None {
 
     @Override
     public boolean onItemUse(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, int ordinalSide, float hitX, float hitY, float hitZ) {
         return false;
     }
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if ((aWorld.isRemote) || (aStack.stackSize != 1)) {
             return false;
         }
         boolean rOutput = false;
 
-        ForgeDirection tDirection = ForgeDirection.getOrientation(aSide);
-        aX += tDirection.offsetX;
-        aY += tDirection.offsetY;
-        aZ += tDirection.offsetZ;
-        if ((!GT_Utility.isBlockAir(aWorld, aX, aY, aZ)) || (!aPlayer.canPlayerEdit(aX, aY, aZ, aSide, aStack))) {
+        aX += side.offsetX;
+        aY += side.offsetY;
+        aZ += side.offsetZ;
+        if ((!GT_Utility.isBlockAir(aWorld, aX, aY, aZ))
+            || (!aPlayer.canPlayerEdit(aX, aY, aZ, side.ordinal(), aStack))) {
             return false;
         }
         prepare(aStack);
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_None.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_None.java
index 6928222af1..4ce7f60da0 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_None.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_None.java
@@ -13,6 +13,7 @@ import net.minecraft.entity.projectile.EntityArrow;
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.EnumFacing;
 import net.minecraft.world.World;
+import net.minecraftforge.common.util.ForgeDirection;
 
 import gregtech.api.enums.SubTag;
 import gregtech.api.interfaces.IItemBehaviour;
@@ -28,13 +29,13 @@ public class Behaviour_None implements IItemBehaviour<GT_MetaBase_Item> {
 
     @Override
     public boolean onItemUse(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, int ordinalSide, float hitX, float hitY, float hitZ) {
         return false;
     }
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         return false;
     }
 
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Essentia.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Essentia.java
index 16c1f1dec1..a3c46ea991 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Essentia.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Essentia.java
@@ -27,7 +27,7 @@ public class Behaviour_Plunger_Essentia extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if (aWorld.isRemote) {
             return false;
         }
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Fluid.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Fluid.java
index 95e895ba91..c62cb5c4e5 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Fluid.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Fluid.java
@@ -30,7 +30,7 @@ public class Behaviour_Plunger_Fluid extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if (aWorld.isRemote) {
             return false;
         }
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Item.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Item.java
index 41df9d6e95..a5fa868a00 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Item.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Plunger_Item.java
@@ -8,6 +8,7 @@ import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.item.ItemStack;
 import net.minecraft.tileentity.TileEntity;
 import net.minecraft.world.World;
+import net.minecraftforge.common.util.ForgeDirection;
 
 import gregtech.api.enums.SoundResource;
 import gregtech.api.interfaces.metatileentity.IMetaTileEntity;
@@ -30,13 +31,13 @@ public class Behaviour_Plunger_Item extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if (aWorld.isRemote) {
             return false;
         }
         TileEntity aTileEntity = aWorld.getTileEntity(aX, aY, aZ);
-        if ((aTileEntity instanceof IGregTechTileEntity)) {
-            IMetaTileEntity tMetaTileEntity = ((IGregTechTileEntity) aTileEntity).getMetaTileEntity();
+        if (aTileEntity instanceof IGregTechTileEntity gtTE) {
+            IMetaTileEntity tMetaTileEntity = gtTE.getMetaTileEntity();
             if ((tMetaTileEntity instanceof IMetaTileEntityItemPipe)) {
                 for (IMetaTileEntityItemPipe tTileEntity : GT_Utility
                     .sortMapByValuesAcending(
@@ -48,13 +49,13 @@ public class Behaviour_Plunger_Item extends Behaviour_None {
                         if (tTileEntity.isValidSlot(i)) {
                             if ((tTileEntity.getStackInSlot(i) != null) && ((aPlayer.capabilities.isCreativeMode)
                                 || (((GT_MetaGenerated_Tool) aItem).doDamage(aStack, this.mCosts)))) {
-                                ItemStack tStack = tTileEntity.decrStackSize(i, 64);
+                                final ItemStack tStack = tTileEntity.decrStackSize(i, 64);
                                 if (tStack != null) {
-                                    EntityItem tEntity = new EntityItem(
+                                    final EntityItem tEntity = new EntityItem(
                                         aWorld,
-                                        ((IGregTechTileEntity) aTileEntity).getOffsetX((byte) aSide, 1) + 0.5D,
-                                        ((IGregTechTileEntity) aTileEntity).getOffsetY((byte) aSide, 1) + 0.5D,
-                                        ((IGregTechTileEntity) aTileEntity).getOffsetZ((byte) aSide, 1) + 0.5D,
+                                        gtTE.getOffsetX(side, 1) + 0.5D,
+                                        gtTE.getOffsetY(side, 1) + 0.5D,
+                                        gtTE.getOffsetZ(side, 1) + 0.5D,
                                         tStack);
                                     tEntity.motionX = 0.0D;
                                     tEntity.motionY = 0.0D;
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Prospecting.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Prospecting.java
index 43a35cff05..e64722fb87 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Prospecting.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Prospecting.java
@@ -41,7 +41,7 @@ public class Behaviour_Prospecting extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if (aWorld.isRemote) {
             return false;
         }
@@ -80,9 +80,9 @@ public class Behaviour_Prospecting extends Behaviour_None {
                 int tX = aX, tY = aY, tZ = aZ;
                 Block tBlock;
                 for (int i = 0, j = 6 + tQuality; i < j; i++) {
-                    tX -= ForgeDirection.getOrientation(aSide).offsetX;
-                    tY -= ForgeDirection.getOrientation(aSide).offsetY;
-                    tZ -= ForgeDirection.getOrientation(aSide).offsetZ;
+                    tX -= side.offsetX;
+                    tY -= side.offsetY;
+                    tZ -= side.offsetZ;
 
                     tBlock = aWorld.getBlock(tX, tY, tZ);
                     if (tBlock == Blocks.lava || tBlock == Blocks.flowing_lava) {
@@ -109,16 +109,16 @@ public class Behaviour_Prospecting extends Behaviour_None {
                     }
                 }
 
-                Random tRandom = new XSTR(aX ^ aY ^ aZ ^ aSide);
+                final Random tRandom = new XSTR(aX ^ aY ^ aZ ^ side.ordinal());
                 for (int i = 0, j = 9 + 2 * tQuality; i < j; i++) {
                     tX = aX - 4 - tQuality + tRandom.nextInt(j);
                     tY = aY - 4 - tQuality + tRandom.nextInt(j);
                     tZ = aZ - 4 - tQuality + tRandom.nextInt(j);
                     tBlock = aWorld.getBlock(tX, tY, tZ);
                     if (tBlock instanceof GT_Block_Ores_Abstract) {
-                        TileEntity tTileEntity = aWorld.getTileEntity(tX, tY, tZ);
+                        final TileEntity tTileEntity = aWorld.getTileEntity(tX, tY, tZ);
                         if (tTileEntity instanceof GT_TileEntity_Ores) {
-                            Materials tMaterial = GregTech_API.sGeneratedMaterials[((GT_TileEntity_Ores) tTileEntity).mMetaData
+                            final Materials tMaterial = GregTech_API.sGeneratedMaterials[((GT_TileEntity_Ores) tTileEntity).mMetaData
                                 % 1000];
                             if (tMaterial != null && tMaterial != Materials._NULL) {
                                 GT_Utility.sendChatToPlayer(
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Scanner.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Scanner.java
index c041e02881..22b708e384 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Scanner.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Scanner.java
@@ -9,6 +9,7 @@ import net.minecraft.item.ItemStack;
 import net.minecraft.nbt.NBTTagCompound;
 import net.minecraft.util.EnumChatFormatting;
 import net.minecraft.world.World;
+import net.minecraftforge.common.util.ForgeDirection;
 
 import gregtech.api.enums.SoundResource;
 import gregtech.api.interfaces.IItemBehaviour;
@@ -24,15 +25,15 @@ public class Behaviour_Scanner extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
-        NBTTagCompound tNBT = aStack.getTagCompound();
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
+        final NBTTagCompound tNBT = aStack.getTagCompound();
         if (((aPlayer instanceof EntityPlayerMP)) && (aItem.canUse(aStack, 20000.0D))) {
-            ArrayList<String> tList = new ArrayList<>();
+            final ArrayList<String> tList = new ArrayList<>();
             if (aItem.use(
                 aStack,
-                GT_Utility.getCoordinateScan(tList, aPlayer, aWorld, 1, aX, aY, aZ, aSide, hitX, hitY, hitZ),
+                GT_Utility.getCoordinateScan(tList, aPlayer, aWorld, 1, aX, aY, aZ, side, hitX, hitY, hitZ),
                 aPlayer)) {
-                int tList_sS = tList.size();
+                final int tList_sS = tList.size();
                 tNBT.setInteger("dataLinesCount", tList_sS);
                 for (int i = 0; i < tList_sS; i++) {
                     tNBT.setString("dataLines" + i, tList.get(i));
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Screwdriver.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Screwdriver.java
index 3e15849e3f..d940226b84 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Screwdriver.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Screwdriver.java
@@ -5,6 +5,7 @@ import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.init.Blocks;
 import net.minecraft.item.ItemStack;
 import net.minecraft.world.World;
+import net.minecraftforge.common.util.ForgeDirection;
 
 import gregtech.api.enums.SoundResource;
 import gregtech.api.items.GT_MetaBase_Item;
@@ -23,7 +24,7 @@ public class Behaviour_Screwdriver extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if (aWorld.isRemote) {
             return false;
         }
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Sense.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Sense.java
index 8adcc8a59d..b2ab0031ad 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Sense.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Sense.java
@@ -6,6 +6,7 @@ import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.item.ItemStack;
 import net.minecraft.tileentity.TileEntity;
 import net.minecraft.world.World;
+import net.minecraftforge.common.util.ForgeDirection;
 
 import gregtech.api.items.GT_MetaBase_Item;
 import gregtech.api.items.GT_MetaGenerated_Tool;
@@ -24,7 +25,7 @@ public class Behaviour_Sense extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if (aWorld.isRemote) {
             return false;
         }
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_SensorKit.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_SensorKit.java
index 7362621df9..b12159d058 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_SensorKit.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_SensorKit.java
@@ -9,6 +9,7 @@ import net.minecraft.item.ItemStack;
 import net.minecraft.nbt.NBTTagCompound;
 import net.minecraft.tileentity.TileEntity;
 import net.minecraft.world.World;
+import net.minecraftforge.common.util.ForgeDirection;
 
 import gregtech.api.enums.ItemList;
 import gregtech.api.interfaces.tileentity.IGregTechDeviceInformation;
@@ -24,7 +25,7 @@ public class Behaviour_SensorKit extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if ((aPlayer instanceof EntityPlayerMP)) {
             TileEntity tTileEntity = aWorld.getTileEntity(aX, aY, aZ);
             if (((tTileEntity instanceof IInventory)) && (!((IInventory) tTileEntity).isUseableByPlayer(aPlayer))) {
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_SoftHammer.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_SoftHammer.java
index 7f529282dd..159a0c4b27 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_SoftHammer.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_SoftHammer.java
@@ -7,6 +7,7 @@ import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.init.Blocks;
 import net.minecraft.item.ItemStack;
 import net.minecraft.world.World;
+import net.minecraftforge.common.util.ForgeDirection;
 
 import gregtech.api.enums.SoundResource;
 import gregtech.api.items.GT_MetaBase_Item;
@@ -26,7 +27,7 @@ public class Behaviour_SoftHammer extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if (aWorld.isRemote) {
             return false;
         }
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Sonictron.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Sonictron.java
index 02413bf143..dbab764507 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Sonictron.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Sonictron.java
@@ -6,6 +6,7 @@ import net.minecraft.item.ItemStack;
 import net.minecraft.nbt.NBTTagCompound;
 import net.minecraft.nbt.NBTTagList;
 import net.minecraft.world.World;
+import net.minecraftforge.common.util.ForgeDirection;
 
 import gregtech.api.enums.GT_Values;
 import gregtech.api.interfaces.IItemBehaviour;
@@ -99,7 +100,7 @@ public class Behaviour_Sonictron extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         setCurrentIndex(aStack, -1);
         return false;
     }
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Spray_Color.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Spray_Color.java
index f9621a8da4..cab8e9a7a0 100644
--- a/src/main/java/gregtech/common/items/behaviors/Behaviour_Spray_Color.java
+++ b/src/main/java/gregtech/common/items/behaviors/Behaviour_Spray_Color.java
@@ -65,12 +65,12 @@ public class Behaviour_Spray_Color extends Behaviour_None {
 
     @Override
     public boolean onItemUseFirst(GT_MetaBase_Item aItem, ItemStack aStack, EntityPlayer aPlayer, World aWorld, int aX,
-        int aY, int aZ, int aSide, float hitX, float hitY, float hitZ) {
+        int aY, int aZ, ForgeDirection side, float hitX, float hitY, float hitZ) {
         if ((aWorld.isRemote) || (aStack.stackSize != 1)) {
             return false;
         }
         boolean rOutput = false;
-        if (!aPlayer.canPlayerEdit(aX, aY, aZ, aSide, aStack)) {
+        if (!aPlayer.canPlayerEdit(aX, aY, aZ, side.ordinal(), aStack)) {
             return false;
         }
         NBTTagCompound tNBT = aStack.getTagCompound();
@@ -97,7 +97,7 @@ public class Behaviour_Spray_Color extends Behaviour_None {
         } else {
             lookSide = look.zCoord > 0 ? ForgeDirection.SOUTH : ForgeDirection.NORTH;
         }
-        while ((GT_Utility.areStacksEqual(aStack, this.mUsed, true)) && (colorize(aWorld, aX, aY, aZ, aSide))) {
+        while ((GT_Utility.areStacksEqual(aStack, this.mUsed, true)) && (colorize(aWorld, aX, aY, aZ, side))) {
             GT_Utility.sendSoundToPlayers(aWorld, SoundResource.IC2_TOOLS_PAINTER, 1.0F, 1.0F, aX, aY, aZ);
             if (!aPlayer.capabilities.isCreativeMode) {
                 tUses -= 1L;
@@ -136,8 +136,8 @@ public class Behaviour_Spray_Color extends Behaviour_None {
         return rOutput;
     }
 
-    private boolean colorize(World aWorld, int aX, int aY, int aZ, int aSide) {
-        Block aBlock = aWorld.getBlock(aX, aY, aZ);
+    private boolean colorize(World aWorld, int aX, int aY, int aZ, ForgeDirection side) {
+        final Block aBlock = aWorld.getBlock(aX, aY, aZ);
         if ((aBlock != Blocks.air)
             && ((this.mAllowedVanillaBlocks.contains(aBlock)) || ((aBlock instanceof BlockColored)))) {
             if (aBlock == Blocks.hardened_clay) {
@@ -158,7 +158,7 @@ public class Behaviour_Spray_Color extends Behaviour_None {
             aWorld.setBlockMetadataWithNotify(aX, aY, aZ, (~this.mColor) & 0xF, 3);
             return true;
         }
-        return aBlock.recolourBlock(aWorld, aX, aY, aZ, ForgeDirection.getOrientation(aSide), (~this.mColor) & 0xF);
+        return aBlock.recolourBlock(aWorld, aX, aY, aZ, side, (~this.mColor) & 0xF);
     }
 
     @Override
diff --git a/src/main/java/gregtech/common/items/behaviors/Behaviour_Switch_Metadata.java b/src/main/java/gregtech/common/items/behaviors/Behaviour_Switch_Metadata.java
index 20bc59a171..03cf29b9dd 100644
--- a/