1.12 TileEntity not saving either Inventory or custom capabaility storage

[sudwood]

Hello I have been working on a mod for 1.12 and have manged to get the block all set except for the part where it saves. The ItemStackHandler calls an error when reading from NBT and my custom capabilities are not saving at all. The custom capabilities are working because when the coal is consumed the essence list is updated and displayed on the client.

Error:

Spoiler

[21:31:43] [main/INFO] [GradleStart]: Extra: []
[21:31:43] [main/INFO] [GradleStart]: Running with arguments: [--userProperties, {}, --assetsDir, C:/Users/shado/.gradle/caches/minecraft/assets, --assetIndex, 1.12, --accessToken{REDACTED}, --version, 1.12.2, --tweakClass, net.minecraftforge.fml.common.launcher.FMLTweaker, --tweakClass, net.minecraftforge.gradle.tweakers.CoremodTweaker]
[21:31:43] [main/INFO] [LaunchWrapper]: Loading tweak class name net.minecraftforge.fml.common.launcher.FMLTweaker
[21:31:43] [main/INFO] [LaunchWrapper]: Using primary tweak class name net.minecraftforge.fml.common.launcher.FMLTweaker
[21:31:43] [main/INFO] [LaunchWrapper]: Loading tweak class name net.minecraftforge.gradle.tweakers.CoremodTweaker
[21:31:43] [main/INFO] [LaunchWrapper]: Calling tweak class net.minecraftforge.fml.common.launcher.FMLTweaker
[21:31:43] [main/INFO] [FML]: Forge Mod Loader version 14.23.5.2768 for Minecraft 1.12.2 loading
[21:31:43] [main/INFO] [FML]: Java is Java HotSpot(TM) 64-Bit Server VM, version 1.8.0_191, running on Windows 10:amd64:10.0, installed at C:\Program Files\Java\jdk1.8.0_191\jre
[21:31:43] [main/ERROR] [FML]: Apache Maven library folder was not in the format expected. Using default libraries directory.
[21:31:43] [main/ERROR] [FML]: Full: C:\Users\shado\.gradle\caches\modules-2\files-2.1\org.apache.maven\maven-artifact\3.5.3\7dc72b6d6d8a6dced3d294ed54c2cc3515ade9f4\maven-artifact-3.5.3.jar
[21:31:43] [main/ERROR] [FML]: Trimmed: c:/users/shado/.gradle/caches/modules-2/files-2.1/org.apache.maven/maven-artifact/3.5.3/
[21:31:43] [main/INFO] [FML]: Managed to load a deobfuscated Minecraft name- we are in a deobfuscated environment. Skipping runtime deobfuscation
[21:31:43] [main/INFO] [FML]: Detected deobfuscated environment, loading log configs for colored console logs.
2018-11-09 21:31:44,816 main WARN Disabling terminal, you're running in an unsupported environment.
[21:31:44] [main/INFO] [FML]: Ignoring missing certificate for coremod FMLCorePlugin (net.minecraftforge.fml.relauncher.FMLCorePlugin), we are in deobf and it's a forge core plugin
[21:31:44] [main/INFO] [FML]: Ignoring missing certificate for coremod FMLForgePlugin (net.minecraftforge.classloading.FMLForgePlugin), we are in deobf and it's a forge core plugin
[21:31:44] [main/INFO] [FML]: Searching C:\Users\shado\OneDrive\Documents\Modding1.12.2\run\.\mods for mods
[21:31:44] [main/INFO] [LaunchWrapper]: Calling tweak class net.minecraftforge.gradle.tweakers.CoremodTweaker
[21:31:44] [main/INFO] [GradleStart]: Injecting location in coremod net.minecraftforge.fml.relauncher.FMLCorePlugin
[21:31:44] [main/INFO] [GradleStart]: Injecting location in coremod net.minecraftforge.classloading.FMLForgePlugin
[21:31:44] [main/INFO] [LaunchWrapper]: Loading tweak class name net.minecraftforge.fml.common.launcher.FMLInjectionAndSortingTweaker
[21:31:44] [main/INFO] [LaunchWrapper]: Loading tweak class name net.minecraftforge.fml.common.launcher.FMLDeobfTweaker
[21:31:44] [main/INFO] [LaunchWrapper]: Loading tweak class name net.minecraftforge.gradle.tweakers.AccessTransformerTweaker
[21:31:44] [main/INFO] [LaunchWrapper]: Calling tweak class net.minecraftforge.fml.common.launcher.FMLInjectionAndSortingTweaker
[21:31:44] [main/INFO] [LaunchWrapper]: Calling tweak class net.minecraftforge.fml.common.launcher.FMLInjectionAndSortingTweaker
[21:31:44] [main/INFO] [LaunchWrapper]: Calling tweak class net.minecraftforge.fml.relauncher.CoreModManager$FMLPluginWrapper
[21:31:46] [main/ERROR] [FML]: FML appears to be missing any signature data. This is not a good thing
[21:31:46] [main/INFO] [LaunchWrapper]: Calling tweak class net.minecraftforge.fml.relauncher.CoreModManager$FMLPluginWrapper
[21:31:46] [main/INFO] [LaunchWrapper]: Calling tweak class net.minecraftforge.fml.common.launcher.FMLDeobfTweaker
[21:31:47] [main/INFO] [LaunchWrapper]: Calling tweak class net.minecraftforge.gradle.tweakers.AccessTransformerTweaker
[21:31:47] [main/INFO] [LaunchWrapper]: Loading tweak class name net.minecraftforge.fml.common.launcher.TerminalTweaker
[21:31:47] [main/INFO] [LaunchWrapper]: Calling tweak class net.minecraftforge.fml.common.launcher.TerminalTweaker
[21:31:47] [main/INFO] [LaunchWrapper]: Launching wrapped minecraft {net.minecraft.client.main.Main}
[21:31:47] [main/INFO] [minecraft/Minecraft]: Setting user: Player63
[21:31:50] [main/WARN] [minecraft/GameSettings]: Skipping bad option: lastServer:
[21:31:50] [main/INFO] [minecraft/Minecraft]: LWJGL Version: 2.9.4
[21:31:51] [main/INFO] [FML]: -- System Details --
Details:
	Minecraft Version: 1.12.2
	Operating System: Windows 10 (amd64) version 10.0
	Java Version: 1.8.0_191, Oracle Corporation
	Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Oracle Corporation
	Memory: 843984104 bytes (804 MB) / 1037959168 bytes (989 MB) up to 1037959168 bytes (989 MB)
	JVM Flags: 3 total; -Xincgc -Xmx1024M -Xms1024M
	IntCache: cache: 0, tcache: 0, allocated: 0, tallocated: 0
	FML: 
	Loaded coremods (and transformers): 
	GL info: ' Vendor: 'NVIDIA Corporation' Version: '4.6.0 NVIDIA 416.34' Renderer: 'GeForce RTX 2080/PCIe/SSE2'
