Draco18s

What is this? Why are you creating a new one every frame?

7. Cancel the event 8. Reduce the used item stack's size

So close. So very close. How is this supposed to return the same item with 1 more damage? You never pass in the original item stack so how do you know what the damage is? This is pointless. getContainerItem isn't called unless hasContainerItem returns true already. Oh look, the original item stack. Call damageItem() on it and return it (though you should probably clone it first...).

"Broken_Saw" and "Ingot_Saw" are two different items, yes? They're different instances of the class, you can distinguish them. Or are they the same instance, with different damage? You can still distinguish them. In either case, storing the return of the default implementation of hasContainerItem (return false) in the class is never ever going to do what you want.

Absolutely not. You need to override hasContainerItem and getContainerItem in your Saw class.

Milk Buckets use the Container Item functionality. That should work for most recipes.

That's the only way to supply a different unlocalized name for each metadata of the item stack.

Custom ItemBlock classes.

No it doesn't. Yes, Forge is installed. No, Forge is not what crashed. Even so, you should have posted in Support and Bug Reports. Not the section under "Mod Developer Central" You ran out of memory. Allocate more.

Your in the wrong place. This forum is where people who are making mods come for help making mods. That said: Either: You are trying to load a client side mod on the server This mod is badly written and you should report it to the author

I think he's looking for a way to detect the player replacing a block of water with a block of dirt.

Cascading chunk gen happens when you attempt to place blocks into a neighboring chunk that hasn't been created yet. Vanilla avoids this by offsetting the generation by +8 and +8 (as the neighboring positive X and Z chunks are guaranteed to already exist, and won't cause them to get generated)

And this is why I thought there was a 3 character minimum, but I've already had that conversation with Lex.

Capabilities

Don't use AT when you can use Reflection. Reflection is cleaner.

Correct. And this is why I have my registry code done up the way I have. https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/hardlib/client/ClientEasyRegistry.java#L64-L71 I search for all valid states of the block (note that this method is not good for things like leaves which have states that don't appear in the inventory) and register an item model for each.

You might be right. I haven't actually played in a while.

No, it does not. Because item models aren't block models. They are registered separately. You want your item variants to have textures? Register your item variants to have textures. https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/hardlib/client/ClientEasyRegistry.java#L100-L103

this.targetTasks.addTask(4, new EntityInfestedEnderman.AIInfestEndermanAttack(this)); Just because it's a static class doesn't mean you don't call new.

Isn't CTRL the key used to sneak? Check if the player is sneaking.

Redstone materials break instantly, yes. They bypass the hardness check.

You still need to register their models with the ModelLoader

Huh. That's weird. Mine doesn't.

You are not doing this.