From 5d86ef1e1d9ffd80ddba54c0ee0e152badaf1734 Mon Sep 17 00:00:00 2001
From: winkidney <winkidney@gmail.com>
Date: Fri, 29 Nov 2019 17:25:37 +0800
Subject: [PATCH] Feature: Add login-form with error-handling

---
 pinry-spa/src/components/LoginForm.vue | 97 ++++++++++++++++++++++++++
 pinry-spa/src/components/PHeader.vue   | 15 ++++
 pinry-spa/src/components/api.js        |  2 +-
 3 files changed, 113 insertions(+), 1 deletion(-)
 create mode 100644 pinry-spa/src/components/LoginForm.vue

diff --git a/pinry-spa/src/components/LoginForm.vue b/pinry-spa/src/components/LoginForm.vue
new file mode 100644
index 0000000..456196e
--- /dev/null
+++ b/pinry-spa/src/components/LoginForm.vue
@@ -0,0 +1,97 @@
+<template>
+  <div class="login-modal">
+    <form action="">
+      <div class="modal-card" style="width: auto">
+        <header class="modal-card-head">
+          <p class="modal-card-title">Login</p>
+        </header>
+        <section class="modal-card-body">
+          <b-field label="Username"
+                   :type="username.type"
+                   :message="username.error">
+            <b-input
+              type="string"
+              v-model="username.value"
+              placeholder="Your Username"
+              maxlength="30"
+              required>
+            </b-input>
+          </b-field>
+
+          <b-field label="Password"
+                   :type="password.type"
+                   :message="password.error">
+            <b-input
+              type="password"
+              v-model="password.value"
+              password-reveal
+              placeholder="Your password"
+              required>
+            </b-input>
+          </b-field>
+        </section>
+        <footer class="modal-card-foot">
+          <button class="button" type="button" @click="$parent.close()">Close</button>
+          <button
+            @click="doLogin"
+            class="button is-primary">Login</button>
+        </footer>
+      </div>
+    </form>
+  </div>
+</template>
+
+<script>
+import api from './api';
+
+export default {
+  name: 'LoginForm',
+  data() {
+    return {
+      username: {
+        value: null,
+        error: null,
+        type: null,
+      },
+      password: {
+        value: null,
+        error: null,
+        type: null,
+      },
+    };
+  },
+  methods: {
+    resetStatus() {
+      this.resetField('username');
+      this.resetField('password');
+    },
+    resetField(fieldName) {
+      this[fieldName].type = 'is-info';
+      this[fieldName].error = null;
+    },
+    markFieldAsDanger(fieldName, errorMsg) {
+      this[fieldName].error = errorMsg;
+      this[fieldName].type = 'is-danger';
+    },
+    doLogin() {
+      this.resetStatus();
+      const self = this;
+      const promise = api.User.logIn(self.username.value, self.password.value);
+      promise.then(
+        (user) => {
+          self.$emit('login.succeed', user);
+          self.$parent.close();
+        },
+        (resp) => {
+          if (resp.data.username) {
+            self.markFieldAsDanger('username', resp.data.username);
+          }
+          if (resp.data.password) {
+            self.markFieldAsDanger('password', resp.data.password);
+          }
+        },
+      );
+    },
+  },
+};
+</script>
diff --git a/pinry-spa/src/components/PHeader.vue b/pinry-spa/src/components/PHeader.vue
index f90e4c9..44e9644 100644
--- a/pinry-spa/src/components/PHeader.vue
+++ b/pinry-spa/src/components/PHeader.vue
@@ -66,6 +66,7 @@
                 </a>
                 <a
                   v-show="!user.loggedIn"
+                  v-on:click="logIn"
                   class="button is-light">
                   Log in
                 </a>
@@ -86,6 +87,7 @@
 
 <script>
 import api from './api';
+import LoginForm from './LoginForm.vue';
 
 export default {
   name: 'p-header',
@@ -102,6 +104,9 @@ export default {
     toggleMenu() {
       this.active = !this.active;
     },
+    onLoginSucceed() {
+      this.initializeUser();
+    },
     logOut() {
       api.User.logOut().then(
         () => {
@@ -109,6 +114,16 @@ export default {
         },
       );
     },
+    logIn() {
+      this.$buefy.modal.open({
+        parent: this,
+        component: LoginForm,
+        hasModalCard: true,
+        events: {
+          'login.succeed': this.onLoginSucceed,
+        },
+      });
+    },
     initializeUser() {
       const self = this;
       api.User.fetchUserInfo().then(
diff --git a/pinry-spa/src/components/api.js b/pinry-spa/src/components/api.js
index 6f17b97..a262d2a 100644
--- a/pinry-spa/src/components/api.js
+++ b/pinry-spa/src/components/api.js
@@ -55,7 +55,7 @@ const User = {
           },
           (error) => {
             console.log('Failed to log in due to unexpected error:', error);
-            reject(error);
+            reject(error.response);
           },
         );
       },