[21:31:51] [main/INFO] [FML]: MinecraftForge v14.23.5.2768 Initialized
[21:31:51] [main/INFO] [FML]: Starts to replace vanilla recipe ingredients with ore ingredients.
[21:31:51] [main/INFO] [FML]: Replaced 1036 ore ingredients
[21:31:51] [main/INFO] [FML]: Searching C:\Users\shado\OneDrive\Documents\Modding1.12.2\run\.\mods for mods
[21:31:52] [main/INFO] [FML]: Forge Mod Loader has identified 5 mods to load
[21:31:53] [main/INFO] [FML]: Attempting connection with missing mods [minecraft, mcp, FML, forge, cencial] at CLIENT
[21:31:53] [main/INFO] [FML]: Attempting connection with missing mods [minecraft, mcp, FML, forge, cencial] at SERVER
[21:31:53] [main/INFO] [minecraft/SimpleReloadableResourceManager]: Reloading ResourceManager: Default, FMLFileResourcePack:Forge Mod Loader, FMLFileResourcePack:Minecraft Forge, FMLFileResourcePack:Cencial Mod
[21:31:53] [Thread-3/INFO] [FML]: Using sync timing. 200 frames of Display.update took 104700004 nanos
[21:31:53] [main/INFO] [FML]: Processing ObjectHolder annotations
[21:31:53] [main/INFO] [FML]: Found 1168 ObjectHolder annotations
[21:31:53] [main/INFO] [FML]: Identifying ItemStackHolder annotations
[21:31:53] [main/INFO] [FML]: Found 0 ItemStackHolder annotations
[21:31:53] [main/INFO] [FML]: Configured a dormant chunk cache size of 0
[21:31:53] [Forge Version Check/INFO] [forge.VersionCheck]: [forge] Starting version check at http://files.minecraftforge.net/maven/net/minecraftforge/forge/promotions_slim.json
[21:31:53] [main/WARN] [FML]: Potentially Dangerous alternative prefix `minecraft` for name `cencial_basicplinth`, expected `cencial`. This could be a intended override, but in most cases indicates a broken mod.
[21:31:53] [main/INFO] [FML]: OBJLoader: Domain cencial has been added.
[21:31:53] [main/INFO] [FML]: Applying holder lookups
[21:31:53] [main/INFO] [FML]: Holder lookups applied
[21:31:53] [main/INFO] [FML]: Applying holder lookups
[21:31:54] [main/INFO] [FML]: Holder lookups applied
[21:31:54] [main/INFO] [FML]: Applying holder lookups
[21:31:54] [main/INFO] [FML]: Holder lookups applied
[21:31:54] [main/INFO] [FML]: Applying holder lookups
[21:31:54] [main/INFO] [FML]: Holder lookups applied
[21:31:54] [main/INFO] [FML]: Injecting itemstacks
[21:31:54] [main/INFO] [FML]: Itemstack injection complete
[21:31:54] [Forge Version Check/INFO] [forge.VersionCheck]: [forge] Found status: UP_TO_DATE Target: null
[21:31:57] [Sound Library Loader/INFO] [minecraft/SoundManager]: Starting up SoundSystem...
[21:31:57] [Thread-5/INFO] [minecraft/SoundManager]: Initializing LWJGL OpenAL
[21:31:57] [Thread-5/INFO] [minecraft/SoundManager]: (The LWJGL binding of OpenAL.  For more information, see http://www.lwjgl.org)
[21:31:57] [Thread-5/INFO] [minecraft/SoundManager]: OpenAL initialized.
[21:31:58] [Sound Library Loader/INFO] [minecraft/SoundManager]: Sound engine started
[21:32:02] [main/INFO] [FML]: Max texture size: 16384
[21:32:03] [main/INFO] [minecraft/TextureMap]: Created: 512x512 textures-atlas
[21:32:04] [main/INFO] [FML]: Applying holder lookups
[21:32:04] [main/INFO] [FML]: Holder lookups applied
[21:32:04] [main/INFO] [FML]: Injecting itemstacks
[21:32:04] [main/INFO] [FML]: Itemstack injection complete
[21:32:04] [main/INFO] [FML]: Forge Mod Loader has successfully loaded 5 mods
[21:32:04] [main/WARN] [minecraft/GameSettings]: Skipping bad option: lastServer:
[21:32:04] [main/INFO] [mojang/NarratorWindows]: Narrator library for x64 successfully loaded
[21:32:05] [Realms Notification Availability checker #1/INFO] [mojang/RealmsClient]: Could not authorize you against Realms server: Invalid session id
[21:32:07] [Server thread/INFO] [minecraft/IntegratedServer]: Starting integrated minecraft server version 1.12.2
[21:32:07] [Server thread/INFO] [minecraft/IntegratedServer]: Generating keypair
[21:32:07] [Server thread/INFO] [FML]: Injecting existing registry data into this server instance
[21:32:07] [Server thread/INFO] [FML]: Applying holder lookups
[21:32:07] [Server thread/INFO] [FML]: Holder lookups applied
[21:32:07] [Server thread/INFO] [FML]: Loading dimension 0 (New World) (net.minecraft.server.integrated.IntegratedServer@116c1b25)
[21:32:08] [Server thread/INFO] [minecraft/AdvancementList]: Loaded 488 advancements
[21:32:08] [Server thread/INFO] [FML]: Loading dimension -1 (New World) (net.minecraft.server.integrated.IntegratedServer@116c1b25)
[21:32:08] [Server thread/INFO] [FML]: Loading dimension 1 (New World) (net.minecraft.server.integrated.IntegratedServer@116c1b25)
[21:32:08] [Server thread/INFO] [minecraft/MinecraftServer]: Preparing start region for level 0
[21:32:09] [Server thread/INFO] [FML]: Unloading dimension -1
[21:32:09] [Server thread/INFO] [FML]: Unloading dimension 1
[21:32:09] [Server thread/INFO] [minecraft/IntegratedServer]: Changing view distance to 12, from 10
[21:32:11] [Netty Local Client IO #0/INFO] [FML]: Server protocol version 2
[21:32:11] [Netty Server IO #1/INFO] [FML]: Client protocol version 2
[21:32:11] [Netty Server IO #1/INFO] [FML]: Client attempting to join with 5 mods : minecraft@1.12.2,cencial@0.1,FML@8.0.99.99,forge@14.23.5.2768,mcp@9.42
[21:32:11] [Netty Local Client IO #0/INFO] [FML]: [Netty Local Client IO #0] Client side modded connection established
[21:32:11] [Server thread/INFO] [FML]: [Server thread] Server side modded connection established
[21:32:11] [Server thread/INFO] [minecraft/PlayerList]: Player63[local:E:c64daffe] logged in with entity id 120 at (259.9770757001834, 2.5055695295825395, -347.3426296308576)
[21:32:11] [Server thread/INFO] [minecraft/MinecraftServer]: Player63 joined the game
[21:32:12] [Server thread/INFO] [minecraft/IntegratedServer]: Saving and pausing game...
[21:32:12] [Server thread/INFO] [minecraft/MinecraftServer]: Saving chunks for level 'New World'/overworld
[21:32:12] [pool-2-thread-1/WARN] [mojang/YggdrasilMinecraftSessionService]: Couldn't look up profile properties for com.mojang.authlib.GameProfile@25ea3c9[id=ca94d3c2-08bb-3cb6-9c61-61015e7666c6,name=Player63,properties={},legacy=false]
com.mojang.authlib.exceptions.AuthenticationException: The client has sent too many requests within a certain amount of time
	at com.mojang.authlib.yggdrasil.YggdrasilAuthenticationService.makeRequest(YggdrasilAuthenticationService.java:79) ~[YggdrasilAuthenticationService.class:?]
	at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillGameProfile(YggdrasilMinecraftSessionService.java:180) [YggdrasilMinecraftSessionService.class:?]
	at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService$1.load(YggdrasilMinecraftSessionService.java:60) [YggdrasilMinecraftSessionService$1.class:?]
	at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService$1.load(YggdrasilMinecraftSessionService.java:57) [YggdrasilMinecraftSessionService$1.class:?]
	at com.google.common.cache.LocalCache$LoadingValueReference.loadFuture(LocalCache.java:3716) [guava-21.0.jar:?]
	at com.google.common.cache.LocalCache$Segment.loadSync(LocalCache.java:2424) [guava-21.0.jar:?]
	at com.google.common.cache.LocalCache$Segment.lockedGetOrLoad(LocalCache.java:2298) [guava-21.0.jar:?]
	at com.google.common.cache.LocalCache$Segment.get(LocalCache.java:2211) [guava-21.0.jar:?]
	at com.google.common.cache.LocalCache.get(LocalCache.java:4154) [guava-21.0.jar:?]
	at com.google.common.cache.LocalCache.getOrLoad(LocalCache.java:4158) [guava-21.0.jar:?]
	at com.google.common.cache.LocalCache$LocalLoadingCache.get(LocalCache.java:5147) [guava-21.0.jar:?]
	at com.google.common.cache.LocalCache$LocalLoadingCache.getUnchecked(LocalCache.java:5153) [guava-21.0.jar:?]
	at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillProfileProperties(YggdrasilMinecraftSessionService.java:170) [YggdrasilMinecraftSessionService.class:?]
	at net.minecraft.client.Minecraft.getProfileProperties(Minecraft.java:3181) [Minecraft.class:?]
	at net.minecraft.client.resources.SkinManager$3.run(SkinManager.java:138) [SkinManager$3.class:?]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_191]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_191]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [?:1.8.0_191]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [?:1.8.0_191]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_191]
[21:32:34] [main/WARN] [minecraft/SoundManager]: Unable to play empty soundEvent: minecraft:entity.small_slime.jump
[21:32:35] [main/WARN] [minecraft/SoundManager]: Unable to play empty soundEvent: minecraft:entity.small_slime.squish
[21:32:35] [Server thread/INFO] [minecraft/IntegratedServer]: Saving and pausing game...
[21:32:35] [Server thread/INFO] [minecraft/MinecraftServer]: Saving chunks for level 'New World'/overworld
[21:32:37] [Server thread/INFO] [minecraft/IntegratedServer]: Saving and pausing game...
[21:32:37] [Server thread/INFO] [minecraft/MinecraftServer]: Saving chunks for level 'New World'/overworld
[21:32:43] [Server thread/INFO] [minecraft/IntegratedServer]: Changing view distance to 2, from 12
[21:32:58] [Server thread/INFO] [minecraft/MinecraftServer]: [Player63: Teleported Player63 to 0.5, 10.0, 0.5]
[21:32:58] [main/INFO] [minecraft/GuiNewChat]: [CHAT] Teleported Player63 to 0.5, 10.0, 0.5
[21:33:04] [main/INFO] [minecraft/GuiNewChat]: [CHAT] The number you have entered (100000000.50) is too big, it must be at most 30000000
[21:33:11] [Server thread/INFO] [minecraft/MinecraftServer]: [Player63: Teleported Player63 to 100000.5, 4.0, 1000000.5]
[21:33:11] [main/INFO] [minecraft/GuiNewChat]: [CHAT] Teleported Player63 to 100000.5, 4.0, 1000000.5
[21:33:24] [Server thread/INFO] [minecraft/MinecraftServer]: [Player63: Teleported Player63 to 0.5, 10.0, 0.5]
[21:33:24] [main/INFO] [minecraft/GuiNewChat]: [CHAT] Teleported Player63 to 0.5, 10.0, 0.5
[21:33:32] [Server thread/ERROR] [minecraft/TileEntity]: Failed to load data for block entity minecraft:cencial_basicplinth
java.lang.NullPointerException: null
	at net.minecraftforge.items.ItemStackHandler.deserializeNBT(ItemStackHandler.java:196) ~[ItemStackHandler.class:?]
	at com.sudwood.cencial.tileentities.TileEntityBasicPlinth.readFromNBT(TileEntityBasicPlinth.java:45) ~[TileEntityBasicPlinth.class:?]
	at net.minecraft.tileentity.TileEntity.create(TileEntity.java:131) [TileEntity.class:?]
	at net.minecraft.world.chunk.storage.AnvilChunkLoader.loadEntities(AnvilChunkLoader.java:530) [AnvilChunkLoader.class:?]
	at net.minecraftforge.common.chunkio.ChunkIOProvider.syncCallback(ChunkIOProvider.java:101) [ChunkIOProvider.class:?]
	at net.minecraftforge.common.chunkio.ChunkIOExecutor.syncChunkLoad(ChunkIOExecutor.java:94) [ChunkIOExecutor.class:?]
	at net.minecraft.world.gen.ChunkProviderServer.loadChunk(ChunkProviderServer.java:130) [ChunkProviderServer.class:?]
	at net.minecraft.world.gen.ChunkProviderServer.loadChunk(ChunkProviderServer.java:101) [ChunkProviderServer.class:?]
	at net.minecraft.world.gen.ChunkProviderServer.provideChunk(ChunkProviderServer.java:147) [ChunkProviderServer.class:?]
	at net.minecraft.world.World.getChunkFromChunkCoords(World.java:362) [World.class:?]
	at net.minecraft.entity.Entity.setPositionAndRotation(Entity.java:1560) [Entity.class:?]
	at net.minecraft.network.NetHandlerPlayServer.setPlayerLocation(NetHandlerPlayServer.java:651) [NetHandlerPlayServer.class:?]
	at net.minecraft.command.CommandTP.teleportEntityToCoordinates(CommandTP.java:161) [CommandTP.class:?]
	at net.minecraft.command.CommandTP.execute(CommandTP.java:81) [CommandTP.class:?]
	at net.minecraft.command.CommandHandler.tryExecute(CommandHandler.java:126) [CommandHandler.class:?]
	at net.minecraft.command.CommandHandler.executeCommand(CommandHandler.java:98) [CommandHandler.class:?]
	at net.minecraft.network.NetHandlerPlayServer.handleSlashCommand(NetHandlerPlayServer.java:1003) [NetHandlerPlayServer.class:?]
	at net.minecraft.network.NetHandlerPlayServer.processChatMessage(NetHandlerPlayServer.java:979) [NetHandlerPlayServer.class:?]
	at net.minecraft.network.play.client.CPacketChatMessage.processPacket(CPacketChatMessage.java:47) [CPacketChatMessage.class:?]
	at net.minecraft.network.play.client.CPacketChatMessage.processPacket(CPacketChatMessage.java:8) [CPacketChatMessage.class:?]
	at net.minecraft.network.PacketThreadUtil$1.run(PacketThreadUtil.java:21) [PacketThreadUtil$1.class:?]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_191]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_191]
	at net.minecraft.util.Util.runTask(Util.java:53) [Util.class:?]
	at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:798) [MinecraftServer.class:?]
	at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:743) [MinecraftServer.class:?]
	at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:192) [IntegratedServer.class:?]
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:592) [MinecraftServer.class:?]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_191]
[21:33:32] [Server thread/ERROR] [FML]: A TileEntity minecraft:cencial_basicplinth(com.sudwood.cencial.tileentities.TileEntityBasicPlinth) has thrown an exception during loading, its state cannot be restored. Report this to the mod author
java.lang.NullPointerException: null
	at net.minecraftforge.items.ItemStackHandler.deserializeNBT(ItemStackHandler.java:196) ~[ItemStackHandler.class:?]
	at com.sudwood.cencial.tileentities.TileEntityBasicPlinth.readFromNBT(TileEntityBasicPlinth.java:45) ~[TileEntityBasicPlinth.class:?]
	at net.minecraft.tileentity.TileEntity.create(TileEntity.java:131) [TileEntity.class:?]
	at net.minecraft.world.chunk.storage.AnvilChunkLoader.loadEntities(AnvilChunkLoader.java:530) [AnvilChunkLoader.class:?]
	at net.minecraftforge.common.chunkio.ChunkIOProvider.syncCallback(ChunkIOProvider.java:101) [ChunkIOProvider.class:?]
	at net.minecraftforge.common.chunkio.ChunkIOExecutor.syncChunkLoad(ChunkIOExecutor.java:94) [ChunkIOExecutor.class:?]
	at net.minecraft.world.gen.ChunkProviderServer.loadChunk(ChunkProviderServer.java:130) [ChunkProviderServer.class:?]
	at net.minecraft.world.gen.ChunkProviderServer.loadChunk(ChunkProviderServer.java:101) [ChunkProviderServer.class:?]
	at net.minecraft.world.gen.ChunkProviderServer.provideChunk(ChunkProviderServer.java:147) [ChunkProviderServer.class:?]
	at net.minecraft.world.World.getChunkFromChunkCoords(World.java:362) [World.class:?]
	at net.minecraft.entity.Entity.setPositionAndRotation(Entity.java:1560) [Entity.class:?]
	at net.minecraft.network.NetHandlerPlayServer.setPlayerLocation(NetHandlerPlayServer.java:651) [NetHandlerPlayServer.class:?]
	at net.minecraft.command.CommandTP.teleportEntityToCoordinates(CommandTP.java:161) [CommandTP.class:?]
	at net.minecraft.command.CommandTP.execute(CommandTP.java:81) [CommandTP.class:?]
	at net.minecraft.command.CommandHandler.tryExecute(CommandHandler.java:126) [CommandHandler.class:?]
	at net.minecraft.command.CommandHandler.executeCommand(CommandHandler.java:98) [CommandHandler.class:?]
	at net.minecraft.network.NetHandlerPlayServer.handleSlashCommand(NetHandlerPlayServer.java:1003) [NetHandlerPlayServer.class:?]
	at net.minecraft.network.NetHandlerPlayServer.processChatMessage(NetHandlerPlayServer.java:979) [NetHandlerPlayServer.class:?]
	at net.minecraft.network.play.client.CPacketChatMessage.processPacket(CPacketChatMessage.java:47) [CPacketChatMessage.class:?]
	at net.minecraft.network.play.client.CPacketChatMessage.processPacket(CPacketChatMessage.java:8) [CPacketChatMessage.class:?]
	at net.minecraft.network.PacketThreadUtil$1.run(PacketThreadUtil.java:21) [PacketThreadUtil$1.class:?]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_191]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_191]
	at net.minecraft.util.Util.runTask(Util.java:53) [Util.class:?]
	at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:798) [MinecraftServer.class:?]
	at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:743) [MinecraftServer.class:?]
	at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer.java:192) [IntegratedServer.class:?]
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:592) [MinecraftServer.class:?]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_191]
[21:33:32] [Server thread/INFO] [minecraft/MinecraftServer]: [Player63: Teleported Player63 to 100000.5, 4.0, 1000000.5]
[21:33:32] [main/INFO] [minecraft/GuiNewChat]: [CHAT] Teleported Player63 to 100000.5, 4.0, 1000000.5
[21:33:35] [Server thread/INFO] [minecraft/IntegratedServer]: Saving and pausing game...
[21:33:35] [Server thread/INFO] [minecraft/MinecraftServer]: Saving chunks for level 'New World'/overworld
[21:33:36] [Server thread/INFO] [minecraft/MinecraftServer]: Stopping server
[21:33:36] [Server thread/INFO] [minecraft/MinecraftServer]: Saving players
[21:33:36] [Server thread/INFO] [minecraft/NetHandlerPlayServer]: Player63 lost connection: Disconnected
[21:33:36] [Server thread/INFO] [minecraft/MinecraftServer]: Player63 left the game
[21:33:36] [Server thread/INFO] [minecraft/NetHandlerPlayServer]: Stopping singleplayer server as player logged out
[21:33:36] [Server thread/INFO] [minecraft/MinecraftServer]: Saving worlds
[21:33:36] [Server thread/INFO] [minecraft/MinecraftServer]: Saving chunks for level 'New World'/overworld
[21:33:36] [Server thread/INFO] [FML]: Unloading dimension 0
[21:33:36] [Server thread/INFO] [FML]: Applying holder lookups
[21:33:36] [Server thread/INFO] [FML]: Holder lookups applied
[21:33:37] [main/INFO] [minecraft/Minecraft]: Stopping!
[21:33:37] [main/INFO] [minecraft/SoundManager]: SoundSystem shutting down...
[21:33:37] [main/WARN] [minecraft/SoundManager]: Author: Paul Lamb, www.paulscode.com
Java HotSpot(TM) 64-Bit Server VM warning: Using incremental CMS is deprecated and will likely be removed in a future release

 

