Farmer's Delight

Farmer's Delight

95M Downloads

[1.16.5] Cooking Board recipes use wrong items

NielsPilgaard opened this issue ยท 6 comments

commented

Description
The Cutting Board accepts Armor from the mod Better End in recipes that require an axe

Steps to reproduce:
I looked in JEI, can be replicated.

Expected behavior:
I expected only axes to be valid in axe recipes

Screenshots
image

Mod list:

commented

That sounds like a wacky integration that went wrong. Either someone tagged these chestplates as forge:tools/axes, or the chestplates somehow have the axe ToolType...

Is it happening exclusively to BetterEnd armor?

commented

BetterEnd Armor and the Quark Pickarang I believe :)
Edit: No strange tool tags on the armor :P

commented

And I couldn't verify ToolTypes either; their repo doesn't seem to have Terminite and Thallasium armors (the ones that became usable in the board) registered on their ModItems. The other two armor materials seem to be fine, though, and can't be used as axes.

This is kinda puzzling... I may have to dig deeper. Maybe the BetterEnd devs might be able to help?

commented

It is a bug on our part. In this class you can see the registration of the Terminite/Thallasium items, and as you can see, we use the same instance of Item.Properties for every item, which means that the armor will get every tool type, since the tooltypes are added to the Item.Properties instance in the constructors of the various tool item classes (AxeItem, PickaxeItem etc).

commented

Good to know the cause was found! Thanks for the help! ๐Ÿ˜„

commented

NIce find Vemerion, this was a very subtle issue! It now can be fixed very easily