From 0a17525eeb101d538d334c2208eb8ee44cda1883 Mon Sep 17 00:00:00 2001
From: Benjamin Auder <benjamin.auder@somewhere>
Date: Fri, 6 Mar 2020 18:37:04 +0100
Subject: [PATCH] Fix isConnected indicator in case of multi-accounts on same
 browser

---
 client/src/views/Game.vue | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

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 (
         (
-- 
2.44.0