Cosmetics
authorBenjamin Auder <benjamin.auder@somewhere>
Fri, 28 Jun 2019 16:50:32 +0000 (18:50 +0200)
committerBenjamin Auder <benjamin.auder@somewhere>
Fri, 28 Jun 2019 16:50:32 +0000 (18:50 +0200)
client/src/views/Game.vue
server/sockets.js

index 7d20dbe..ea5ea36 100644 (file)
@@ -303,8 +303,7 @@ export default {
         if (!!this.game.oppid)
         {
           // Send ping to server (answer pong if players[s] are connected)
         if (!!this.game.oppid)
         {
           // Send ping to server (answer pong if players[s] are connected)
-          this.st.conn.send(JSON.stringify({code:"ping",
-            target:this.game.oppid, gameId:this.gameRef.id}));
+          this.st.conn.send(JSON.stringify({code:"ping", target:this.game.oppid}));
         }
       };
       if (!!game)
         }
       };
       if (!!game)
index 09434a0..a3f4649 100644 (file)
@@ -83,14 +83,13 @@ module.exports = function(wss) {
         case "newchat":
           clients[obj.target].send(JSON.stringify({code:"newchat",msg:obj.msg}));
           break;
         case "newchat":
           clients[obj.target].send(JSON.stringify({code:"newchat",msg:obj.msg}));
           break;
-        // Transmit chats and moves to current room
         // TODO: WebRTC instead in this case (most demanding?)
         case "newmove":
           clients[obj.target].send(JSON.stringify({code:"newmove",move:obj.move}));
           break;
         // TODO: WebRTC instead in this case (most demanding?)
         case "newmove":
           clients[obj.target].send(JSON.stringify({code:"newmove",move:obj.move}));
           break;
-        // TODO: generalize that for several opponents
         case "ping":
         case "ping":
-          socket.send(JSON.stringify({code:"pong",gameId:obj.gameId}));
+          // If this code is reached, then obj.target is connected
+          socket.send(JSON.stringify({code:"pong"}));
           break;
         case "lastate":
           const oppId = obj.target;
           break;
         case "lastate":
           const oppId = obj.target;