Bodge to default to '1 users' when you create a room, which is better than blindly assuming a recents controller is writing to rootScope.rooms and setting numUsersInRoom there.

1 files changed, 5 insertions, 0 deletions

diff --git a/webclient/recents/recents-controller.js b/webclient/recents/recents-controller.js
index b4762acd1d..701c935742 100644
--- a/webclient/recents/recents-controller.js
+++ b/webclient/recents/recents-controller.js
@@ -82,6 +82,11 @@ angular.module('RecentsController', ['matrixService', 'matrixFilter', 'eventHand
         eventHandlerService.waitForInitialSyncCompletion().then(
             function(initialSyncData) {
             
+                // XXX FIXME TODO:
+                // Any assignments to the rootScope here should be done in
+                // event handler service and not here, because we could have
+                // many controllers manipulating and clobbering each other, and
+                // are unecessarily repeating http requests.
                 var rooms = initialSyncData.data.rooms;
                 for (var i=0; i<rooms.length; i++) {
                     var room = rooms[i];