PneumaticCraft: Repressurized

PneumaticCraft: Repressurized

50M Downloads

Create public default constructor for SmartChestItemHandler

juh9870 opened this issue ยท 3 comments

commented

Please create public constructor for SmartChestItemHandler, so it can be serialized/deserialized by other mods without usage of mixins/reflection. Specifically, I'm working on Create addon that adds support for modded storage, and I can't add SmartChestItemHandler deserialization without extensive usage of reflections

commented

What specifically do you need from SmartChestItemHandler that isn't available through normal capability querying?

commented

Create moving contraption can pull SmartChestItemHandler from chest normally and it will work untill you serialize and deserialize contraption, at which point SmartChestItemHandler is deserialized as default ItemStackHandler, breaking all custom logic, and since tile entity does not exist at that point we can't just pull another handler, the only option is to create new SmartChestItemHandler and deserialize it from previos SmartChestItemHandler NBT, which is my addon does with custom handlers from other mods, but I can't do that here because I can't instantiate SmartChestItemHandler outside of tile entity

commented

Added in 2.14.6 release