Jump to content

[1.12.2] Custom Villager Trades Help


Recommended Posts

Hey guys, 


I'm attempting to implement custom villager trades for a mod I'm working on. I can't seem to get it to work, I followed the instructions here in this article, but I can't get a villager to spawn with the trade I'm wanting to add.

Here's my code:


public class SellTrades implements EntityVillager.ITradeList {
	private ItemStack itemToSell;
	private EntityVillager.PriceInfo sellPrice;
	private boolean enchant;
	public SellTrades(Item sell, EntityVillager.PriceInfo price, boolean shouldBeEnchanted) {
		itemToSell = new ItemStack(sell, 1);
		sellPrice = price;
		enchant = shouldBeEnchanted;
	public SellTrades(ItemStack sell, EntityVillager.PriceInfo price, boolean shouldBeEnchanted) {
		itemToSell = sell;
		sellPrice = price;
		enchant = shouldBeEnchanted;
	public void addMerchantRecipe(IMerchant merchant, MerchantRecipeList recipeList, Random random) {
		if (enchant)
			itemToSell = EnchantmentHelper.addRandomEnchantment(random, new ItemStack(this.itemToSell.getItem(), 1, this.itemToSell.getMetadata()), 5 + random.nextInt(15), false);
		ItemStack itemstack = new ItemStack(Items.EMERALD, sellPrice.getPrice(random));
		recipeList.add(new MerchantRecipe(itemstack, itemToSell));


Which works well. This allows me to specify a new trade with an item the villager is selling, and a price range for the number of emeralds required to buy said item.

So, according to the article linked above, I need to add a recipe to the villagers using my class. Here's how I did that:

VillagerRegistry.VillagerCareer smith = ForgeRegistries.VILLAGER_PROFESSIONS.getValue(new ResourceLocation("minecraft:smith")).getCareer(3);
    smith.addTrade(1, new SellTrades(BlockHandler.iMithrilPickaxe, new EntityVillager.PriceInfo(8, 11), true));

These two lines of code are in my ServerProxy/CommonProxy class. Initially, I had them in my init() method, and then I moved it to my preInit() method. Is this the correct way to register the trade? The article linked above doesn't mention anything about having to register the trade so I'm thinking I don't need to.



Link to comment
Share on other sites

I recommend learning how to use the debug tools to track problems down. You can use the debugger mode of Eclipse to set breakpoints and watch field values and you can use console or logger statements to help trace the execution. So in each of your methods, why don't you put a print statement to make sure it is executing and what the values are at the time of execution? Computers are fully logical so by simply observing the execution you will quickly find any problem.

Check out my tutorials here: http://jabelarminecraft.blogspot.com/

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

  • Create New...

Important Information

By using this site, you agree to our Terms of Use.