All I'm trying to do is add doors with no extra functionality for the wood blocks i've added. I've been at this for hours and Everything I try has problems no one can seem to help with. So I'll go back as far as I think is correct.

Iv'e got an item that extends ItemDoor:

public class ItemVarietyDoor extends ItemDoor{
public ItemVarietyDoor(Block block) {
	super(block);
}
}

I've got a block that extends BlockDoor

public class BlockVarietyDoor extends BlockDoor{

private String Wood;

public BlockVarietyDoor(String wood) {
	super(Material.wood);
	this.setStepSound(Block.soundTypeWood);
	this.setHardness(3.0F);
	this.disableStats();
	Wood = wood;
}

@Override
public ItemStack getPickBlock(MovingObjectPosition target, World world,
		BlockPos pos, EntityPlayer player) {
	return new ItemStack(getItem());
}

@Override
public Item getItemDropped(IBlockState state, Random rand, int fortune)
    {
        return state.getValue(HALF) == BlockDoor.EnumDoorHalf.UPPER ? null : getItem();
    }

public Item getItem(){
	return GameRegistry.findItem(VarietyTrees.MODID, Wood+"_door");
}

}

 

I've got a blockstates.json "apple_door": http://pastebin.com/E8aqEPJz

I've got a block model json "apple_door_bottom":

{
    "parent": "block/door_bottom",
    "textures": {
        "bottom": "varietytrees:blocks/door_apple_lower",
        "top": "varietytrees:blocks/door_apple_upper"
    }
}

I've got a block model json "apple_door_bottom_rh":

{
    "parent": "block/door_bottom_rh",
    "textures": {
        "bottom": "varietytrees:blocks/door_apple_lower",
        "top": "varietytrees:blocks/door_apple_upper"
    }
}

I've got a block model json "apple_door_top":

{
    "parent": "block/door_top",
    "textures": {
        "bottom": "varietytrees:blocks/door_apple_lower",
        "top": "varietytrees:blocks/door_apple_upper"
    }
}

I've got a block model json "apple_door_top_rh":

{
    "parent": "block/door_top_rh",
    "textures": {
        "bottom": "varietytrees:blocks/door_apple_lower",
        "top": "varietytrees:blocks/door_apple_upper"
    }
}

 

What goes in my main class to get this to work properly?

I know broadly what needs to go there, I need to register things and renderers, but every time I try, it's wrong

These classes aren't complete enough to function.  Your door classes don't have any getState method overrides, which means that it will use the vanilla door variants (not what you want).

That part I actually did have working fine before. But maybe its part of the reason other parts didn't, so I overrode all of the state functions too.

What else?

Have you...

- Registered your block?

- Registered your item?

- Registered renderers?

AppleDoor = new BlockVarietyDoor("apple");
        GameRegistry.registerBlock(AppleDoor, "apple_door");
        AppleDoorItem = new ItemVarietyDoor(AppleDoor);
        GameRegistry.registerItem(AppleDoorItem, "apple_door");
        
        RenderItem renderItem = Minecraft.getMinecraft().getRenderItem();    
        renderItem.getItemModelMesher().register(Item.getItemFromBlock(AppleDoor),0, new ModelResourceLocation(MODID + ":" + "apple_door", "inventory"));

 

Crashes with:

The name varietytrees:apple_door has been registered twice, for net.minecraft.item.ItemBlock@46916d68 and panda.varietytrees.trees.ItemVarietyDoor@79b07ccd.

 

At the line

GameRegistry.registerItem(AppleDoorItem, "apple_door");

 

Even though i am not using an itemblock

Hi

 

registerBlock also registers an ItemBlock automatically, corresponding to the block.

    public static Block registerBlock(Block block, String name)
    {
        return registerBlock(block, ItemBlock.class, name);
    }

 

 

If you want to use your own ItemBLock, use this method instead

    /**
     * Register a block with the world, with the specified item class and block name
     *
     * @param block     The block to register
     * @param itemclass The item type to register with it : null registers a block without associated item.
     * @param name      The mod-unique name to register it as, will get prefixed by your modid.
     */
    public static Block registerBlock(Block block, Class<? extends ItemBlock> itemclass, String name)
    {
        return registerBlock(block, itemclass, name, new Object[] {});
    }

 

-TGG

 

 

Hello!

So I need to make my own ItemDoor class that extends ItemBlock then? I cannot use one that extends minecraft ItemDoor? Because Itemdoor does not extend ItemBlock..

Hi

 

In that case, try using registerBlock with null in place of itemClass, and then register your item separately, like in the code you posted.

 

-TGG

That worked! What an elegant solution.

 

The only thing now is my Item is rendering as the missing texture cube. I double checked my json and it looks correct to me, so I'm not sure there.

Console output is:  Model definition for location varietytrees:apple_door#inventory not found

Hi

 

Do you have a json for your item in

assets/yourmodid/models/item?

 

You might find this example project useful to get the structure of the resources correct

https://github.com/TheGreyGhost/MinecraftByExample/tree/1-8final

 

The mbe01 and mbe10 are probably the closest to what you're doing.

 

-TGG

Yes. And its' file name is the same as what I registered the item renderer with

Console output is:  Model definition for location varietytrees:apple_door#inventory not found

I think that means that you forgot to set the custom model resource location during client-proxy preinit. There are many threads around here with examples.