MC-179352The leatherworker's second leather tunic trade give too little villager XP
Journeyman leatherworker villagers will sell a randomly dyed leather tunic for 7 emeralds. However, this trade only gives 1 XP to the villager. This amount is inconsistent with every other trade at this level, both on leatherworkers and other villagers.
This issue occurs because the trade is only defined as the following:
DyedArmorForEmeralds(Items.LEATHER_CHESTPLATE, 7)
This means the trade will cost 7 emeralds and use the default value for villagerXp, which is set to 1 for this specific trade type.
This issue could easily be fixed by changing that line to the following (or similar):
DyedArmorForEmeralds(Items.LEATHER_CHESTPLATE, 7, 12, 10)
Created Issue:
The leatherworker's second leather tunic trade give too little villager XP
Journeyman leatherworker villagers will sell a randomly dyed leather tunic for 7 emeralds. However, this trade only gives 1 XP to the villager. This amount is inconsistent with every other trade at this level, both on leatherworkers and other villagers.
This issue occurs because the trade is only defined as the following:
DyedArmorForEmeralds(Items.LEATHER_CHESTPLATE, 7)
This means the trade will cost 7 emeralds and use the default value for villagerXp, which is set to 1 for this specific trade type.
This issue could easily be fixed by changing that line to the following (or similar):
DyedArmorForEmeralds(Items.LEATHER_CHESTPLATE, 7, 12, 10)
![[Mod] violine1101](../../../data/avatars/users/92858.png){kind=small}
![[Mod] tryashtar](../../../data/avatars/users/41963.png){kind=small}
![[Mod] Michael Wobst](../../../data/avatars/users/37868.png){kind=small}
![[Bot] Arisa](../../../data/avatars/users/43007.png){kind=small}
Can confirm that this is an issue in 20w51a.
Issue is still present as of Snapshot 24w45a