Backbone apps - simplify clearCompleted logic

tastejs · Jan 6, 2013 · 7f17ef8 · 7f17ef8
1 parent 8e20c96
commit 7f17ef8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/architecture-examples/backbone/js/views/app.js b/architecture-examples/backbone/js/views/app.js
@@ -111,10 +111,7 @@ $(function( $ ) {
 
 		// Clear all completed todo items, destroying their models.
 		clearCompleted: function() {
-			_.each( app.Todos.completed(), function( todo ) {
-				todo.destroy();
-			});
-
+			_.invoke(app.Todos.completed(), 'destroy');
 			return false;
 		},
 

diff --git a/dependency-examples/backbone_require/js/views/app.js b/dependency-examples/backbone_require/js/views/app.js
@@ -112,10 +112,7 @@ define([
 
 		// Clear all completed todo items, destroying their models.
 		clearCompleted: function() {
-			_.each( Todos.completed(), function( todo ) {
-				todo.destroy();
-			});
-
+			_.invoke(Todos.completed(), 'destroy');
 			return false;
 		},