Here are the code links:

https://github.com/Sudwood/Cencial/blob/master/java/com/sudwood/cencial/tileentities/TileEntityBasicPlinth.java

https://github.com/Sudwood/Cencial/blob/master/java/com/sudwood/cencial/proxy/CommonProxy.java

https://github.com/Sudwood/Cencial/blob/master/java/com/sudwood/cencial/capability/CapabilityHandler.java

https://github.com/Sudwood/Cencial/blob/master/java/com/sudwood/cencial/essence/EssenceHolderProvider.java

https://github.com/Sudwood/Cencial/blob/master/java/com/sudwood/cencial/essence/EssenceHolderStorage.java

https://github.com/Sudwood/Cencial/blob/master/java/com/sudwood/cencial/essence/EssenceHolder.java

 

Thanks for any help!

Edited November 10, 2018 by sudwood
put the links in the wrong spot

[V0idWa1k3r]

Use the debugger, looks like the NBT data being deserialized doesn't have any tag by the inventory key. Did you add the inventory later after adding the TE to your world? 

In order to avoid these kind of errors don't use the raw getTag method, use the getter that corresponds to the type of the object you want to get. In your case NBTTagCompound#getCompoundTag. These getters will never return null.

 

Now for the non-related issues:

Quote

public class CommonProxy

