[1.16] Server crash when trying to put a bottle in a black hole tank with potion
mthp1999 opened this issue · 3 comments
Industrial Foregoing Version:
1.16.4-4.1.2-129
Titanium Version:
1.16.4-3.2.5
Crashlog If Applicable (Upload to pastebin/gist):
https://pastebin.com/TFd8GzpN
Happened when I tried to put a glass bottle into the black hole tank when it was full with a potion fluid.
I just had a look at the crash, I don't think this is on our side. Yes, we do sometimes call simulate with empty (but not null) fluid stacks (and we should probably change that), but that's not what's crashing here. Here you are calling orElseGet
with null
as the other
-argument, and part of the specification for orElseGet
is
Throws: NullPointerException – If
other
isnull
and thisLazyOptional
is non-empty (sic: should be "is empty")
So my guess is that potion bottles do not have any fluid handler (we don't attach one in IE, I think we may have done at some point), so it tries to use your "supplier" and fails. You probably want orElse
, without the Get
.