Minecraft Forums

Override onBlockActivated in your custom block to check for your structure, and if it is valid, open a gui.

ModelLoader.setCustomModelResourceLocation takes the same arguments, must be called in preInit.

That's a problem. Both client AND server need to know what your blocks are.

Move the call to init() to your common proxy

Minecraft.getMinecraft().thePlayer.capabilities.isCreativeMode

This will crash the server. Minecraft.class is client-only. If you want to alert nearby players that the entity spawned, use World#getEntitiesWithinAABB(EntityPlayer.class, entity.boundingBox.expand(RANGE, RANGE, RANGE) and iterate through that list of players.

I have this old method you can use:

public static int getNonAirNonLiquidFromAbove(World world, int x, int z)
{
	int y = 255;
	boolean foundGround = false;
	while(!foundGround && y-- >= 0)
	{
		Block blockAt = world.getBlock(x,y,z);
		// "ground" for our purposes is anything but air, liquid, leaves
		foundGround = blockAt != Blocks.air && !blockAt.getMaterial().isLiquid() && !(blockAt instanceof BlockLeaves);
	}

	return y;
}

If you want to get the first air block from below, just switch a few things around:

public static int getFirstAirFromBelow(World world, int x, int z)
{
	int y = 0;
	boolean foundAir = false;
	while(!foundAir && y++ < 255)
	{
		Block blockAt = world.getBlock(x,y,z);
		foundAir = blockAt == Blocks.air;
	}

	return y;
}

Feel free to use these and modify them for your purposes. If you do use them, make sure to check that y is indeed between 0 and 255 before doing anything.

Well... you could have one call the other. I don't think it would cause any looping or anything terrible.

public EnumActionResult onItemUse(ItemStack itemStackIn, EntityPlayer playerIn, World worldIn, BlockPos pos, EnumHand hand, EnumFacing facing, float hitX, float hitY, float hitZ) 
{
	this.onItemRightClick(itemStackIn, worldIn, playerIn, hand);
	return super.onItemUse(itemStackIn, playerIn, worldIn, pos, hand, facing, hitX, hitY, hitZ);
}

The Item class has a method for getting the player's look vector. Call this from onItemRightClick before returning an ActionResult.

RayTraceResult rtr = this.rayTrace(worldIn, player, true);

Then check that it hit a block like this:

if (rtr == null)
{
    return ActionResult.newActionResult(EnumActionResult.FAIL, itemStackIn);
}
else if(rtr.typeOfHit == RayTraceResult.Type.BLOCK)
{
    BlockPos hitPos = rtr.getBlockPos();
    // other stuff
}

I do not know if this works flawlessly for blocks that do not have a selection box -- specifically air, fire, etc. In your case, detecting clicking on fire is very important. You may want to set hitPos to rtr.getBlockPos().offset(rtr.sideHit, 1) and see how that works.

Quote from EchoReaper»

How much effort do you estimate it would take to port a mod the size of Mo' Creatures from 1.8 to 1.10?

A few things:

- Items should be registered in preInit. I didn't know it even worked to try to register them in init

- You should be using ModelLoader.setCustomModelResourceLocation instead of Minecraft.getMinecraft().getRenderItem()...

- - the ModelLoader method should be called from preInit as well. It takes the same arguments.

- You should not be using LanguageRegistry

- - use the .lang file instead, adding an entry to translate the unlocalized name. See here for instructions.

Actually, looking at your writeToNBT, it doesn't look like you're writing the items tag to the main NBTTagCompound.

That would explain why it's not entering the for loop -- NBTTagCompound#getTagList returns an empty list if it does not find the tag compound.

public void writeToNBT(NBTTagCompound tagCompound){
            super.writeToNBT(tagCompound);
            NBTTagList items = new NBTTagList();
            for (int i = 0; i < getSizeInventory(); i++){
                ItemStack stack = this.getStackInSlot(i);
                if (stack != null){
                    NBTTagCompound writeitem = new NBTTagCompound();
                    writeitem.setByte("Slot", (byte) i);
                    stack.writeToNBT(writeitem);
                    items.appendTag(writeitem);
                }
            }
            // right here  you should call tagCompound.setTag("Items", items); unless it's called something else
            tagCompound.setInteger("BurnTime", this.furnaceBurnTime);
            tagCompound.setInteger("CookTime", this.furnaceBurnTime);
        }

Quote from sky_01»

My impression was that for each entity, it fires a
RenderLivingEvent.Pre, then a RenderLivingEvent.Post, then moves on to
the next Entity. What's going on?

Well yes, that is what happen, and what you see in your log.
I guess you didn't realize rendering is really fast. Many many times a second.
And when an entity doesn't render, it doesn't reach RenderLivingEvent.Pre nor RenderLivingEvent.Post events once.
You should only register client events through your client proxy.
Also, since only one entity is rendered at a time you didn't need the list.
Finally, you could simply do this in your custom entity renderer. That will do the same, without requiring the events.

The effect you wanted could be done roughly like this:

//In you entity class
private boolean isRendered;//Server-tick accuracy
private int timeSinceLastRender;

public void onUpdate(){
//Some other code here
if(timeSinceLastRender > 0)
setRendered(false);
timeSinceLastRender++;
}

//Call setRendered(true) in your rendering method/event
public void setRendered(boolean render){
if(render)
timeSinceLastRender = 0;
isRendered = render;
}