Removed isStackValidForSlot from TileBuildCraft and moved it to subclasses. Fixes #878

common/buildcraft/builders/TileArchitect.java

@@ -242,6 +242,12 @@ public class TileArchitect extends TileBuildCraft implements IInventory {
 		return 1;
 	}
 	
+	@Override
+	public boolean isStackValidForSlot(int i, ItemStack itemstack) {
+		// TODO Auto-generated method stub
+		return false;
+	}
+
 	@Override
 	public boolean isUseableByPlayer(EntityPlayer entityplayer) {
 		return worldObj.getBlockTileEntity(xCoord, yCoord, zCoord) == this;

common/buildcraft/builders/TileBlueprintLibrary.java

@@ -228,6 +228,12 @@ public class TileBlueprintLibrary extends TileBuildCraft implements IInventory {
 		return 64;
 	}
 	
+	@Override
+	public boolean isStackValidForSlot(int i, ItemStack itemstack) {
+		// TODO Auto-generated method stub
+		return false;
+	}
+
 	@Override
 	public boolean isUseableByPlayer(EntityPlayer entityplayer) {
 		return worldObj.getBlockTileEntity(xCoord, yCoord, zCoord) == this;

common/buildcraft/builders/TileBuilder.java

@@ -443,6 +443,12 @@ public class TileBuilder extends TileBuildCraft implements IBuilderInventory, IP
 		return 64;
 	}
 	
+	@Override
+	public boolean isStackValidForSlot(int i, ItemStack itemstack) {
+		// TODO Auto-generated method stub
+		return false;
+	}
+
 	@Override
 	public boolean isUseableByPlayer(EntityPlayer entityplayer) {
 		return worldObj.getBlockTileEntity(xCoord, yCoord, zCoord) == this;

common/buildcraft/builders/TileFiller.java

@@ -235,6 +235,11 @@ public class TileFiller extends TileBuildCraft implements ISidedInventory, IPowe
 		return "Filler";
 	}
 	
+	@Override
+	public boolean isStackValidForSlot(int i, ItemStack itemstack) {
+		return true;
+	}
+
 	@Override
 	public void readFromNBT(NBTTagCompound nbttagcompound) {
 		super.readFromNBT(nbttagcompound);

common/buildcraft/core/TileBuildCraft.java

@@ -12,7 +12,6 @@ package buildcraft.core;
 import java.util.HashMap;
 import java.util.Map;
 
-import net.minecraft.item.ItemStack;
 import net.minecraft.network.packet.Packet;
 import net.minecraft.tileentity.TileEntity;
 import buildcraft.api.power.IPowerReceptor;
@@ -120,9 +119,4 @@ public abstract class TileBuildCraft extends TileEntity implements ISynchronized
         return false;
     }
 
-    public boolean isStackValidForSlot(int i, ItemStack itemstack)
-    {
-        // TODO Auto-generated method stub
-        return false;
-    }
 }

common/buildcraft/energy/Engine.java

@@ -203,6 +203,10 @@ public abstract class Engine {
 	public void setInventorySlotContents(int i, ItemStack itemstack) {
 	}
 	
+	public boolean isStackValidForSlot(int i, ItemStack itemstack){
+		return false;
+	}
+
 	public abstract ILiquidTank getTank(ForgeDirection direction, LiquidStack type);
 
 }

common/buildcraft/energy/TileEngine.java

@@ -293,6 +293,14 @@ public class TileEngine extends TileBuildCraft implements IPowerReceptor, IInven
 		}
 	}
 	
+	@Override
+	public boolean isStackValidForSlot(int i, ItemStack itemstack) {
+		if (engine != null){
+			return engine.isStackValidForSlot(i, itemstack);
+		}
+		return false;
+	}
+
 	@Override
 	public String getInvName() {
 		return "Engine";

common/buildcraft/factory/TileQuarry.java

@@ -728,6 +728,11 @@ public class TileQuarry extends TileMachine implements IMachine, IPowerReceptor,
 		return 0;
 	}
 	
+	@Override
+	public boolean isStackValidForSlot(int i, ItemStack itemstack) {
+		return false;
+	}
+
 	@Override
 	public boolean isUseableByPlayer(EntityPlayer entityplayer) {
 		return false;

common/buildcraft/factory/TileRefinery.java

@@ -93,6 +93,11 @@ public class TileRefinery extends TileMachine implements ITankContainer, IPowerR
 		return 0;
 	}
 	
+	@Override
+	public boolean isStackValidForSlot(int i, ItemStack itemstack) {
+		return false;
+	}
+
 	@Override
 	public boolean isUseableByPlayer(EntityPlayer entityplayer) {
 		return worldObj.getBlockTileEntity(xCoord, yCoord, zCoord) == this;