diff options
author | Akuket <32392661+Akuket@users.noreply.github.com> | 2018-10-09 14:49:12 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-10-09 14:49:12 +0200 |
commit | 28a01862d04654784bd6010035c3a6fe1690ba60 (patch) | |
tree | ee52d3c84b12f5ada9a3faf12925dd5806385a91 /server | |
parent | 3b4f285fea4a90ee96bfce855e1539adcec9b7aa (diff) | |
parent | 58855ada97677d25475ba4721834c0f576ed3b47 (diff) | |
download | wekan-28a01862d04654784bd6010035c3a6fe1690ba60.tar.gz wekan-28a01862d04654784bd6010035c3a6fe1690ba60.tar.bz2 wekan-28a01862d04654784bd6010035c3a6fe1690ba60.zip |
Merge branch 'edge' into edge
Diffstat (limited to 'server')
-rw-r--r-- | server/rulesHelper.js | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/server/rulesHelper.js b/server/rulesHelper.js index e9139933..833092d0 100644 --- a/server/rulesHelper.js +++ b/server/rulesHelper.js @@ -3,7 +3,9 @@ RulesHelper = { const matchingRules = this.findMatchingRules(activity); for(let i = 0; i< matchingRules.length; i++){ const action = matchingRules[i].getAction(); - this.performAction(activity, action); + if(action !== undefined){ + this.performAction(activity, action); + } } }, findMatchingRules(activity){ @@ -16,7 +18,12 @@ RulesHelper = { const matchingTriggers = Triggers.find(matchingMap); const matchingRules = []; matchingTriggers.forEach(function(trigger){ - matchingRules.push(trigger.getRule()); + const rule = trigger.getRule(); + // Check that for some unknown reason there are some leftover triggers + // not connected to any rules + if(rule !== undefined){ + matchingRules.push(trigger.getRule()); + } }); return matchingRules; }, @@ -65,10 +72,10 @@ RulesHelper = { const emailSubject = action.emailSubject; try { Email.send({ - to, + emailTo, from: Accounts.emailTemplates.from, - subject, - text, + emailSubject, + emailMsg, }); } catch (e) { return; |