Merge pull request tastejs#545 from thebakeryio/feature/backbone-mari…

…onette-model-shortcuts use Marionette's event hashes for Model and Collection events
passy · May 5, 2013 · 41ef0d4 · 41ef0d4
2 parents 37174a6 + dccfdac
commit 41ef0d4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/labs/architecture-examples/backbone_marionette/js/TodoMVC.Layout.js b/labs/architecture-examples/backbone_marionette/js/TodoMVC.Layout.js
@@ -48,9 +48,12 @@ TodoMVC.module('Layout', function (Layout, App, Backbone) {
 			'click #clear-completed': 'onClearClick'
 		},
 
+		collectionEvents: {
+			'all': 'updateCount'
+		},
+
 		initialize: function () {
 			this.listenTo(App.vent, 'todoList:filter', this.updateFilterSelection, this);
-			this.listenTo(this.collection, 'all', this.updateCount, this);
 		},
 
 		onRender: function () {

diff --git a/labs/architecture-examples/backbone_marionette/js/TodoMVC.TodoList.Views.js b/labs/architecture-examples/backbone_marionette/js/TodoMVC.TodoList.Views.js
@@ -23,8 +23,8 @@ TodoMVC.module('TodoList.Views', function (Views, App, Backbone, Marionette, $)
 			'click .toggle': 'toggle'
 		},
 
-		initialize: function () {
-			this.listenTo(this.model, 'change', this.render, this);
+		modelEvents: {
+			'change': 'render'
 		},
 
 		onRender: function () {
@@ -93,8 +93,8 @@ TodoMVC.module('TodoList.Views', function (Views, App, Backbone, Marionette, $)
 			'click #toggle-all': 'onToggleAllClick'
 		},
 
-		initialize: function () {
-			this.listenTo(this.collection, 'all', this.update, this);
+		collectionEvents: {
+			'all' : 'update'
 		},
 
 		onRender: function () {