Fix ItemType#isEdible to also check for DataComponents#CONSUMABLE

[aerulion]

This fix only allows items that can actually be eaten by the player. For example, since version 1.21.11, fish buckets have also provided the FOOD component, but they cannot be eaten.

I wasn't sure if this was worth dealing with right now, so I asked about it on Discord here, but it seemed to go unnoticed. If PRs aren't welcome yet, please close it; I just didn't want it to be overlooked.

[lynxplay]

Sounds sensible to me.

[Doc94]

Im not sure what was the "edible" definition when the method was added.. maybe can be good add a note about that? because for me edible can be just consumable.

[Lulu13022002]

Yes, the javadocs could be updated to clear any confusion that it only applies for players

[aerulion]

Im not sure what was the "edible" definition when the method was added.. maybe can be good add a note about that? because for me edible can be just consumable.

But just checking consumable would definitely be a behavior change since it would also include things like Potions, Ominous Bottles, etc., that weren't considered edible before. Also, then one could just use .hasDefaultData(DataComponentTypes.CONSUMABLE), and I wouldn't really see the point of why this component would have its own method while the others don't (although that already has been the case with the old FOOD component check).

But I will add a bit more detailed Javadocs.