Draco18s

You've gone all bonkers with the file. The first problem is that BlockPos does not exist in 1.7.10, but you're trying to use it. This is what the class should look like.

Stop using unlocalized name and start using setRegistryName. Don't ever use getUnlocalizedName().substring(5) for anything, ever. Stop doing it. That's bad. Really really bad, perpetuated by a shitty tutorial that follows tons of bad practices and should be taken down.

Also, oh god, that if-stack checking whether the item given was equal to an array of stuff. Jesus christ. Cleaned it up

The NBT in my context was the NBT tag attached to an item stack, not the TileEntity's saveToNBT

So you moved all of your item registration into your ClientProxy. Good job. Now what does the server know about your items? Jack shit is what.

You open the vanilla GUI, which checks to see if the player is using a vanilla workbench, and immediately closes the gui if they are not. You need to open your custom GUI using a GuiHandler and player.openGui(...) which includes passing your mod ID.

What you want is a coremod that operates via ASM. ASM is a very deep, dark abyss and you will get minimal help figuring out how to make a coremod. Primarily because: 1) You should not be modifying base classes at all ever, doing so can very easily introduce mod conflicts. 2) ASM requires understanding Java bytecode, you will need to know this before you can start, at all. 3) Providing resources on how to coremod will cause more people to go "yay, I can modify base classes!" when they didn't need to. Given that you are interested in doing this on 1.9, I suggest actually supplying a patch back to Forge as a Pull Request that inserts an event hook where you want it, that way you don't have to modify vanilla classes.

https://github.com/Draco18s/Artifacts/blob/master/main/java/com/draco18s/artifacts/client/PedestalRenderer.java#L57-L106

And then doesn't give a shit what the actual value of the metadata was, because it performed the same check on all of them: meta == i It doesn't matter what value meta was from the world (because it can only be 0 to 15), at some point during that loop that check will be true. Following that, because you are doing return true in the block-check, as soon as it determines that yes, one of these blocks is what we're looking for it never even fucking checks your loop.

Rather than incrementing a counter every tick, store the start time and compare with the current time.

Also, pretty sure that statement will never return false. //somewhere... stack new ItemStack(yourItem); //somewhere else... stack.getItem().getContainerItem(stack)

Or you can compare the player's lastPos values with its pos values and see if they have changed.

http://www.minecraftforge.net/forum/index.php?topic=22764.0

Do note that in order to add AI tasks, you need to wait until after the entity has spawned in the world. EntityLivingBase#tasks is null until then.

...look at TileEntitySign and see how it does it?

You need to look at ClassLoaders. ClassLoaders are objects that load other classes when needed. This means that you can write a custom class loader that fetches a class, examines it, makes changes (if it wants), and then passes it off to the JVM.

Just to nitpick: Damage is metadata >..>

How long is the sound file (in seconds)? Option a: make it shorter. Option b: make it a sound like the jukebox or minecart, which can be canceled.

Hesus Christie. Put the setRegistryName chunk with the same chunk that does the unlocalized name and max stack size. Or better yet, put all of that into the constructors for those classes, the whole "method returns itself" pattern is dumb IMO.

if(block == otherBlock) ..?

Good job not knowing how proxies work.

And some are weirdly related, like seeds. Seeds are absolutely not the item form of a block, yet when used they create a block, when that block is middle-clicked, you get a seed, and when the block is broken, you get the seed (usually!) But you can also get seeds from breaking tall grass (which has its own itemblock!)

.getUnlocalizedName() == this.getUnlocalizedName(); Hesus Christie! You have no idea what you're doing, do you.

You initialize Items before Blocks, but pass a Block (which is null) to your Item's constructor.