CommonProxy makes no sense. Proxies exist to separate sided-only code. If your code is common it goes anywhere else but your proxy. It makes even less sense in your case since you are using it as the event handler for whatever reason. And handling FML lifecycle events there. Handle the FML lifecycle events in your main mod class, and handle the other forge events in a separate handler.

 

Quote

GameRegistry.registerTileEntity(TileEntityBasicPlinth.class, CencialMod.MODID+"_basicplinth");

Don't use this overload that is marked as deprecated. Use the one that takes a ResourceLocation as it's second parameter.

 

Quote

public class GuiProxy

What on earth is a gui proxy?

 

Quote

public class BaseBlock

BaseBlock is anantipattern. There is already a BlockBase, it's called Block. Also:

Quote

@SideOnly(Side.CLIENT)

public void initModel()

IHasModel is stupid. All items need models, no exception, and nothing about model registration requires access to private/protected data of the block/item. Register your models in the ModelRegistryEvent directly. And yes, this is just another variant of IHasModel, you just have a class instead of an interface.

 

Quote

public static BlockBasicPlinth basicPlinth = new BlockBasicPlinth("basicplinth");

Don't ever use static initializers. Instantinate your things in the appropriate RegistryEvent directly.

 

Quote

ItemBlock itemBlock = new ItemBlock(this);

