Draco18s

ModelLoader is a static class. That means in stead of: Minecraft.getMinecraft().getRenderItem() You put ModelLoader And then because method names are case-sensitive, you need to use: setCustomModelResourceLocation instead of setcustommodelresourcelocation This really isn't that hard.

You're doing everything wrong. 1) you should not pass the event to your GUI. 2) You should create buttons doing initGui. 3) Get button clicks during actionPerformed.

Don't use setItemDamage, instead use damageItem. That allows the item stack to resist the damage (unbreakable) as well as a few other things.

Well, technically machine intervention. The code was never used on the server in vanilla code, so the automated "strip out things not used on the server" process stripped it.

Use the constructor that takes two float arguments.

You can. ByteBufUtils.writeTag(buffer, nbt); You can also send more than one value in a packet...

You will need to replicate the behavior of the raytrace yourself.

You need to create a block state container. https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/ores/block/BlockAxel.java#L60-L63

Your method needs to be static.

*I* do not.

The entire system you're working with in 1.7 has been completely replaced with a new system. Currently supported versions are 1.10.2*, 1.11.2, and 1.12.2 *Officially the supported versions are "current" and "previous" but a lot of the systems are the same between 1.10 and 1.11, so 1.10.2 hasn't quite fallen off the list yet.

Yes

1.7.10 is no longer supported here. Update.

You need to send a list of UUIDs.

You need to use the registry events. Models need to be registered (exactly as you have) but in the ModelRegistryEvent

Oh yeah, making a custom subclass is easy. You just have to override like, two methods. It just so happens that for the most part, there's already ItemMultiTexture (iirc) that already does it.

Minecraft, they're used by wool, planks, etc. They're easy enough to find. Right click -> open type hierarchy. You'll want the one not called "colored" because it inverts the metadata value because Mojang made colors backwards.

This is an event, you do not invoke it yourself. The @SubscribeEvent annotation is what allows the code to be called when the class it is in is correctly registered as an event handler. http://mcforge.readthedocs.io/en/latest/concepts/registries/#registering-things

Wool has subtypes, so does logs and planks. Iron Ore does not.

There are already two subclasses of ItemBlock, one of which should do what you need. The issue is that the ItemBlock class (by default, unless override by a subclass) ignores metadata when placing its block.

You have two Living Hurt events: https://github.com/Insane-96/GalaxiteMod/blob/master/common/net/insane96mcp/galaxite/lib/CustomEventHandler.java#L20-L21 https://github.com/Insane-96/GalaxiteMod/blob/master/common/net/insane96mcp/galaxite/lib/CustomEventHandler.java#L79 I'm going to bet that one of them works and the other doesn't.

Prior to the addition of the registry events, this is the wrong way to do item substitution. You were supposed to use GameRegistry.addSubstitutionAlias which may throw a ExistingSubstitutionException so you had to wrap it in a try...catch block.

This is because the ModelMesher is Mojang's code and must be called at a very specific point during startup, whereas the ModelLoader is Forge supplied so that you (as a modder) can't fuck up.

My comments were less a comment on you than a comment on the people still using ModelMesher. People bitch at me all the time. No joke.