Merge pull request #664 from thatsIch/b-663-disabled-facades

Fixes #663 NPE on disabled Facades

src/main/java/appeng/facade/FacadeContainer.java

@@ -55,7 +55,7 @@ public class FacadeContainer implements IFacadeContainer
 		for (int x = 0; x < this.facades; x++)
 		{
 			if ( this.getFacade( ForgeDirection.getOrientation( x ) ) != null )
-				facadeSides = facadeSides | (1 << x);
+				facadeSides |= ( 1 << x );
 		}
 		out.writeByte( (byte) facadeSides );
 
@@ -79,7 +79,7 @@ public class FacadeContainer implements IFacadeContainer
 
 		boolean changed = false;
 
-		int ids[] = new int[2];
+		int[] ids = new int[2];
 		for (int x = 0; x < this.facades; x++)
 		{
 			ForgeDirection side = ForgeDirection.getOrientation( x );
@@ -205,7 +205,7 @@ public class FacadeContainer implements IFacadeContainer
 	@Override
 	public void rotateLeft()
 	{
-		IFacadePart newFacades[] = new FacadePart[6];
+		IFacadePart[] newFacades = new FacadePart[6];
 
 		newFacades[ForgeDirection.UP.ordinal()] = this.storage.getFacade( ForgeDirection.UP.ordinal() );
 		newFacades[ForgeDirection.DOWN.ordinal()] = this.storage.getFacade( ForgeDirection.DOWN.ordinal() );

src/main/java/appeng/parts/PartPlacement.java

@@ -41,6 +41,7 @@ import cpw.mods.fml.common.eventhandler.SubscribeEvent;
 import cpw.mods.fml.common.gameevent.TickEvent;
 
 import appeng.api.AEApi;
+import appeng.api.definitions.Items;
 import appeng.api.parts.IFacadePart;
 import appeng.api.parts.IPartHost;
 import appeng.api.parts.IPartItem;
@@ -92,11 +93,16 @@ public class PartPlacement
 				if ( te instanceof IPartHost && this.wasCanceled )
 					event.setCanceled( true );
 			}
-			else if ( event.entityPlayer != null )
+			else
 			{
 				ItemStack held = event.entityPlayer.getHeldItem();
-				boolean supportedItem = AEApi.instance().items().itemMemoryCard.sameAsStack( held )
-						|| AEApi.instance().items().itemColorApplicator.sameAsStack( held );
+				final Items items = AEApi.instance().items();
+
+				final boolean sameAsMemoryCard = items.itemMemoryCard != null && items.itemMemoryCard.sameAsStack( held );
+				final boolean sameAsColorApp = items.itemColorApplicator != null && items.itemColorApplicator.sameAsStack( held );
+
+				final boolean supportedItem = sameAsMemoryCard || sameAsColorApp;
+
 				if ( event.entityPlayer.isSneaking() && held != null && supportedItem )
 				{
 					NetworkHandler.instance.sendToServer( new PacketClick( event.x, event.y, event.z, event.face, 0, 0, 0 ) );

src/main/java/appeng/recipes/game/FacadeRecipe.java

@@ -18,29 +18,52 @@
 
 package appeng.recipes.game;
 
+
+import net.minecraft.inventory.IInventory;
 import net.minecraft.inventory.InventoryCrafting;
+import net.minecraft.item.Item;
 import net.minecraft.item.ItemStack;
 import net.minecraft.item.crafting.IRecipe;
 import net.minecraft.world.World;
 
+import com.google.common.base.Optional;
+
 import appeng.api.AEApi;
 import appeng.api.util.AEItemDefinition;
 import appeng.items.parts.ItemFacade;
 
+
 public class FacadeRecipe implements IRecipe
 {
 
-	private final AEItemDefinition anchor = AEApi.instance().parts().partCableAnchor;
-	private final ItemFacade facade = (ItemFacade) AEApi.instance().items().itemFacade.item();
+	private final Optional<AEItemDefinition> maybeAnchor;
+	private final Optional<AEItemDefinition> maybeFacade;
 
-	private ItemStack getOutput(InventoryCrafting inv, boolean createFacade)
+	public FacadeRecipe()
 	{
-		if ( inv.getStackInSlot( 0 ) == null && inv.getStackInSlot( 2 ) == null && inv.getStackInSlot( 6 ) == null && inv.getStackInSlot( 8 ) == null )
+		this.maybeFacade = Optional.fromNullable( AEApi.instance().items().itemFacade );
+		this.maybeAnchor = Optional.fromNullable( AEApi.instance().parts().partCableAnchor );
+	}
+
+	@Override
+	public boolean matches( InventoryCrafting inv, World w )
+	{
+		return this.getOutput( inv, false ) != null;
+	}
+
+	private ItemStack getOutput( IInventory inv, boolean createFacade )
+	{
+		if ( this.maybeAnchor.isPresent() && this.maybeFacade.isPresent() && inv.getStackInSlot( 0 ) == null && inv.getStackInSlot( 2 ) == null && inv.getStackInSlot( 6 ) == null && inv.getStackInSlot( 8 ) == null )
 		{
-			if ( this.anchor.sameAsStack( inv.getStackInSlot( 1 ) ) && this.anchor.sameAsStack( inv.getStackInSlot( 3 ) ) && this.anchor.sameAsStack( inv.getStackInSlot( 5 ) )
-					&& this.anchor.sameAsStack( inv.getStackInSlot( 7 ) ) )
+			final AEItemDefinition anchorDefinition = this.maybeAnchor.get();
+			final AEItemDefinition facadeDefinition = this.maybeFacade.get();
+
+			if ( anchorDefinition.sameAsStack( inv.getStackInSlot( 1 ) ) && anchorDefinition.sameAsStack( inv.getStackInSlot( 3 ) ) && anchorDefinition.sameAsStack( inv.getStackInSlot( 5 ) ) && anchorDefinition.sameAsStack( inv.getStackInSlot( 7 ) ) )
 			{
-				ItemStack facades = this.facade.createFacadeForItem( inv.getStackInSlot( 4 ), !createFacade );
+				final Item itemDefinition = facadeDefinition.item();
+				final ItemFacade facade = (ItemFacade) itemDefinition;
+
+				ItemStack facades = facade.createFacadeForItem( inv.getStackInSlot( 4 ), !createFacade );
 				if ( facades != null && createFacade )
 					facades.stackSize = 4;
 				return facades;
@@ -50,13 +73,7 @@ public class FacadeRecipe implements IRecipe
 	}
 
 	@Override
-	public boolean matches(InventoryCrafting inv, World w)
-	{
-		return this.getOutput( inv, false ) != null;
-	}
-
-	@Override
-	public ItemStack getCraftingResult(InventoryCrafting inv)
+	public ItemStack getCraftingResult( InventoryCrafting inv )
 	{
 		return this.getOutput( inv, true );
 	}
@@ -72,5 +89,4 @@ public class FacadeRecipe implements IRecipe
 	{
 		return null;
 	}
-
 }