From 424243d2ab05055bff32d9255c9c8d5c357ea8b1 Mon Sep 17 00:00:00 2001
From: Michael Rienstra <mrienstra@gmail.com>
Date: Mon, 20 Feb 2012 15:48:56 -0800
Subject: [PATCH] Retrieving room count in a better way (using Group count
 method instead of counting the Group users property). Added notification of
 leaving and entering -- though this is a bit crude, may be better to leave it
 out, rather than show it being done somewhat poorly.

---
 examples/multiroomchat_example/multiroomchat.html      | 2 +-
 examples/multiroomchat_example/multiroomchat_server.js | 9 +++++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/examples/multiroomchat_example/multiroomchat.html b/examples/multiroomchat_example/multiroomchat.html
index 9e6e2ee..dc61388 100644
--- a/examples/multiroomchat_example/multiroomchat.html
+++ b/examples/multiroomchat_example/multiroomchat.html
@@ -8,7 +8,7 @@
 <script>
 $(document).ready(function(){
   now.receiveMessage = function(name, message){
-    $("#messages").append("<br>" + name + ": " + message);
+    $("#messages").append("<br><b>" + name + "</b>: " + message);
   }
   
   $("#send-button").click(function(){
diff --git a/examples/multiroomchat_example/multiroomchat_server.js b/examples/multiroomchat_example/multiroomchat_server.js
index 87f275e..b2209ed 100644
--- a/examples/multiroomchat_example/multiroomchat_server.js
+++ b/examples/multiroomchat_example/multiroomchat_server.js
@@ -25,11 +25,16 @@ nowjs.on('disconnect', function(){
 });
 
 everyone.now.changeRoom = function(newRoom){
+  this.now.distributeMessage("[leaving " + this.now.room + "]");
   nowjs.getGroup(this.now.room).removeUser(this.user.clientId);
   nowjs.getGroup(newRoom).addUser(this.user.clientId);
   this.now.room = newRoom;
-  var others = Object.keys(nowjs.getGroup(this.now.room).users).length - 1;
-  this.now.receiveMessage("SERVER", "You're now in " + this.now.room + " (" + others + " other(s) in this room).");
+  this.now.distributeMessage("[entering " + this.now.room + "]");
+  var that = this;
+  nowjs.getGroup(this.now.room).count(function(count){
+    var prettyCount = (count === 1) ? "Room is empty." : (count - 1) + " other(s) in room.";
+    that.now.receiveMessage("SERVER", "You're now in " + that.now.room + ". " + prettyCount);
+  });
 }
 
 everyone.now.distributeMessage = function(message){