Reapply Patch Optimize hasItemMeta (remove getItemMeta call)

[MelodyYuuka]

Reapply Patch #1279

Spigot 1.13 checks if any field (which are manually copied from the ItemStack's "tag" NBT tag) on the ItemMeta class of an ItemStack is set.

We could just check if the "tag" NBT tag is empty, albeit that would break some plugins. The only general tag added on 1.13 is "Damage", and we can just check if the "tag" NBT tag contains any other tag that's not "Damage" (https://minecraft.gamepedia.com/Player.dat_format#Item_structure) making the hasItemStack method behave as before.

Returns true if getDamage() != 0 or has damage tag or other tag is set.

Lastly, in Paper 1.20.4 there was still this patch: https://github.com/PaperMC/Paper-archive/blob/ver/1.20.4/patches/server/0221-Don-t-call-getItemMeta-on-hasItemMeta.patch, probably just did not survive the component rewrite

[Lulu13022002]

Given this already break since 1.20.6, i'm not sure the damage exclusion is really needed now (which will break things again).

[MelodyYuuka]

Given this already break since 1.20.6, i'm not sure the damage exclusion is really needed now (which will break things again).

We could replace this damage check by iterating over DataComponentPatch#entrySet() and comparing it with the ItemStack prototype. However, I feel like just checking the damage value would be enough, no one has opened an issue about hasItemMeta behaving differently from Spigot before. What do you think?

for (final Map.Entry<DataComponentType<?>, Optional<?>> entry : this.handle.getComponentsPatch().entrySet()) {
    if (!Objects.equals(this.handle.getPrototype().get(entry.getKey()), entry.getValue().orElse(null))) {
        return true;
    }
}
return false;