we appear to be back up to speed

vincentwoo · Apr 13, 2012 · a11f49e · a11f49e
1 parent d768450
commit a11f49e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 9 deletions.
diff --git a/client/client.js b/client/client.js
@@ -6,7 +6,7 @@ var socket
   , lastMsg
   , preventRefresh = false
   , fixwrap
-  , SERVER_EVENTS = ['init', 'join', 'rejoin', 'taken', 'setHash', 'remaining', 'puzzled',
+  , SERVER_EVENTS = ['init', 'join', 'leave', 'rejoin', 'taken', 'setHash', 'remaining', 'puzzled',
       'add', 'hint', 'start', 'win', 'msg'];
 
 $(function() {
@@ -40,7 +40,7 @@ function startGame() {
   socket.on('reconnecting', socket_reconnect);
   socket.on('reconnect_failed', socket_reconnect_failed);
 
-  $('#hint').click(hint);
+  $('#hint').click(requestHint);
   $('#input').keydown(input);
   $('#input').focus();
 
@@ -199,7 +199,7 @@ function fadeOutAllLastSets() {
   }
 }
 
-function hint(event) {
+function requestHint(event) {
   socket.emit('hint');
   $('#hint').animate({opacity:0});
   showPuzzled(me);
@@ -362,7 +362,7 @@ function setHash(hash) {
 
 function leave(player) {
   var update = {};
-  update[obj.player] = {online: false};
+  update[player] = {online: false};
   updatePlayers(update);
 }
 

diff --git a/game.js b/game.js
@@ -80,6 +80,11 @@ Game.prototype.registerClient = function(socket, sess) {
   if (this.numPlayers() >= this.players.length) return false;
   var self = this;
 
+  //socket.join(this.hash);
+  CLIENT_EVENTS.forEach(function(event) {
+    socket.on(event, self.handleClientMessage(event, socket));
+  });
+
   for (var i = 0; i < this.players.length; i++) {
     var player = this.players[i];
     if (player === null) continue;
@@ -95,10 +100,6 @@ Game.prototype.registerClient = function(socket, sess) {
       return true;
     }
   }
-  //socket.join(this.hash);
-  CLIENT_EVENTS.forEach(function(event) {
-    socket.on(event, self.handleClientMessage(event, socket));
-  });
 
   var playerIdx = this.firstAvailablePlayerSlot();
   this.broadcast('join', playerIdx);
@@ -147,7 +148,7 @@ Game.prototype.updateRemaining = function() {
 }
 
 Game.prototype.broadcast = function(event, message) {
-  console.log(this.hash + ' broadcasting: ');
+  console.log(this.hash + ' broadcasting event ' + event + ': ');
   console.log(message);
   this.players.forEach( function(player) {
     if (player !== null) player.socket.emit(event, message);