Curios Compat Layer for Accessories

Curios Compat Layer for Accessories

719k Downloads

Missing support for `Inventory.contains(Predicate<ItemStack>)`

EnnuiL opened this issue ยท 2 comments

commented

Hello,
Currently, Ok Zoomer needs extra code in order to explicitly support Accessories, however, it doesn't have to be this way;

While the other two contains methods are supported natively by the mod, the predicate-based contains (named hasAnyMatching in 1.20.1) doesn't have that same support, which on Curios' side was confirmed to be an oversight;

It would be nice if this vanilla method was supported on Accessories itself so that I could delete the compat code and trust the mod; I would send a PR, but I need to minimize the amount of work I do right now

commented

The latest update for 1.21.1 contains such a change to fix what was missed

commented

Ah yes, such was missed I guess. Will be added within 1.21.5 and planning a backport of 1.21.5 codebase to 1.21.1 coming up which will fix it there.