itemBlock.setRegistryName(name);

CencialItems.ITEMS.add(itemBlock);

See above point.

 

https://github.com/Sudwood/Cencial/blob/master/java/com/sudwood/cencial/blocks/BlockBasicPlinth.java#L98

You don't need all this awkward TE manipulations. Just override TileEntity#shouldRefresh.

[Cadiboo]

22 minutes ago, sudwood said:

CommonProxy.java

CommonProxy makes no sense. Proxies are meant to separate sided-only code. If the code is common it goes into your main class, not in your proxy.

GuiProxy.java

What???

inventory.deserializeNBT((NBTTagCompound) compound.getTag("inventory"));

What if the NBTTagCompound doesn't have the "inventory" tag?

[Cadiboo]

Also, what if the NBTTagCompound is null?

[sudwood]

Thank you for the detailed response. I fixed some of what you mentioned, however I do not understand any of:

Quote

Quote

@SideOnly(Side.CLIENT)

public void initModel()

IHasModel is stupid. All items need models, no exception, and nothing about model registration requires access to private/protected data of the block/item. Register your models in the ModelRegistryEvent directly. And yes, this is just another variant of IHasModel, you just have a class instead of an interface.

or

Quote

Quote

public static BlockBasicPlinth basicPlinth = new BlockBasicPlinth("basicplinth");

Don't ever use static initializers. Instantinate your things in the appropriate RegistryEvent directly.

and none of that has fixed the not saving problem. I have set the compound.getTag to compound.getCompoundTag in the readNBT function of the tileEntity https://github.com/Sudwood/Cencial/blob/master/java/com/sudwood/cencial/tileentities/TileEntityBasicPlinth.java#L43

and that did not fix the problem. I appreciate the help.

[Cadiboo]

1 minute ago, sudwood said:

compound.getCompoundTag in the readNBT function of the tileEntity https://github.com/Sudwood/Cencial/blob/master/java/com/sudwood/cencial/tileentities/TileEntityBasicPlinth.java#L43

 

“Inventory”!=“inventory”

also that’s now how comparisons in java work. You need to use .equals()

[sudwood]

2 minutes ago, Cadiboo said:

“Inventory”!=“inventory”

also that’s now how comparisons in java work. You need to use .equals()

Well its not saving still, but I no longer get any save errors in the console so maybe that did something?

if(!compound.getCompoundTag("inventory").equals(null))
		inventory.deserializeNBT((NBTTagCompound) compound.getCompoundTag("inventory"));

 

[V0idWa1k3r]

27 minutes ago, Cadiboo said:

also that’s now how comparisons in java work. You need to use .equals()

