Exception on server start with new forge #787

Closed
Cisien opened this Issue Apr 27, 2013 · 2 comments

Projects

None yet

2 participants

@Cisien
Contributor
Cisien commented Apr 27, 2013

Minecraft 1.5.1
Forge 674
Buildcraft BuildCraft-1.5.1-jenkins-Buildcraft-126.126

This is possibly related to new liquid code in buildcraft

2013-04-26 22:08:09 [SEVERE] [ForgeModLoader] A TileEntity type buildcraft.transport.TileGenericPipe has throw an exception trying to write state. It
will not persist. Report this to the mod author
java.lang.IllegalArgumentException: Empty string not allowed
at net.minecraft.nbt.NBTTagString.(SourceFile:15)
at net.minecraft.nbt.NBTTagCompound.func_74778_a(SourceFile:77)
at net.minecraftforge.liquids.LiquidStack.writeToNBT(LiquidStack.java:57)
at buildcraft.transport.PipeTransportLiquids$PipeSection.writeToNBT(PipeTransportLiquids.java:122)
at buildcraft.transport.PipeTransportLiquids.writeToNBT(PipeTransportLiquids.java:329)
at buildcraft.transport.Pipe.writeToNBT(Pipe.java:201)
at buildcraft.transport.TileGenericPipe.func_70310_b(TileGenericPipe.java:103)
at net.minecraft.world.chunk.storage.AnvilChunkLoader.func_75820_a(AnvilChunkLoader.java:310)
at net.minecraft.world.chunk.storage.AnvilChunkLoader.func_75816_a(AnvilChunkLoader.java:127)
at net.minecraft.world.gen.ChunkProviderServer.func_73242_b(ChunkProviderServer.java:232)
at net.minecraft.world.gen.ChunkProviderServer.func_73151_a(ChunkProviderServer.java:284)
at net.minecraft.world.WorldServer.func_73044_a(WorldServer.java:906)
at net.minecraft.server.MinecraftServer.func_71267_a(MinecraftServer.java:344)
at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:576)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:468)
at net.minecraft.server.ThreadMinecraftServer.run(SourceFile:573)

@CovertJaguar
Member

Looks like LiquidStack needs a null check.

@CovertJaguar
Member

For clarification, I think this only happens with the liquid isn't registered in the Liquid Dictionary.

@CovertJaguar CovertJaguar referenced this issue in MinecraftForge/MinecraftForge May 11, 2013
Merged

Possible NPE if the liquid isn't in the LD #565

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment