From 2b2dcfe7ce66fafb53410e45ba13a6b343223ff6 Mon Sep 17 00:00:00 2001
From: Matteo Pagliazzi <matteopagliazzi@gmail.com>
Date: Wed, 6 Jan 2016 12:25:53 +0100
Subject: [PATCH] do not delete completed todos that belongs to a challenge

---
 website/src/middlewares/api-v3/cron.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/website/src/middlewares/api-v3/cron.js b/website/src/middlewares/api-v3/cron.js
index 0d6d46b900..f606a72aea 100644
--- a/website/src/middlewares/api-v3/cron.js
+++ b/website/src/middlewares/api-v3/cron.js
@@ -34,6 +34,7 @@ export default function cronMiddleware (req, res, next) {
     cron({user, tasksByType, now, daysMissed, analytics});
 
     // Clean completed todos - 30 days for free users, 90 for subscribers
+    // Do not delete challenges completed todos TODO unless the task is broken?
     Task.remove({
       userId: user._id,
       type: 'todo',
@@ -41,6 +42,7 @@ export default function cronMiddleware (req, res, next) {
       dateCompleted: {
         $lt: moment(now).subtract(user.isSubscribed() ? 90 : 30, 'days'),
       },
+      'challenge.id': {$exists: false},
     }).exec(); // TODO catch error or at least log it
 
     let ranCron = user.isModified();