MachineMuse / MachineMusePowersuits

Minecraft mod, take 2
236 stars 105 forks source link

Ore poison crash on mining with full suit on and world disappeared from list [Exception in server tick loop] #458

Closed benten2000 closed 10 years ago

benten2000 commented 10 years ago

During my singlerplayer.

Found some Uranium ore from Atomic Science and with one hit from pickaxe Minecraft 1.6.4 closed to desktop.

Using Minecraft 1.6.4 / with ModularPowersuits from Jenkins #84 (all 5 files)

Here is a copy of the crash report (http://pastebin.com/UzfmB1Hi)

Also what was terrible was upon loading Minecraft again and going back to singleplayer my World was not on the list anymore but the world folder of it still resides in the Minecraft folder.

CodeRage2 commented 10 years ago

This also happens when someone on the server picks up an Atomic Science fuel rod. There is some type of conflict with the radiation poisoning and power suits. The problem can be fixed for the user by using /invsee the second they log back in and removing the fuel rod/radioactive item from their inventory. They will crash again, but keep their inventory open until they log back in again or it won't save.

remexre commented 10 years ago

I may not know Scala, but it looks like ItemPowerArmor or ItemElectricArmor needs to override isProtectedFromPoison(ItemStack, EntityLivingBase, String). If I can learn Scala today, I'll submit a patch (although I'm not sure what "intended behaviour" is.

axelpx commented 10 years ago

This makes Atomic Science pretty much uncompatible with MPS. Thats quite sad. Hope this will be fixed soon, cuz if you crash like this on the server you can't log back in without asking OP to delete your inventory...

MachineMuse commented 10 years ago

Try build 89 please.

axelpx commented 10 years ago

Just tested it in single and multiplayer and it seems to be fixed! Thanks guys :D

MachineMuse commented 10 years ago

awesome :D