In 1.7.10 to register a custom villager with a custom trade system it was as simple as.

VillagerRegistry.instance().registerVillagerId(ConfigurationHandler.getVillagerID());
VillagerRegistry.instance().registerVillagerSkin(ConfigurationHandler.getVillagerID(), new ResourceLocation(Reference.MOD_ID.toLowerCase(), "textures/entity/medic.png")); // ClientProxy
VillagerRegistry.instance().registerVillageTradeHandler(ConfigurationHandler.getVillagerID(), new MedicVillagerTradeHandler()); //Where MedicVilalgerTradeHandler implements IVillageTradeHandler

 

However in 1.8 IVillageTradeHandler is gone, registerVillageTradeHandler is also gone, I've gotten to the point where I have code like this.

 

VillagerRegistry.instance().registerVillagerId(ConfigurationHandler.getVillagerID());
VillagerRegistry.instance().registerVillagerSkin(ConfigurationHandler.getVillagerID(), new ResourceLocation(Reference.MOD_ID.toLowerCase(), "textures/entity/medic.png")); // ClientProxy
VillagerProfession prof = new VillagerProfession("zombieinfection:medic", "zombieinfection:textures/entity/medic.png");
VillagerRegistry.instance().register(prof);
(new VillagerCareer(prof, "Medic")).init();

 

Is what appears to be the trades, the argument passed is EntityVillager.ITradeList[][] I'm not sure how this would look like and how I would create my own to pass as an argument. Any help would be great.

If anyone has the solution, i would like to know too.

Thanks

No solution, I posted an issue on the MinecraftForge github and Lex said he forgot about the VillagerRegistry, it needs a complete rewrite and will probably come at a later time.

Thanks for posting on this.  Hopefully, we can get a getCareer method to go with the existing getProfession method as well.

I am waiting for this too, but it hasn't fixed yet or maybe I didn't see.

For those tracking this issue, you can find the open issue below:

 

https://github.com/MinecraftForge/MinecraftForge/issues/1687