From: Benjamin Auder <benjamin.auder@somewhere>
Date: Fri, 6 Mar 2020 17:37:04 +0000 (+0100)
Subject: Fix isConnected indicator in case of multi-accounts on same browser
X-Git-Url: https://git.auder.net/doc/%7B%7B%20asset%28%27mixstore/current/img/mini-custom.min.css?a=commitdiff_plain;h=0a17525eeb101d538d334c2208eb8ee44cda1883;p=vchess.git

Fix isConnected indicator in case of multi-accounts on same browser
---

diff --git a/client/src/views/Game.vue b/client/src/views/Game.vue
index 214f8196..df6930cb 100644
--- a/client/src/views/Game.vue
+++ b/client/src/views/Game.vue
@@ -232,12 +232,10 @@ export default {
     isConnected: function(index) {
       const player = this.game.players[index];
       // Is it me ?
-      if (
-        (this.st.user.sid == player.sid || this.st.user.id == player.uid) &&
-        (!this.st.user.name || this.st.user.name == player.name)
-      ) {
-        return true;
-      }
+      if (this.st.user.sid == player.sid || this.st.user.id == player.uid)
+        // Still have to check for name (because of potential multi-accounts
+        // on same browser, although this should be rare...)
+        return (!this.st.user.name || this.st.user.name == player.name);
       // Try to find a match in people:
       return (
         (