[#bugs] It'd be nice if merchants that use `builder`s to dynamically populate their inve...

Marked for crossposting by: roko's snoopylisk

Message (jump):

<kernelmethod> It'd be nice if merchants that use builders to dynamically populate their inventory were switched over to use the GenericInventoryRestocker part with the GenericInventoryRestockerPopulationTable tag (esp. dromads, Consortium merchants, and NPCs with useful supplies like Jacobo and Sparafucile).

From a moddability perspective, it would be much easier to make changes to their inventories. And it appears that in general, the BaseMerchantWares builders do not interact very well with GenericInventoryRestocker (*), meaning you can't easily merge into a creature's definition to modify the items they stock.

(*) it appears that GenericInventoryRestocker and the builders run independently and both trigger a restock (which also causes the <creature> has restocked their inventory! message to appear twice in the event log). My experience has also been that after a few restocks, the builders typically override the population table from GenericInventoryRestockerPopulationTable.

Comments (4)