search

copygirl / BetterStorage

A Minecraft mod aimed at offering more storage options.
http://copy.mcft.net/mc/BetterStorage/
MIT License
58 stars 33 forks source link

Crash with WAWLA when placing backpack #233

Closed spikeforever closed 10 years ago

spikeforever commented 10 years ago

i can supply further information about the crash if necessary.

http://pastebin.com/2QEw7TNk

copygirl commented 10 years ago

Flying over the crash log I saw Waila in there, almost assumed it's the same problem as I've seen before, but this is new. Why is writeToNBT called on the client? Marking this wontfix for now until I hear back from ProfMobius.

oitsjustjose commented 10 years ago

I experienced the same issue with NEI's built-in Highlight Tooltips too, so maybe you might want to speak with Chickenbones too? http://pastebin.com/LJvJ9Gpv

copygirl commented 10 years ago

@oitsjustjose If you look at the crash you can see it's not the same one. The issue you're mentioning, I believe, is the backpack returning null for getPickBlock (or whatever it's called), which in my opinion should be valid. This error is caused by the writeToNBT function being called on the client, where the data to be "saved" might not be available.

Victorious3 commented 10 years ago

I looked around the crash report for a little bit and it might actually be WAWLA and not WAILA that's causing this. I'll test it for a little bit and then there is someone I have to poke about it...

Victorious3 commented 10 years ago

I talked to darkhax and he already fixed the issue a while ago. @spikeforever Update to WAWLA version 1.0.6