[olavjens777]

Fixes #32

Problem

loadAdditional calls updateActiveState(this.level, ...) whenever the tag contains Active, and Active is always saved. When a chunk is loaded, the block entity doesn't have a level yet, so this throws a NullPointerException. Minecraft then skips the block entity, and the machine loses all of its data.

Fix

Only update the active state when there is a server level. That is the case when a machine is placed with block entity data, which is what bc454f5 was for. When a chunk is loaded, the block state stored in the chunk is already correct.

Testing

• New gametest generator.loadWithoutLevel: it loads a generator with BlockEntity.loadStatic, like chunk loading does, and checks that its inventory is still there. It fails without the fix and passes with it.
• ./gradlew build runGametest passes.
• Checked in game with Galacticraft: machines keep their inventory and configuration after reloading the world.

[Roelymole]

Thanks!