summaryrefslogtreecommitdiffstats
path: root/server
diff options
context:
space:
mode:
authorAkuket <32392661+Akuket@users.noreply.github.com>2018-10-09 14:49:12 +0200
committerGitHub <noreply@github.com>2018-10-09 14:49:12 +0200
commit28a01862d04654784bd6010035c3a6fe1690ba60 (patch)
treeee52d3c84b12f5ada9a3faf12925dd5806385a91 /server
parent3b4f285fea4a90ee96bfce855e1539adcec9b7aa (diff)
parent58855ada97677d25475ba4721834c0f576ed3b47 (diff)
downloadwekan-28a01862d04654784bd6010035c3a6fe1690ba60.tar.gz
wekan-28a01862d04654784bd6010035c3a6fe1690ba60.tar.bz2
wekan-28a01862d04654784bd6010035c3a6fe1690ba60.zip
Merge branch 'edge' into edge
Diffstat (limited to 'server')
-rw-r--r--server/rulesHelper.js17
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;