Draco18s

You don't need this. You can get the same effect by calling new ElementBase("cu","copper",0xffb732); instead of new Copper(); A class that does fuckall does not need to exist. There's no class for Sticks or for Iron Ingots. I haven't.

Is this the name of the png file, exactly? I think you have an extra "ashed_" in there, judging by the other texture names.

The player's inventory is public and an array. In order to make the slot empty you have to set it to ItemStack.EMPTY

Server creates the packet by calling new YourPacket(some, params), right? What do you think the client does, it doesn't know what some, params are, but it still needs to create the packet instance so that it can take the buffer it does have and pipe it through fromBytes(buf). And the only way that even makes sense is to call a parameterless constructor.

You haven't registered your tile entity.

Yes.

Blocklight is from blocks (torches, glowstone, etc) while Skylight is light from the sky (aka the sun and moon). Hence the names.

GIMP is also good, though I personally like version 2.6, the UI changes that got introduced after that irk me.

Have you heard of loops?

Guis are client side, but the server has absolute authority on data. You need to send a packet. Also, use Capabilities, not NBT.

ItemStacks cannot be null. They cannot be null because a stack of size zero is empty and should be treated as a non-existent stack. This is why ItemStack.EMPTY exists.

Or use smaller models. If your model is large enough to fill another block space, therte should be a block there anyway, and you can put the other part of the model there.

Ok, so, your first code generates a registry name with two colons in it. Hence "dangerous prefix." Your fixed code is fine, the reason for the missing registry entries is because your old code generated different IDs than your new code. This is fine, either load the world and have the blocks be deleted from it or generate a new world.

Check also Soul Sand.

Override getDrops

Or, you know, its parent class:

GetItemDropped had fuckall to do with falling blocks. That is the item that is dropped when the block is harvested. Redstone ore drops redstone dust, etc, Take a look at BlockFalling instead.

I mean, if you want to do it the hard way, sure.

Yes. You should. Here's the thing: Would you rather deal with integers and floats? Or the zip-compressed string representation thereof?

You're in the wrong place. We don't fix problems with using mods, we fix problems with making mods. also:

Your method isn't static. When using the @EventBusSubscriber annotation, all event methods must be static.

1. Capabilities do not store NBT data. NBT is a serialization format (like txt or json or xml). 2. Your read and write NBT methods do fuck all, what did you expect to happen?

Yeah. That's LoreMaster. I've made comments on his videos about the sort of crap he ends up recommending, whether intentionally or not.

This is not going to work, for two reasons: 1) You're cloning an array, not the item stacks 2) You check for null *after* calling .clone() on it.

This is the purview of IBlockStates and the blockstate json file.