Mekanism

Mekanism

111M Downloads

TileComponentChunkLoader#write assumes that tick() has already been called

Aaron1011 opened this issue ยท 1 comments

commented

In TileComponentChunkloader#write, prevCoord.write is called. Normally, tick() is always called before write, ensuring that prevCoord has been properly initialized.

However, SpongeForge creates a Forge BlockSnapshot in order to properly fire an event. Since BlockSnapshot calls TileEntity#writeToNBT, prevCoord ends up being used before being initialized, leading to a NullPointerException.

It would be a great help to Sponge if you could either add a field initializer for prevCoord, or ensure that it's not null before using it in TileComponentChunkloader#write. Thanks!

commented

Done!