Draco18s

Your Item class is broken. You need to call Item::setHasSubtypes(true) https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/ores/item/ItemDustSmall.java#L23

No

It's a registry name, it doesn't point AT anything. new ResourceLocation(SuperTNTMod.MODID + ":large_tnt") ...is fine, its just used as the registry name. Also, resource locations should never include a file type (do your blocks and items? no). The texture referenced by your renderer is null. Namely, this line:

You need one of them, otherwise the game doesn't know that your entity exists and can't serialize it. And one of the two is wrong. So you absolutely positively need the other one.

IMO if it isn't broke, don't fix it. The values are specified in the parent (or somewhere in the parent hierarchy).

Correct. That's why there are mod IDs.

Even for single player you need to register it on the server thread.

Smelting does not use the JSON system

Why are you registering your recipes only on the client?

Do not make core mods.

This will not work. Ever.

You see how you have this line? System.out.println("register Sapphire Block -----------------------------------"); Why don't you do this: System.out.println("register Sapphire Block: " + Sapphire_Block.getRegistryName().toString()); That will tell you what the block's registry name is as it attempts to register it. And if the block itself is null, you'll get a NPE there. This gives you SO MUCH MORE INFORMATION than "I got here!"

This: variant -> normal does not match this: variant -> inventory Also, you don't need this unless you are intentionally wanting a different set of values (why?)

That paste is blank

sonictech:ore[type=aluminium] is a BlockState, not a Block. You set your block's registry name to "ore", then declare variants using an enum, one of which is aluminium. Ergo: sonictech:ore[type=aluminium], but your blockstate file is wrongly constructed. You told the game that the name of the variant was called "type" but your blockstate file uses "variant". Ergo sonictech:ore[type=aluminium] does not equal sonictech:ore[variant=aluminium]

You want to register stuff in the right order? Look no farther than the EasyRegistry! https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/hardlib/EasyRegistry.java https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/hardlib/client/ClientEasyRegistry.java https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/hardlib/HardLib.java#L29-L30 All you gotta do is update the "HardLib" reference to your own classes!

A bad one that should never be used.

No. Bad modder.

Also: BlockPos.getAllInBox(...) is a thing.

Something is very wrong with your code if a 5x5x5 region (125 blocks) takes that long to scan. It should take less than 400,000 nanos (0.4 ms) to scan 20,000 blocks (and do nothing). Show your code.

You can't use null for a registry name.

You never call setRegistryName() for your blocks.

You should not be using setHeldItem at all. This is how I do something similar: https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/farming/block/BlockTanner.java#L121-L148

Caused by: java.io.FileNotFoundException: pm:models/item/ruby.json You may have to clean your cache.