Weird interaction between Better Combat, MC Dungeons Weapons and TooManyOrigins
finnglink opened this issue · 7 comments
Hey!
I'm having a really weird issue with my little pack, which seems to come from some sort of correlation between the three named mods: When I equip a two handed weapon form MCDW / BC while holding something in my offhand, that item gets deleted. And this seems to be caused by TMO, removing that fixes it. Any ideas why the might be happening?
Better Combat issue and duplicate of #51.
I can really only bandaid fix this in Apugli (a library that TooManyOrigins uses) by making the check specifically for ItemStack.EMPTY, rather than ItemStack#isEmpty.
That mod should probably not return non empty ItemStacks with 2h weapons as empty to avoid situations like this in the future.
A more detailed explanation can be found below:
ZsoltMolnarrr/BetterCombat#268 (comment)
@MerchantPug awesome! Feel free to ping me to test it, and I'll just drop it in my pack to check
and my previous commit didn't actually fix it... At least it got me to optimise everything a bit.
Re-opening as it was agreed upon that the fix would be cleaner if it were done on my end.
Fixed, it scared me initially because I didn't know how the mechanic worked, but your items should now be preserved in your offhand. If any of you have a server with players, please give them back their items that they lost due to this bug.
