From 8d929e07a480e057eaa5af71204631f3dd3bbdb3 Mon Sep 17 00:00:00 2001 From: Dennis Eichhorn Date: Mon, 22 May 2017 20:55:04 +0200 Subject: [PATCH] "Fix" same action type --- UI/ActionManager.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/UI/ActionManager.js b/UI/ActionManager.js index e685ad8..c4cb370 100644 --- a/UI/ActionManager.js +++ b/UI/ActionManager.js @@ -75,14 +75,13 @@ actionLength = listeners[i].action.length; for (let j = 1; j < actionLength; j++) { - this.app.eventManager.attach(e.id + listeners[i].action[j - 1].type, function (data) + console.log(e); + this.app.eventManager.attach(e.id + listeners[i].action[j - 1].key, function (data) { self.runAction(e, listeners[i].action[j], data); }, false, true); - // todo the true here is a memory leak since it should be removed at some point?! + // todo: the true here is a memory leak since it should be removed at some point?! // todo: handle onload action right after registering everything. this will be used for onload api calls in order to get content such as lists or models. Maybe in the main application after registering a invoke('onload') should be called if the application wants to execute the onload elements - // todo: right now one event type can only exist once... needs fixing!!! - //this.app.eventManager.addGroup(e.id + listeners[i].action[j - 1].type, listeners[i].action[j - 1].type); } // Register event for first action @@ -119,7 +118,7 @@ this.actions[action.type](action, function (data) { - self.app.eventManager.trigger(e.id + action.type, e.id, data); + self.app.eventManager.trigger(e.id + action.key, e.id, data); }); };