浏览代码

fix(jade): simplify rules headers and conditions; add helpers to expose ruleName safely

Lauri Ojansivu 22 小时之前
父节点
当前提交
1808ea7384

+ 5 - 7
client/components/rules/rulesActions.jade

@@ -1,9 +1,7 @@
 template(name="rulesActions")
 template(name="rulesActions")
   h2
   h2
     | ✨
     | ✨
-    | {{_ 'r-rule' }} "
-    = ruleName.get()
-    | " - {{_ 'r-add-action'}}
+    | {{_ 'r-rule' }} "{{ruleNameStr}}" - {{_ 'r-add-action'}}
   .triggers-content
   .triggers-content
     .triggers-body
     .triggers-body
       .triggers-side-menu
       .triggers-side-menu
@@ -17,13 +15,13 @@ template(name="rulesActions")
           li.js-set-mail-actions
           li.js-set-mail-actions
             | @
             | @
       .triggers-main-body
       .triggers-main-body
-        if ($eq currentActions.get 'board')
+        if $eq currentActions.get 'board'
           +boardActions(ruleName=data.ruleName triggerVar=data.triggerVar)
           +boardActions(ruleName=data.ruleName triggerVar=data.triggerVar)
-        else if ($eq currentActions.get 'card')
+        else if $eq currentActions.get 'card'
           +cardActions(ruleName=data.ruleName triggerVar=data.triggerVar)
           +cardActions(ruleName=data.ruleName triggerVar=data.triggerVar)
-        else if ($eq currentActions.get 'checklist')
+        else if $eq currentActions.get 'checklist'
           +checklistActions(ruleName=data.ruleName triggerVar=data.triggerVar)
           +checklistActions(ruleName=data.ruleName triggerVar=data.triggerVar)
-        else if ($eq currentActions.get 'mail')
+        else if $eq currentActions.get 'mail'
           +mailActions(ruleName=data.ruleName triggerVar=data.triggerVar)
           +mailActions(ruleName=data.ruleName triggerVar=data.triggerVar)
   div.rules-back
   div.rules-back
         button.js-goback
         button.js-goback

+ 9 - 0
client/components/rules/rulesActions.js

@@ -5,6 +5,15 @@ BlazeComponent.extendComponent({
     this.currentActions = new ReactiveVar('board');
     this.currentActions = new ReactiveVar('board');
   },
   },
 
 
+  ruleNameStr() {
+    const rn = this.data() && this.data().ruleName;
+    try {
+      return rn && typeof rn.get === 'function' ? rn.get() : '';
+    } catch (_) {
+      return '';
+    }
+  },
+
   setBoardActions() {
   setBoardActions() {
     this.currentActions.set('board');
     this.currentActions.set('board');
     $('.js-set-card-actions').removeClass('active');
     $('.js-set-card-actions').removeClass('active');

+ 1 - 3
client/components/rules/rulesTriggers.jade

@@ -1,9 +1,7 @@
 template(name="rulesTriggers")
 template(name="rulesTriggers")
   h2
   h2
     | ✨
     | ✨
-    | {{_ 'r-rule' }} "
-    = ruleName.get()
-    | " - {{_ 'r-add-trigger'}}
+    | {{_ 'r-rule' }} "{{ruleNameStr}}" - {{_ 'r-add-trigger'}}
   .triggers-content
   .triggers-content
     .triggers-body
     .triggers-body
       .triggers-side-menu
       .triggers-side-menu

+ 9 - 0
client/components/rules/rulesTriggers.js

@@ -7,6 +7,15 @@ BlazeComponent.extendComponent({
     this.showChecklistTrigger = new ReactiveVar(false);
     this.showChecklistTrigger = new ReactiveVar(false);
   },
   },
 
 
+  ruleNameStr() {
+    const rn = this.data() && this.data().ruleName;
+    try {
+      return rn && typeof rn.get === 'function' ? rn.get() : '';
+    } catch (_) {
+      return '';
+    }
+  },
+
   setBoardTriggers() {
   setBoardTriggers() {
     this.showBoardTrigger.set(true);
     this.showBoardTrigger.set(true);
     this.showCardTrigger.set(false);
     this.showCardTrigger.set(false);