Logistics Pipes

Logistics Pipes

13M Downloads

Polymorphic item sink does not look at item of a locked empty JABBA barrel

bziemons opened this issue ยท 7 comments

commented

Look if it is possible to get the item from a locked but empty JABBA barrel and still sink items of the type to that barrel.

Partly a bug, but might not be fixable.

commented

That change looks inconsistent..

commented

Why? The only places that ask for the method are Specific Interests and The Polymorphic ItemSink.

commented

The current state is fine with it, but inconsistencies can break future code. Those two methods should not differ in the amount of item types they return. When you just look at their names (and/or the missing Javadoc) that does not make sense.

commented

So guess we probably only need to remove the itemstack.size>0
BTW are we handling TE Caches the same way?

commented

Probably.
IIRC we currently don't handle TE Caches at all.

commented

https://github.com/RS485/LogisticsPipes/blob/mc17/common/logisticspipes/proxy/specialinventoryhandler/JABBAInventoryHandler.java#L94
"items.stackSize-(_hideOnePerStack?1:0)" should not make it -1 though.

And it is needed to see where the methods are used and to see, if the check for the itemstack size is needed there instead.