GotoLink

if(!Minecraft.getMinecraft().sndManager.sndSystem.playing("your_music") && Mouse.isButtonDown(1)) { Minecraft.getMinecraft().sndManager.sndSystem.play("your_music"); }

Read again, or learn how to use booleans. This is the null class: public class null { //Really, just void } Do you see a getItem() method here ?

if(weapon!=null) { //do things }

Do you have Java for 64bits and 32bits ?

As for help files, there is a "Read me before posting" sticky. The "Container and GUIs" tutorial is merely a practical example of a simple use case. Learn what you need from it: in Forge, the simplest way to pop a GUI is to use a player instance and implement a IGuiHandler. Buttons are implemented into GuiScreen.

Probably not since nothing in Minecraft would use it. You can use the "look" vector (math is required ) from the player and its position to check blocks in a straight line, and stop when said block isn't air.

Well your method has a "void" as return type. So you can't return null (which is an object). The other issue is you didn't check if "weapon" isn't null before calling getItem().

:'( Copy pasting and not understanding the code... That var1 is completely useless. By the way, since markBlockForUpdate(x,y,z) didn't work, that method won't either. The issue is within the TileEntity. getCurrentLightValInt() probably doesn't return same value on both sides.

bench.gettingEXP() In the Gui, client side...but exp is only updated server side. Send packets.

None of your slots use the player inventory. Check your layoutInventory() in NetControllerContainer.

Huh ? Why do you use EntityPlayerMP instead of EntityPlayer ?

Then make your own timer. @Override public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer entityPlayer) { if(time<limit) { time++; } else { //do things } return stack; }

Then we set the item in use on right click. @Override public ItemStack onItemRightClick(ItemStack stack, World world, EntityPlayer entityPlayer) { entityPlayer.setItemInUse(stack, this.getMaxItemUseDuration(stack)); return stack; } Then @Override public void onPlayerStoppedUsing(ItemStack stack, World world, EntityPlayer entityPlayer, int amountUsed) { //do things... if (!world.isRemote) { world.spawnEntityInWorld(new EntityTear(world, entityPlayer)); } }

entityplayerbalances is null. It isn't set in WelcomeMessages before player login.

Override: /** * How long it takes to use or consume an item */ public int getMaxItemUseDuration(ItemStack par1ItemStack) { return 0;//change that }

Do you call registerSound() yourself ? What is in EventSounds class ?

If you want the light change to happen fast, you'll need to call a block rerendering, with world.markBlockForUpdate(x,y,z), for example.

You can put a setter for the drop in a custom block class file, so you can chain it into your block initialization line.

Yes, initializing your mod before world generation is highly recommended.

p (aka player) can be null too.

Your class which implements IExtendedEntityProperties is on both sides. I would suggest using it.

extend ItemBow and change only what you need to.

Show your item class.

See the originals: public Block func_111022_d(String par1Str) { this.field_111026_f = par1Str; return this; } @SideOnly(Side.CLIENT) protected String func_111023_E() { return this.field_111026_f == null ? "MISSING_ICON_TILE_" + this.blockID + "_" + this.unlocalizedName : this.field_111026_f; } public void registerIcons(IconRegister iconRegister){ this.blockIcon = iconRegister.registerIcon(this.func_111023_E()); } Use func_111022_d, the texture setter.

Drop the overriden method. You only have one texture, use the texture setter. (its name is obfuscated, but pretty obvious anyway) Also, @Instance should get the modid, and items and blocks should be done in PreInit.