From 83649c6734b8c9a298e38de5457d45a5c257b3dc Mon Sep 17 00:00:00 2001
From: Tyler Renelle <tylerrenelle@gmail.com>
Date: Wed, 10 Apr 2013 19:24:33 -0400
Subject: [PATCH] fix clear-completed

---
 src/app/tasks.coffee | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/src/app/tasks.coffee b/src/app/tasks.coffee
index 52a726d343..f15502fb91 100644
--- a/src/app/tasks.coffee
+++ b/src/app/tasks.coffee
@@ -66,15 +66,11 @@ module.exports.app = (appExports, model) ->
 
 
   appExports.clearCompleted = (e, el) ->
+    completedIds =  _.pluck( _.where(model.get('_todoList'), {completed:true}), 'id')
     todoIds = user.get('todoIds')
-    removed = false
-    _.each model.get('_todoList'), (task) ->
-      if task.completed
-        removed = true
-        user.del('tasks.'+task.id)
-        todoIds.splice(todoIds.indexOf(task.id), 1)
-    if removed
-      user.set('todoIds', todoIds)
+
+    _.each completedIds, (id) -> user.del "tasks.#{id}"
+    user.set 'todoIds', _.difference(todoIds, completedIds)
 
   appExports.toggleDay = (e, el) ->
     task = model.at(e.target)