Draco18s

That seems pretty self explanatory to me. The only issue is that Block's toString method goes based on its registry entry, which failed, so it returns AIR. You have a block that doesn't have a registry name.

https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/harderores/network/PacketHandler.java#L12 Namely you just say "here's my packet class's constructor, toBytes, fromBytes, etc." You can call those methods whatever you want as long as they have the right signature.

Its not. Its just a comment that your code was unnecessarily verbose. My guess as to which line is line 28 is this one, as its the only one that appears to reference a function in the ItemStack class: You should probably be using attemptDamageItem I think its called. Its the one that takes Unbreaking into account.

Speaking of, where did the high-altitude snowfall logic get moved to?

else is sufficient here, by the way. It is impossible for both of those to be true at the same time or false at the same time.

Note that the more items and blocks you create the more model and blockstate JSON files you'll have to update to reflect the change.

This is a Gui class. You call this class FROM the event handler, you don't MAKE it the event handler.

TheGreyGhost/MinecraftByExample Choonster-Minecraft-Mods/TestMod3 Draco18s/ReasonableRealism (not intended as a "by example" but there are things I do in my mods that I use as reference regularly)

...and you have 64 characters of length to play with.

Its part of the Event parent class.

You'll need a custom recipe for this. For example, I have something similar for crafting: https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/hardlib/api/recipe/RecipeTagOutput.java Furnace (blasting, smelting, cooking) recipes use a similar system, I believe the class to look at is AbstractFurnaceRecipe and its subclasses.

Reflection exists.

Agreed, but I never saw the method there either.

Looks good. Just wanted to make sure there wasn't anything there that might've been a problem too.

Again, that's not what side only does. If you want an event to only be registered on one side, you use the value=Dist.CLIENT parameter in the @EventBusSubscriber annotation. Also, its not called SideOnly any more. Its OnlyIn now. .

Uh... public abstract class MobEntity extends LivingEntity { //... public final GoalSelector goalSelector; and protected void addSpawn(EntityClassification type, Biome.SpawnListEntry spawnListEntry) { //... } Forge build 1.14.4-28.0.45

You haven't set up the state container. https://github.com/Draco18s/ReasonableRealism/blob/1.14.4/src/main/java/com/draco18s/harderores/block/WindvaneBlock.java#L33-L36 Also, show where you create and register your blocks

This does not do what you think it does. Adding this to the Gui class will just cause a class not found exception in another place. Omitting it and registering the gui only from the client proxy, meanwhile, works just fine.

Well, your block is telling the game that it is a block that completely fills its space and is opaque. This is default behavior. As such the game culled all the faces on its neighbors.

Are we sure about that? Because he's never posted the full class. Or even the method signature of where his code is located. And there's NO way he could get an infinite loop unless he was firing a BreakEvent from within a BreakEvent.

You shouldn't be making your item work from inside an event. You don't need an event, you can just override the method in the class....

https://github.com/xWhatsherfacex/Testing/blob/master/java/com/rnh/home/Init/ModItems.java#L26 Problematic Code #14

Yes, but you aren't passing in an event handler class, you're passing in an event.

sorry, had the order wrong.

This doesn't do what you're trying to make it do.