I am trying to place a door from a PlayerInteractEvent, because I want to add some extra conditions on placing a door (I want to require two solid wood blocks where the hinge is going to be). I tried this:

 

Item heldItem = event.getEntityPlayer().getHeldItemMainhand().getItem();

if (heldItem instanceof ItemDoor)
{
	...check various conditions...
	IBlockState iblockstate = Block.getBlockFromItem(heldItem).getDefaultState().withProperty(BlockDoor.FACING, doorFacing).withProperty(BlockDoor.HINGE, isRightHinge ? BlockDoor.EnumHingePosition.RIGHT : BlockDoor.EnumHingePosition.LEFT);
	world.setBlockState(pos, iblockstate.withProperty(BlockDoor.HALF, BlockDoor.EnumDoorHalf.LOWER), 2);
	world.setBlockState(posUp, iblockstate.withProperty(BlockDoor.HALF, BlockDoor.EnumDoorHalf.UPPER), 2);
}

 

but it gives a null reference exception from Block.getBlockFromItem(heldItem) since the held item is not an instance of ItemBlock (it's an ItemDoor). I also tried BlockDoor.getDefaultstate()... but getDefaultState is not a static method, and I don't have an instance of BlockDoor to call it from. I can't make a new instance, because the constructor for BlockDoor is protected. ItemDoor has a block variable, but it's private. I also tried using ItemDoor's built-in placeDoor() method, but its parameters also require a BlockDoor, which I don't have.

 

What do I need to do here? Thanks in advance.

Edited May 28, 20178 yr by Daeruin

22 minutes ago, Daeruin said:

I also tried BlockDoor.getDefaultstate()... but getDefaultState is not a static method, and I don't have an instance of BlockDoor to call it from. I can't make a new instance, because the constructor for BlockDoor is protected.

You DO have an instance of BlockDoor at net.minecraft.init.Blocks, You in fact have 6 of them there. Blocks are singletons.

But that will obviously only work with vanilla doors and only if you hard-code them. If you want your method to work with any door that uses an inplementation of ItemDoor to be placed you would need to use reflection to access the block field of ItemDoor

Blocks.DOOR.getDefaultState()

 

The method isn't static, therefor you need an instance and where do you find that?

 

The Blocks class. Where all the instances are stored.

20 minutes ago, V0idWa1k3r said:

You DO have an instance of BlockDoor at net.minecraft.init.Blocks, You in fact have 6 of them there. Blocks are singletons.

But that will obviously only work with vanilla doors and only if you hard-code them. If you want your method to work with any door that uses an inplementation of ItemDoor to be placed you would need to use reflection to access the block field of ItemDoor

Ah, yes. That makes sense. I've used reflection before and could probably do it again. I'd much prefer a generic solution to hard coding the placement for each type of door. Thanks!