From ee938c04232ab9ac42195a35d38cb7ae11b14d48 Mon Sep 17 00:00:00 2001
From: Ghassen Rjab <rjab.ghassen@gmail.com>
Date: Thu, 31 Aug 2017 06:58:02 +0100
Subject: Delete activity related to checklist item when the item is deleted

---
 models/checklists.js | 25 +++++++++++++++++--------
 1 file changed, 17 insertions(+), 8 deletions(-)

(limited to 'models/checklists.js')

diff --git a/models/checklists.js b/models/checklists.js
index ab4fe61d..59682720 100644
--- a/models/checklists.js
+++ b/models/checklists.js
@@ -164,14 +164,23 @@ if (Meteor.isServer) {
   // The future is now
   Checklists.after.update((userId, doc, fieldNames, modifier) => {
     if (fieldNames.includes('items')) {
-      Activities.insert({
-        userId,
-        activityType: 'addChecklistItem',
-        cardId: doc.cardId,
-        boardId: Cards.findOne(doc.cardId).boardId,
-        checklistId: doc._id,
-        checklistItemId: modifier.$addToSet.items._id,
-      });
+      if (modifier.$addToSet) {
+        Activities.insert({
+          userId,
+          activityType: 'addChecklistItem',
+          cardId: doc.cardId,
+          boardId: Cards.findOne(doc.cardId).boardId,
+          checklistId: doc._id,
+          checklistItemId: modifier.$addToSet.items._id,
+        });
+      } else if (modifier.$pull) {
+        const activity = Activities.findOne({
+          checklistItemId: modifier.$pull.items._id,
+        });
+        if (activity) {
+          Activities.remove(activity._id);
+        }
+      }
     }
   });
 
-- 
cgit v1.2.3-1-g7c22