From 6536b855c987a99749ae9868ddd56b853bf38cd1 Mon Sep 17 00:00:00 2001
From: Tyler Renelle <tylerrenelle@gmail.com>
Date: Tue, 14 Jan 2014 18:03:16 -0700
Subject: [PATCH] instead of clearing completed on cron, simply hide them from
 the angular apps (which only use batch-update API route) @colegleason

---
 src/controllers/user.js | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/src/controllers/user.js b/src/controllers/user.js
index bd42509f08..524fb588f4 100644
--- a/src/controllers/user.js
+++ b/src/controllers/user.js
@@ -451,12 +451,23 @@ api.batchUpdate = function(req, res, next) {
     res.json = oldJson;
     res.send = oldSend;
     if (err) return res.json(500, {err: err});
+
     var response = user.toJSON();
     response.wasModified = res.locals.wasModified;
+
+    // return only drops & streaks
     if (response._tmp && response._tmp.drop){
       res.json(200, {_tmp: {drop: response._tmp.drop}, _v: response._v});
+
+    // Fetch full user object
     }else if(response.wasModified){
+      // Preen 3-day past-completed To-Dos from Angular & mobile app
+      response.todos = _.where(response.todos, function(t) {
+        return !t.completed || (t.challenge && t.challenge.id) || moment(t.created).isAfter(moment().subtract('days',3));
+      });
       res.json(200, response);
+
+    // return only the version number
     }else{
       res.json(200, {_v: response._v});
     }