Draco18s

Just because you're done with it does not mean the topic needs to be locked. Look at the rest of this forum, how many locked topics do you see?

And explain what you mean by "doesn't work."

Nothing about that says it is fired during world generation it says it is fired on the TERRAIN_GEN_BUS.

There's an existent AI task for that too.

Thread: You: [no code] lars: post your code jeffry: you need to do X you: "here's my code!" me: did you read this? (Quoting X.)

And sync the data to the client, of course.

There also shouldn't be empty stacks in the drops array.

No, what you need to do is extract the 2 bits of metadata. EnumFacing.getHorizontal(meta&3)

*cough, did you read this, cough*

return this.getDefaultState().withProperty(FACING, EnumFacing.getHorizontal(meta)) This line won't work if meta > 4 (which BARREL and SPIGOT make it be)

Basically, you'll collide from some directions and not others and general wonkiness. The only, the only, direction it is safe to extend a collision box outside the 1x1x1 cube is +Y and even then only an extra half-block.

If your block has a collision bounds greater than 1x1x1 you're in trouble there too: it won't work either. Getting fences to work properly was a problem Mojang took several versions to get right.

It is a little messy, and facings can be mitigated. You can convert a facing (EnumWest) into x and z offsets. "Forward" gets translated to facing.offsetX() + facing.offsetZ() because a facing will only ever have one of those values as non-zero. For right, you use facing.rotate(UP).offsetX() + facing.rotate(UP).offsetZ() (you can store the rotated facings as their own variables as well).

The functionailiy will continue to exist as Minecraft evolves, but it is possible that the method does get removed and placed somewhere else. It will depend on how Mojang continues to work with the IBlockState format.

Basically not really. The method is marked deprecated which means do not call this method. You should still override it to get the desired behavior.

Well, with a well-defined structure I wouldn't use getAllInBox. I'd go "this is the core, so at x+1 I should find BlockY, at x+2 I should find... And do a big query that way.

FastTESR is still slower than IBakedModel, but it's more performant than a regular TESR

ModelLoader.setCustomModelResourceLocation(item, metadata, modelResourceLocation); Hmm...

For this you should use a FastTESR. There is an animation API in JSON, but it's limited.

Why do you need a TESR (fast or otherwise) to render a block? Use the JSON model system.

Maybe you should ask the mod developer. We here don't even know what mod you're using because you conveniently left that out, must less know anything about it.

public/private change wasn't necessary, it was that the local-scope declaration overrode the parent scope declaration.

repair.isItemEqual(new ItemStack(RepairItem)) can be simplified torepair.getItem() == RepairItem Except that RepairItem is always null because you did this: Item RepairItem = repairitem; That variable is local and discarded as soon as the constructor finishes and theprivate Item RepairItem;declared is left undefined.

1.7.10 is no longer supported here.

There are a lot of things I don't know. I didn't start programming on Java so I generally take what I do know and make it work, it isn't until later that someone says, "hey, there's an easier way."