Progress on BlockDimWall and its respective ItemBlock.

src/main/java/com/zixiken/dimdoors/DimDoors.java

@@ -5,6 +5,7 @@ import java.util.List;
 
 import com.zixiken.dimdoors.items.*;
 import com.zixiken.dimdoors.network.DimDoorsNetwork;
+import com.zixiken.dimdoors.render.BlockRenderManager;
 import com.zixiken.dimdoors.schematic.BlockRotator;
 import com.zixiken.dimdoors.blocks.TransientDoor;
 import com.zixiken.dimdoors.commands.CommandListDungeons;
@@ -226,6 +227,8 @@ public class DimDoors {
         proxy.registerSidedHooks();
 
         DimDoorsNetwork.init();
+
+        BlockRenderManager.addModelVariants();
 	}
 
 	@Mod.EventHandler
@@ -267,6 +270,8 @@ public class DimDoors {
 		// Register loot chests
 		DDLoot.registerInfo(properties);
         MinecraftForge.EVENT_BUS.register(new ConnectionHandler());
+
+        BlockRenderManager.registerBlockRenderers();
 	}
 
 	@Mod.EventHandler

src/main/java/com/zixiken/dimdoors/blocks/BlockDimWall.java

@@ -7,21 +7,28 @@ import com.zixiken.dimdoors.DimDoors;
 import net.minecraft.block.Block;
 import net.minecraft.block.BlockContainer;
 import net.minecraft.block.material.Material;
+import net.minecraft.block.properties.PropertyInteger;
+import net.minecraft.block.state.BlockState;
+import net.minecraft.block.state.IBlockState;
 import net.minecraft.creativetab.CreativeTabs;
 import net.minecraft.entity.Entity;
 import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.item.Item;
 import net.minecraft.item.ItemBlock;
 import net.minecraft.item.ItemStack;
+import net.minecraft.util.BlockPos;
+import net.minecraft.util.MovingObjectPosition;
 import net.minecraft.world.World;
 import com.zixiken.dimdoors.client.PrivatePocketRender;
