Minecraft Forums

Be aware that calling any methods from the GUI is client-side only. If you want to run anything on the server (which you usually do), you need to send a packet to the server with information about where the TileEntity is and which button was pressed.

new ItemStack(ModItems.chisel, OreDictionary.WILDCARD_VALUE, -1)

You have the arguments mixed up. The constructor of ItemStack is new ItemStack(Item item, int stackSize, int metadata)

Change it to have 1 (not -1) as stack size and WILDCARD_VALUE as metadata.

Quote from M_Dragon4»

Where do I want to create the new ItemStack of my item? Also, where do I do damage to the item?

Quote from M_Dragon4»

I'm sorry, but I don't follow. Do I need to add @Override hasContainerItem(ItemStack itemStack){ return true; } ?

@Override // this is just for your compiler, but it helps you
public boolean hasContainerItem(ItemStack stack)
{
        return true;
}

Most of the Minecraft blocks make a public static final AxisAlignedBB for each set of bounds so they aren't making a new one every time it is called.

Other than that, TechMage66 is correct.

HarvestDropsEvent contains a List<ItemStack> of all the items that will drop from that block. All you have to do is modify that list based on whatever conditions. To replace drops, you always have to clear the list -- to add "bonus" drops, clearing is not necessary. I have an example here where breaking gravel with my custom tool has a 50% chance to drop flint (as opposed to the normal 10% chance).

I got a lot of crashes if I forgot to check that getHarvester() is not null -- this code is called by explosions, too, where there is no harvester.

Jabelar has a good explanation of what events are, how to use them, etc. here.

setBlockBoundsBasedOnState --> public AxisAlignedBB getBoundingBox(IBlockState state, IBlockAccess source, BlockPos pos)

ChatComponentText --> TextComponentBase and TextComponentTranslation

EnumChatFormatting --> TextFormatting

EnumBlockLayer --> BlockRenderLayer

Many of these you can find just by looking in the Block class or other places where you first found the old one. All of them can be found at the MCP changelog page, which I know Choonster has a link to but I can never find.

FoodStats fs = new FoodStats();

You're going to have to get the actual FoodStats from the player instead of making a new one and hoping it will magically sync to the player's stats.

Side note: Your List intialization can be simplified to this:

aliases = Lists.newArrayList("eat", "feed");

Oh, getMaterial() should be called from the IBlockState, eg, Blocks.log.getDefaultState.getMaterial() or World#getBlockState(pos).getMaterial()

See how to make metadata block in 1.8+

You need to override getStateFromMeta, getMetaFromState, and createBlockState like I do here.

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;
}