When comparing to null it is perfectly fine to use ==.

Also NBTTagCompound#getCompoundTag can't be null. If there is no such key in the tag map it will return a new, empty NBTTagCompound. If there is something with that key in the map but it isn't an instance of NBTTagCompound it will also return a new empty NBTTagCompound. Otherwise it will return the NBTTagCompound stored in the map.

 

26 minutes ago, sudwood said:

Well its not saving still, but I no longer get any save errors in the console so maybe that did something?

Could you please update your github repository to the latest state so I can debug it locally?

[sudwood]

8 hours ago, V0idWa1k3r said:

When comparing to null it is perfectly fine to use ==.

Also NBTTagCompound#getCompoundTag can't be null. If there is no such key in the tag map it will return a new, empty NBTTagCompound. If there is something with that key in the map but it isn't an instance of NBTTagCompound it will also return a new empty NBTTagCompound. Otherwise it will return the NBTTagCompound stored in the map.

 

Could you please update your github repository to the latest state so I can debug it locally?

Thank you for all the help. The current github repository is http://cencial.sudwood.com

[diesieben07]

• Why are you not using @Override when trying to override methods? This, for example, does nothing. The method is never called. Use @Override to inform the Java compiler "hey, I want to override here". If you get an error, then you are not actually overriding! The fix is not to remove the annotation!
• This if statement is never going to be false (and as such it's useless), as was already pointed out by @V0idWa1k3r. Moreover, that is terrible formatting. If leaving out the braces on an if statement, at least indent the 2nd line.
• You must use ItemStack#isEmpty to check if an ItemStack is empty. Never compare against ItemStack.EMTPY. An ItemStack that is not ItemStack.EMPTY can still be empty (example: here).
• Do not call hasCapability and getCapability immediately after each other. Instead just call getCapability and check for null. Moreover, calling getCapability or hasCapability on your own tile entities almost never makes any sense (you do it here, for example). Instead, simply access the field holding the instance directly (in this case the essence field).
• Why not else?
• Please use a language file for stuff like this.
• Do not use raw types (like here).
• Please also fix the issues that @V0idWa1k3r pointed out.

Edited November 11, 2018 by diesieben07