Merge pull request #777 from daleharvey/776

Close GH-776: Make sure updateCount works with async Stores

vanilla-examples/vanillajs/js/controller.js

@@ -199,16 +199,16 @@
 	 * number of todos.
 	 */
 	Controller.prototype._updateCount = function () {
-		var todos = this.model.getCount();
-
-		this.view.render('updateElementCount', todos.active);
-		this.view.render('clearCompletedButton', {
-			completed: todos.completed,
-			visible: todos.completed > 0
-		});
+		this.model.getCount((function(todos) {
+			this.view.render('updateElementCount', todos.active);
+			this.view.render('clearCompletedButton', {
+				completed: todos.completed,
+				visible: todos.completed > 0
+			});
 
-		this.view.render('toggleAll', {checked: todos.completed === todos.total});
-		this.view.render('contentBlockVisibility', {visible: todos.total > 0});
+			this.view.render('toggleAll', {checked: todos.completed === todos.total});
+			this.view.render('contentBlockVisibility', {visible: todos.total > 0});
+		}).bind(this);
 	};
 
 	/**

vanilla-examples/vanillajs/js/model.js

@@ -93,7 +93,7 @@
 	/**
 	 * Returns a count of all todos
 	 */
-	Model.prototype.getCount = function () {
+	Model.prototype.getCount = function (callback) {
 		var todos = {
 			active: 0,
 			completed: 0,
@@ -110,9 +110,8 @@
 
 				todos.total++;
 			});
+			callback(todos);
 		});
-
-		return todos;
 	};
 
 	// Export to window