- edited description
- changed version to 1.0
- changed milestone to moddability
-
assigned issue to
[#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
builder
s to dynamically populate their inventory were switched over to use theGenericInventoryRestocker
part with theGenericInventoryRestockerPopulationTable
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 withGenericInventoryRestocker
(*), 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 fromGenericInventoryRestockerPopulationTable
.
Comments (4)
-
repo owner -
- changed status to open
-
-
- changed status to resolved
Implemented in next update.
- Log in to comment