gliscowo / interactic

Like ItemPhysic, but for fabric and with more gameplay features
https://modrinth.com/mod/interactic
MIT License
8 stars 7 forks source link

Disabling auto pickup interferes with Fabrication's instant pickup #9

Closed copygirl closed 2 years ago

copygirl commented 2 years ago

Versions

To Reproduce

Description

In Fabrication there is a feature called Instant Pickup that will automatically transfer items to a player's inventory when a block is broken or an entity is killed with a melee weapon. It achieves this by setting their pickup delay to 0 and calling onPlayerCollision, presumably to achieve best possible mod support. However, Interactic injects itself into exactly that method, effectively making Fabrication's tweak moot.

This is a very specific interactions between two mods and this particular choice of settings might be unique to what I'm trying to do, but I nonetheless wanted to create an issue. I'm not sure what the solution here could be, but perhaps there is something you can do about it.

In case there is any confusion as to my choice of settings: I like having Fabrication's instant pickup enabled, as I tend to want the blocks I'm mining. This way, I don't lose anything when I'm mining downwards, or have to look after I've done some digging. The pickup delay is also just long enough to be annoying at times. At the same time however, I'm not a huge fan of picking up random stuff that was littered about, or intentionally thrown out to let despawn, so Interactic helps with that.

Thanks for your time, and the nice mod! :green_heart:

gliscowo commented 2 years ago

Hey, thanks for reporting this issue. It has now been resolved by unascribed/Fabrication#460 and should soon be available in a release Have a great day

copygirl commented 2 years ago

Thank you very much for looking into this so quickly.