+import net.minecraftforge.fml.relauncher.Side;
+import net.minecraftforge.fml.relauncher.SideOnly;
 
 public class BlockDimWall extends Block {
     public static final String ID = "blockDimWall";
+    public static final PropertyInteger TYPE = PropertyInteger.create("type", 0, 2);
 
 	private static final float SUPER_HIGH_HARDNESS = 10000000000000F;
 	private static final float SUPER_EXPLOSION_RESISTANCE = 18000000F;
-	private IIcon[] blockIcon = new IIcon[3];
 	
 	public BlockDimWall() {
 		super(Material.iron);
@@ -29,32 +36,32 @@ public class BlockDimWall extends Block {
 		setLightLevel(1.0F);
         setHardness(0.1F);
         setUnlocalizedName(ID);
+        setDefaultState(blockState.getBaseState().withProperty(TYPE, 0));
+	}
+
+	@Override
+	public IBlockState getStateFromMeta(int meta) {
+        if(meta >= 0 && meta <= 2) return getDefaultState().withProperty(TYPE, meta);
+        else return getDefaultState();
+    }
+
+	@Override
+	public int getMetaFromState(IBlockState state) {return state.getValue(TYPE);}
+
+	@Override
+	protected BlockState createBlockState() {return new BlockState(this, TYPE);}
+
+	@Override
+	public float getBlockHardness(World world, int x, int y, int z) {
+		if (world.getBlockMetadata(x, y, z) != 1) return this.blockHardness;
+		else return SUPER_HIGH_HARDNESS;
 	}
 	
 	@Override
-	public float getBlockHardness(World world, int x, int y, int z)
-	{
+    public float getExplosionResistance(Entity entity, World world, int x, int y, int z, double explosionX, double explosionY, double explosionZ) {
 		if (world.getBlockMetadata(x, y, z) != 1)
-		{
-			return this.blockHardness;
-		}
-		else
-		{
-			return SUPER_HIGH_HARDNESS;
-		}
-	}
-	
-	@Override
-    public float getExplosionResistance(Entity entity, World world, int x, int y, int z, double explosionX, double explosionY, double explosionZ)
-    {
-		if (world.getBlockMetadata(x, y, z) != 1)
-		{
 			return super.getExplosionResistance(entity, world, x, y, z, explosionX, explosionY, explosionZ);
-		}
-		else
-		{
-			return SUPER_EXPLOSION_RESISTANCE;
-		}
+		else return SUPER_EXPLOSION_RESISTANCE;
     }
 	
 	public int getRenderType()
@@ -63,57 +70,30 @@ public class BlockDimWall extends Block {
     }
 	
 	@Override
-	public void registerBlockIcons(IIconRegister par1IconRegister)
-    {
-        this.blockIcon[0] = par1IconRegister.registerIcon(DimDoors.modid + ":" + this.getUnlocalizedName());
-        this.blockIcon[1] = par1IconRegister.registerIcon(DimDoors.modid + ":" + this.getUnlocalizedName() + "Perm");
-        this.blockIcon[2] = par1IconRegister.registerIcon(DimDoors.modid + ":" + this.getUnlocalizedName() + "Personal");
-    }
-	
-	@SideOnly(Side.CLIENT)
-	@Override
-	public IIcon getIcon(int par1, int par2)
-	{
-		switch(par2)
-		{
-			case 0:
-				return blockIcon[0];
-			case 1:
-				return blockIcon[1];
-			case 2:
-				return blockIcon[2];
-			default:
-			return blockIcon[0];
-		}
-	}
-	
-	@Override
-	public int damageDropped(int metadata) 
-	{
+	public int damageDropped(IBlockState state) {
+        int metadata = state.getValue(TYPE);
 		//Return 0 to avoid dropping Ancient Fabric even if the player somehow manages to break it
 		return metadata == 1 ? 0 : metadata;
 	}
 	
 	@Override
-	@SuppressWarnings({ "rawtypes", "unchecked" })
 	@SideOnly(Side.CLIENT)
-	public void getSubBlocks(Item unknown, CreativeTabs tab, List subItems)
-	{
-		for (int ix = 0; ix < 3; ix++) 
-		{
-			subItems.add(new ItemStack(this, 1, ix));
-		}
+	public void getSubBlocks(Item itemIn, CreativeTabs tab, List<ItemStack> subItems) {
+		for (int ix = 0; ix < 3; ix++)
+			subItems.add(new ItemStack(itemIn, 1, ix));
 	}
-	
-    @Override
-	public void onBlockDestroyedByPlayer(World par1World, int par2, int par3, int par4, int par5) {}
     
     @Override
-	protected boolean canSilkHarvest()
+	public boolean canSilkHarvest(World world, BlockPos pos, IBlockState state, EntityPlayer player)
     {
         return true;
     }
-    
+
+    @Override
+    public ItemStack getPickBlock(MovingObjectPosition target, World world, BlockPos pos, EntityPlayer player) {
+        return new ItemStack(Item.getItemFromBlock(this), 1, getMetaFromState(world.getBlockState(pos)));
+    }
+
     @Override
 	public int quantityDropped(Random par1Random)
     {
@@ -124,8 +104,7 @@ public class BlockDimWall extends Block {
      * replaces the block clicked with the held block, instead of placing the block on top of it. Shift click to disable. 
      */
     @Override
-	public boolean onBlockActivated(World world, int x, int y, int z, EntityPlayer entityPlayer, int par6, float par7, float par8, float par9)
-    {
+	public boolean onBlockActivated(World world, int x, int y, int z, EntityPlayer entityPlayer, int par6, float par7, float par8, float par9) {
     	//Check if the metadata value is 0 -- we don't want the user to replace Ancient Fabric
         if (entityPlayer.getCurrentEquippedItem() != null && world.getBlockMetadata(x, y, z) != 1)
         {

src/main/java/com/zixiken/dimdoors/items/ItemBlockDimWall.java

@@ -2,24 +2,17 @@ package com.zixiken.dimdoors.items;
 
 import com.zixiken.dimdoors.DimDoors;
 import net.minecraft.block.Block;
-import net.minecraft.client.renderer.texture.IIconRegister;
 import net.minecraft.item.ItemBlock;
 import net.minecraft.item.ItemStack;
 
-public class ItemBlockDimWall extends ItemBlock
-{
-	private final static String[] subNames = {"tile.blockDimWall", "tile.blockAncientWall" , "tile.blockAlteredWall"};
+public class ItemBlockDimWall extends ItemBlock {
+	private final static String[] subNames = {"", "Perm" , "Personal"};
 	
-    public ItemBlockDimWall(Block block)
-    {
-    	  super(block);
-          this.setCreativeTab(DimDoors.dimDoorsCreativeTab);
-          setHasSubtypes(true);
-    }
-    @Override
-	public void registerIcons(IIconRegister par1IconRegister)
-    {
-        this.itemIcon = par1IconRegister.registerIcon(DimDoors.modid + ":" + this.getUnlocalizedName().replace("tile.", ""));
+    public ItemBlockDimWall(Block block) {
+        super(block);
+        setCreativeTab(DimDoors.dimDoorsCreativeTab);
+        setMaxDamage(0);
+        setHasSubtypes(true);
     }
     
     @Override
@@ -28,8 +21,7 @@ public class ItemBlockDimWall extends ItemBlock
 		return damageValue;
 	}
 	
-    public String getUnlocalizedName(ItemStack par1ItemStack)
-    {
-        return subNames[this.getDamage(par1ItemStack)];
+    public String getUnlocalizedName(ItemStack stack) {
+        return super.getUnlocalizedName() + subNames[this.getDamage(stack)];
     }  
 }

src/main/java/com/zixiken/dimdoors/render/BlockRenderManager.java

@@ -0,0 +1,38 @@
+package com.zixiken.dimdoors.render;
+
+import com.zixiken.dimdoors.DimDoors;
+import net.minecraft.block.Block;
+import net.minecraft.client.Minecraft;
+import net.minecraft.client.resources.model.ModelBakery;
+import net.minecraft.client.resources.model.ModelResourceLocation;
+import net.minecraft.item.Item;
+import net.minecraft.util.ResourceLocation;
+
+public class BlockRenderManager {
+    private static final String ID = DimDoors.MODID;
+
+    public static void registerBlockRenderers() {
+        register(DimDoors.blockDimWall);
+        register(DimDoors.blockDimWall, 1, "Perm");
+        register(DimDoors.blockDimWall, 2, "Personal");
+    }
+
+    public static void addModelVariants() {
+        ModelBakery.registerItemVariants(Item.getItemFromBlock(DimDoors.blockDimWall),
+                new ResourceLocation(ID + ":blockDimWall"),
+                new ResourceLocation(ID + ":blockDimWallPerm"),
+                new ResourceLocation(ID + ":blockDimWallPersonal"));
+    }
+
+    private static void register(Block block) {
+        Minecraft.getMinecraft().getRenderItem().getItemModelMesher()
+                .register(Item.getItemFromBlock(block), 0,
+                new ModelResourceLocation(ID + ':' + block.getUnlocalizedName().substring(5)));
+    }
+
+    private static void register(Block block, int meta, String name) {
+        Minecraft.getMinecraft().getRenderItem().getItemModelMesher()
+                .register(Item.getItemFromBlock(block), meta, new ModelResourceLocation(ID + ':' +
+                block.getUnlocalizedName().substring(5) + name, "inventory"));
+    }
+}

src/main/resources/assets/dimdoors/blockstates/blockDimWall.json

@@ -0,0 +1,7 @@
+{
+  "variants": {
+    "type=0": { "model": "dimdoors:blockDimWall" },
+    "type=1": { "model": "dimdoors:blockDimWallPerm" },
+    "type=2": { "model": "dimdoors:blockDimWallPersonal" }
+  }
+}

src/main/resources/assets/dimdoors/models/block/blockDimWall.json

@@ -0,0 +1,4 @@
+{
+  "parent": "block/cube_all",
+  "textures": { "all": "dimdoors:blocks/blockDimWall" }
+}

src/main/resources/assets/dimdoors/models/block/blockDimWallPerm.json

@@ -0,0 +1,4 @@
+{
+  "parent": "block/cube_all",
+  "textures": { "all": "dimdoors:blocks/blockDimWallPerm" }
+}

src/main/resources/assets/dimdoors/models/block/blockDimWallPersonal.json

@@ -0,0 +1,4 @@
+{
+  "parent": "block/cube_all",
+  "textures": { "all": "dimdoors:blocks/blockDimWallPersonal" }
+}

src/main/resources/assets/dimdoors/models/item/blockDimWall.json

@@ -0,0 +1,10 @@
+{
+  "parent": "dimdoors:block/blockDimWall",
+  "display": {
+    "thirdperson": {
+      "rotation": [ 10, -45, 170 ],
+      "translation": [ 0, 1.5, -2.75 ],
+      "scale": [ 0.375, 0.375, 0.375 ]
+    }
+  }
+}

src/main/resources/assets/dimdoors/models/item/blockDimWallPerm.json

@@ -0,0 +1,10 @@
+{
+  "parent": "dimdoors:block/blockDimWallPerm",
+  "display": {
+    "thirdperson": {
+      "rotation": [ 10, -45, 170 ],
+      "translation": [ 0, 1.5, -2.75 ],
+      "scale": [ 0.375, 0.375, 0.375 ]
+    }
+  }
+}

src/main/resources/assets/dimdoors/models/item/blockDimWallPersonal.json

@@ -0,0 +1,10 @@
+{
+  "parent": "dimdoors:block/blockDimWallPersonal",
+  "display": {
+    "thirdperson": {
+      "rotation": [ 10, -45, 170 ],
+      "translation": [ 0, 1.5, -2.75 ],
+      "scale": [ 0.375, 0.375, 0.375 ]
+    }
+  }
+}