Draco18s

*Opens his IDE* *Copy-pastes the code* *Looks at the Javadoc attached to the function* *Finds the answer* Use MinecraftForge#EVENT_BUS instead, they're the same thing.

You need to create the storage in the readFromNBT method, once you know what the capacity and transfer speeds are.

NPEs are easy to solve if you have a stack trace as it tells you exactly what line it occurred on. However, I do not have that information.

No. getRegistryName() does not need the substring(5) bullshit that using getUnlocalizedName() does.

Why you would ever have to pass that through your proxy, I don't know. You already know which side it's on as packets should only go one way: it's sent from the server to the client, the packet handler therefor only exists on the client and it's safe to use Minecraft.getMinecraft() Likewise, going the other way.

Use the constructor that takes 2 floats.

Tile Entities, when reconstructed from the save file on disk, are created using a zero-argument constructor (which yours does not have). This means that it is skipping the storage = new MEnergyStorage(capacity, transferSpeed); line, thus storage is null.

>> No code posted > No help available

You need to synchronize the capability from the server to the client yourself.

I'm not 100% sure. Item stacks can't be null. Use ItemStack.EMPTY

The block only needs to be updated if the TE data effects rendering (my block changes model states based on TE information, such as inventory).

1) You may want to override getUpdateTag as well. 2) You need to call these whenever the server should send updates to the client.

You don't need that empty string between GRAY and ITALIC. Also, I18n.format() is a thing. I don't know why you're using your own localize function.

If you're doing log-y things, you might be interested in extending BlockLogsBase (IIRC).

Why?

Oh no. A custom class. The horror! https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/ores/entities/capabilities/MillableItemsHandler.java

Yep, you're doing it wrong. The TextFormatting enum is a thing. https://github.com/Draco18s/ReasonableRealism/blob/1.12.1/src/main/java/com/draco18s/flowers/item/ItemOreFlower1.java#L55

Show your code. You're probably doing it wrong.

Doing that would be Bad.

Custom ItemHandler classes.

One of the following is true - The itemstack returned by getPickBlock is incorrect - The item model for the itemstack returned by getPickBlock has not been registered

Unfixable. You shouldn't have to, and it's because you've set its unlocalized name wrong. See how it says "item.item"?

That's because the first one uses the same stack, the second one uses a new stack.

I'm not sure, its not something I've tried to do.