Feature Request or bug report? Supplementaries Red Merchant
AnyankaT opened this issue · 5 comments
I'm running LC on my server along with Supplementaries and several other mods. LC recognizes the trader villagers from other mods (Ars Nouveau and Railcraft Reborn) but not the Red Merchant from Supplementaries, and therefore does not replace the trades with currency.
Minecraft 1.20.1
Forge 47.1.43
LC 1.20.1-2.1.2.4b
Supplementaries 1.20-2.7.12
Unfortunately I don't think this will be possible, as their red merchant trades are hard-coded (Source) instead of registered via forges VillagerTradesEvent. Therefore I have no event to hook into to modify the listings without resorting to mixins, and I'm not sure if you can make a mixin require that another mod be loaded...
Just for grins, I made a feature request with Supplementaries to see if they are willing to add an event I can use.
https://legacy.curseforge.com/minecraft/mc-mods/supplementaries/issues/928
It would appear that they did add the event I requested, and with it I should (hopefully) be able to modify the trades the same way I do the vanilla and other modded villagers.
I'll do some testing real quick, and if all goes well it should have its trades modified if the config option to modify modded villagers is enabled (with its "profession" being hard-coded as supplementaries:red_merchant
for use in the replacementCoinOverrides
config option).