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)