Closed GGAlanSmithee closed 6 years ago
The above is actually not true.
The implicit nature of those methods and their arguments is problematic though. In e26ae63ccea90d40f124a09cf9d6409acb629422 this has now been changed to be more explicit, i.e. entityManager.trigger(event: string, opts?: object)
.
Will be merged to dev and eventually to master. closing.
atm,
{eventHandler/entityManager}.{trigger/triggerDelayed}
, triggers all registered events. This is a bit naive and should be extended to actually be useful.Proposal:
trigger
/triggerDelayed
takes a optionalkey
parameter 1.1 ifkey
isnull
, trigger all events (as currently implemented) 1,2 if key is notnull
and only triggers corresponding event