X-Git-Url: https://git.auder.net/?a=blobdiff_plain;f=client%2Fsrc%2Fcomponents%2FUpsertUser.vue;h=ecd51fa916e80bfd7e68f3145e64e2f8a2b10358;hb=dac395887d96e2d642b209c6db6aaacc3ffacb34;hp=93c1fd4b8a9dcad94ce2107c854161062d6e32e8;hpb=3837d4f7885a3c3cdb468da2f3fa3fa1e5a1415a;p=vchess.git

diff --git a/client/src/components/UpsertUser.vue b/client/src/components/UpsertUser.vue
index 93c1fd4b..ecd51fa9 100644
--- a/client/src/components/UpsertUser.vue
+++ b/client/src/components/UpsertUser.vue
@@ -1,7 +1,7 @@
 <template lang="pug">
 div
   input#modalUser.modal(type="checkbox" @change="trySetEnterTime")
-  div(role="dialog")
+  div(role="dialog" data-checkbox="modalUser")
     .card
       label.modal-close(for="modalUser")
       h3 {{ stage }}
@@ -26,7 +26,7 @@ div
           i.material-icons send
         button(v-if="stage!='Update'" @click="toggleStage()")
           span {{ stage=="Login" ? "Register" : "Login" }}
-        button(v-else @click="doLogout()")
+        button#logoutBtn(v-else @click="doLogout()")
           span Logout
       #dialog(:style="{display: displayInfo}") {{ infoMsg }}
 </template>
@@ -46,20 +46,20 @@ export default {
       enterTime: Number.MAX_SAFE_INTEGER, //for a basic anti-bot strategy
     };
   },
-	watch: {
-		nameOrEmail: function(newValue) {
-			if (newValue.indexOf('@') >= 0)
-			{
-				this.user.email = newValue;
-				this.user.name = "";
-			}
-			else
-			{
-				this.user.name = newValue;
-				this.user.email = "";
-			}
-		},
-	},
+  watch: {
+    nameOrEmail: function(newValue) {
+      if (newValue.indexOf('@') >= 0)
+      {
+        this.user.email = newValue;
+        this.user.name = "";
+      }
+      else
+      {
+        this.user.name = newValue;
+        this.user.email = "";
+      }
+    },
+  },
   computed: {
     submitMessage: function() {
       switch (this.stage)
@@ -155,18 +155,24 @@ export default {
       );
     },
     doLogout: function() {
-      ajax(
-        "/logout",
-        "GET",
-        () => {
-          this.user.id = 0;
-          this.user.name = "";
-          this.user.email = "";
-          this.user.notify = false;
-          delete localStorage["myid"];
-          delete localStorage["myname"];
-        }
-      );
+      let logoutBtn = document.getElementById("logoutBtn");
+      logoutBtn.disabled = true;
+      // NOTE: this local cleaning would logically happen when we're sure
+      // that token is erased. But in the case a user clear the cookies,
+      // it would lead to situations where he cannot ("locally") log out.
+      // At worst, if token deletion fails the user can erase cookie manually.
+      this.user.id = 0;
+      this.user.name = "";
+      this.user.email = "";
+      this.user.notify = false;
+      localStorage.removeItem("myid");
+      localStorage.removeItem("myname");
+      ajax("/logout", "GET", () => {
+        logoutBtn.disabled = false; //for symmetry, but not very useful...
+        document.getElementById("modalUser").checked = false;
+        // this.$router.push("/") will fail if logout from Hall, so:
+        document.location.reload(true);
+      });
     },
   },
 };