(flight) fix clearCompleted bug.

passy · Jun 29, 2013 · cceaffa · cceaffa
1 parent 757d0fc
commit cceaffa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/dependency-examples/flight/app/js/data/todos.js b/dependency-examples/flight/app/js/data/todos.js
@@ -77,11 +77,10 @@ define([
 		};
 
 		this.clearCompleted = function () {
-			var todos;
-
 			dataStore.destroyAll({ completed: true });
-			todos = dataStore.all();
-			this.trigger('dataClearedCompleted', { todos: todos });
+
+			this.trigger('uiFilterRequested', { filter: filter });
+			this.trigger('dataClearedCompleted');
 		};
 
 		this.after('initialize', function () {

diff --git a/dependency-examples/flight/app/js/ui/todo_list.js b/dependency-examples/flight/app/js/ui/todo_list.js
@@ -86,7 +86,6 @@ define([
 			this.on(document, 'dataTodoAdded', this.render);
 			this.on(document, 'dataTodosLoaded', this.renderAll);
 			this.on(document, 'dataTodosFiltered', this.renderAll);
-			this.on(document, 'dataClearedCompleted', this.renderAll);
 			this.on(document, 'dataTodoToggledAll', this.renderAll);
 			this.on(document, 'dataTodoRemoved', this.remove);