removed redundant logic checks

When players disconnect from the game socket, the player will be departed from the game's players list. In addition, when the last player departs, the game is removed from the gamesList.

Now while waiting in the lobby, it no longer polls every 5 seconds and reacts to socket events instead. Games list will update when new games are added and when players join existing games.

When gameId or playerId is passed as an array instead of a value to the game module, bad things sometimes happen.

When responding to the connectToGame event, the socketIO packet parses the gameId and playerId values as single member arrays. When gameId is passsed to getGame as an array, I was seeing a crash in gameViewModel because the return value was undefined and was being passed to JSON.stringify.

jshint issues fixed