From 1f02321e2767a4772a35f512fa29798b6aeadb9a Mon Sep 17 00:00:00 2001 From: Angelo Gallarello Date: Fri, 21 Sep 2018 14:53:04 +0200 Subject: Fixed rules about member and box dropdowns width --- client/components/rules/actions/cardActions.js | 8 ++++---- client/components/rules/rules.styl | 2 +- client/components/rules/triggers/cardTriggers.js | 10 +++++----- i18n/en.i18n.json | 2 +- models/cards.js | 6 ++++-- server/rulesHelper.js | 4 ++-- server/triggersDef.js | 4 ++-- 7 files changed, 19 insertions(+), 17 deletions(-) diff --git a/client/components/rules/actions/cardActions.js b/client/components/rules/actions/cardActions.js index a65407c1..b04440bd 100644 --- a/client/components/rules/actions/cardActions.js +++ b/client/components/rules/actions/cardActions.js @@ -58,14 +58,14 @@ BlazeComponent.extendComponent({ const ruleName = this.data().ruleName.get(); const trigger = this.data().triggerVar.get(); const actionSelected = this.find('#member-action').value; - const memberName = this.find('#member-name').value; + const username = this.find('#member-name').value; const boardId = Session.get('currentBoard'); const desc = Utils.getTriggerActionDesc(event, this); if (actionSelected === 'add') { const triggerId = Triggers.insert(trigger); const actionId = Actions.insert({ actionType: 'addMember', - memberName, + username, boardId, desc, }); @@ -81,7 +81,7 @@ BlazeComponent.extendComponent({ const triggerId = Triggers.insert(trigger); const actionId = Actions.insert({ actionType: 'removeMember', - memberName, + username, boardId, desc, }); @@ -101,7 +101,7 @@ BlazeComponent.extendComponent({ const boardId = Session.get('currentBoard'); const actionId = Actions.insert({ actionType: 'removeMember', - 'memberName': '*', + 'username': '*', boardId, desc, }); diff --git a/client/components/rules/rules.styl b/client/components/rules/rules.styl index 68d74d32..45ce4003 100644 --- a/client/components/rules/rules.styl +++ b/client/components/rules/rules.styl @@ -116,7 +116,7 @@ .trigger-dropdown display:inline-block select - width:100px + width:auto height:30px margin:0px margin-left:5px diff --git a/client/components/rules/triggers/cardTriggers.js b/client/components/rules/triggers/cardTriggers.js index 704c7690..2303a85b 100644 --- a/client/components/rules/triggers/cardTriggers.js +++ b/client/components/rules/triggers/cardTriggers.js @@ -67,7 +67,7 @@ BlazeComponent.extendComponent({ datas.triggerVar.set({ activityType: 'joinMember', boardId, - 'memberId': '*', + 'username': '*', desc, }); } @@ -75,7 +75,7 @@ BlazeComponent.extendComponent({ datas.triggerVar.set({ activityType: 'unjoinMember', boardId, - 'memberId': '*', + 'username': '*', desc, }); } @@ -84,13 +84,13 @@ BlazeComponent.extendComponent({ const desc = Utils.getTriggerActionDesc(event, this); const datas = this.data(); const actionSelected = this.find('#spec-member-action').value; - const memberId = this.find('#spec-member').value; + const username = this.find('#spec-member').value; const boardId = Session.get('currentBoard'); if (actionSelected === 'added') { datas.triggerVar.set({ activityType: 'joinMember', boardId, - memberId, + username, desc, }); } @@ -98,7 +98,7 @@ BlazeComponent.extendComponent({ datas.triggerVar.set({ activityType: 'unjoinMember', boardId, - memberId, + username, desc, }); } diff --git a/i18n/en.i18n.json b/i18n/en.i18n.json index 81206ae3..a459be36 100644 --- a/i18n/en.i18n.json +++ b/i18n/en.i18n.json @@ -548,7 +548,7 @@ "r-when-the-label-is": "When the label is", "r-list-name": "List name", "r-when-a-member": "When a member is", - "r-when-the-member": "When the member is", + "r-when-the-member": "When the member", "r-name": "name", "r-is": "is", "r-when-a-attach": "When an attachment", diff --git a/models/cards.js b/models/cards.js index 346b4bdd..d91b9050 100644 --- a/models/cards.js +++ b/models/cards.js @@ -1165,10 +1165,11 @@ function cardMembers(userId, doc, fieldNames, modifier) { // Say hello to the new member if (modifier.$addToSet && modifier.$addToSet.members) { memberId = modifier.$addToSet.members; + let username = Users.findOne(memberId).username; if (!_.contains(doc.members, memberId)) { Activities.insert({ userId, - memberId, + username, activityType: 'joinMember', boardId: doc.boardId, cardId: doc._id, @@ -1179,11 +1180,12 @@ function cardMembers(userId, doc, fieldNames, modifier) { // Say goodbye to the former member if (modifier.$pull && modifier.$pull.members) { memberId = modifier.$pull.members; + let username = Users.findOne(memberId).username; // Check that the former member is member of the card if (_.contains(doc.members, memberId)) { Activities.insert({ userId, - memberId, + username, activityType: 'unjoinMember', boardId: doc.boardId, cardId: doc._id, diff --git a/server/rulesHelper.js b/server/rulesHelper.js index 175af953..631ad4e9 100644 --- a/server/rulesHelper.js +++ b/server/rulesHelper.js @@ -87,7 +87,7 @@ RulesHelper = { card.removeLabel(action.labelId); } if(action.actionType === 'addMember'){ - const memberId = Users.findOne({username:action.memberName})._id; + const memberId = Users.findOne({username:action.username})._id; card.assignMember(memberId); } if(action.actionType === 'removeMember'){ @@ -97,7 +97,7 @@ RulesHelper = { card.unassignMember(members[i]); } }else{ - const memberId = Users.findOne({username:action.memberName})._id; + const memberId = Users.findOne({username:action.username})._id; card.unassignMember(memberId); } } diff --git a/server/triggersDef.js b/server/triggersDef.js index 81dc946f..f6d5333b 100644 --- a/server/triggersDef.js +++ b/server/triggersDef.js @@ -12,10 +12,10 @@ TriggersDef = { matchingFields: ['boardId'], }, joinMember:{ - matchingFields: ['boardId', 'memberId'], + matchingFields: ['boardId', 'username'], }, unjoinMember:{ - matchingFields: ['boardId', 'memberId'], + matchingFields: ['boardId', 'username'], }, addChecklist:{ matchingFields: ['boardId', 'checklistName'], -- cgit v1.2.3-1-g7c22