[1.21] `AttributeModifier` with the Same `ResourceLocation` are deduplicated even across `Attribute`s about accessories HOT 6 OPEN

For problem 1, it's because WrappedCurio::getDynamicModifiers invokes AccessoryAttributeBuilder::addExclusive, which use Map<ResourceLocation, AttributeModificationData>. It should be using Map<Holder<Attribute>, Map<ResourceLocation, AttributeModificationData>> or Table<Holder<Attribute>>, ResourceLocation, AttributeModificationData> instead.

from accessories.

1. Is the problem that AttributeModifier across multiple Attribute share the same ResourceLocation or is it the fact that multiple AttributeModifier have the same ResourceLocation for a single Attribute

from accessories.

@Dragon-Seeker The problem is having several AttributeModifiers sharing the same ResourceLocation for multiple Attributes

The method provides a ResourceLocation that wants the accessory to use for its attribute modifiers. The issue 1 can technically be solved by appending a different suffix for every modifier, but most mods are not doing that

from accessories.

Ah, that is really odd and frankly concerning as the ResourceLocation would be unique across all AttributeModifier's but Minecraft doesn't enforce such even though they merged the id and name into one field to uniquely identifier the attribute with info about why or naming the modifier instance so it kind seems lazy to just share across every modifer but I guess since vanilla allows such I will need to adjust such even if I feel that this should not happen its just not enforced.

2. I did not know the API functioned like such as it is quite odd, to say the least. I understand its reasons but I believe that Accessories should attempt to use the Vanilla method ItemStack.addModifierTooltip and any mods should attempt injection within there to adjust the Attribute tooltips. Understandable that currently, this affects the Curios API but I don't really know exactly fix such an issue as the Accessories API isn't designed for such. Frankly, I don't want to redesign this as it seems like the incorrect method to deal with this.

from accessories.

The tooltip modification part is there to help with the missing Attribute tooltip customization from Forge/NeoForge. I think there is a PR regarding this in NeoForge currently?

from accessories.

If this PR is completed, I wouldn't need to use the odd attribute tooltip method
neoforged/NeoForge#1371

from accessories.