From ed138a6d9ae126eb4679352059b58fa8eb538b05 Mon Sep 17 00:00:00 2001
From: Bubka <858858+Bubka@users.noreply.github.com>
Date: Fri, 28 Jun 2019 01:16:51 +0200
Subject: [PATCH] OTP generation in 2FAccount modal with dots counter

---
 resources/js/components/ModalTwofaccount.vue | 26 ++++++
 resources/js/components/OneTimePassword.vue  | 64 ++++++++++++++
 resources/js/views/Accounts.vue              | 91 ++++++++++----------
 resources/sass/app.scss                      |  4 +-
 4 files changed, 137 insertions(+), 48 deletions(-)
 create mode 100644 resources/js/components/ModalTwofaccount.vue
 create mode 100644 resources/js/components/OneTimePassword.vue

diff --git a/resources/js/components/ModalTwofaccount.vue b/resources/js/components/ModalTwofaccount.vue
new file mode 100644
index 00000000..7e989816
--- /dev/null
+++ b/resources/js/components/ModalTwofaccount.vue
@@ -0,0 +1,26 @@
+<template>
+    <section class="section">
+        <div class="columns is-centered">
+            <div class="column is-three-quarters">
+                <div class="box has-text-centered has-background-black-ter ">
+                    <figure class="image is-64x64" style="display: inline-block">
+                        <img :src="icon">
+                    </figure>
+                    <p class="is-size-4 has-text-grey-light">{{ name }}</p>
+                    <p class="is-size-6 has-text-grey">{{ email }}</p>
+                    <slot></slot>
+                </div>
+            </div>
+        </div>
+    </section>
+</template>
+
+<script>
+    export default {
+        data() {
+            return {
+            }
+        },
+        props: ['name', 'email', 'icon'],
+    }
+</script>
\ No newline at end of file
diff --git a/resources/js/components/OneTimePassword.vue b/resources/js/components/OneTimePassword.vue
new file mode 100644
index 00000000..f551bcc6
--- /dev/null
+++ b/resources/js/components/OneTimePassword.vue
@@ -0,0 +1,64 @@
+<template>
+    <div>
+        <p id="otp" title="refresh" class="is-size-1 has-text-white">{{ totp }}</p>
+        <ul class="dots">
+            <li data-is-active style="display:none"></li><li></li><li></li><li></li><li></li><li></li><li></li><li></li><li></li><li></li>
+        </ul>
+    </div>
+</template>
+
+<script>
+    export default {
+        data() {
+            return {
+                totp : '',
+                componentKey: 0
+            }
+        },
+        props: ['AccountId'],
+        watch: {
+            AccountId: function(newVal, oldVal) {
+                this.getOTP()
+            }
+        },
+        methods: {
+            getOTP: function () {
+                let token = localStorage.getItem('jwt')
+
+                axios.defaults.headers.common['Content-Type'] = 'application/json'
+                axios.defaults.headers.common['Authorization'] = 'Bearer ' + token
+
+                axios.get('api/twofaccounts/' + this.$props.AccountId + '/totp').then(response => {
+                    this.totp = response.data.totp.substr(0, 3) + " " + response.data.totp.substr(3);
+                })
+
+                var self = this;
+                var timer = setInterval(function() {
+                    let dots = self.$el.querySelector('.dots');
+                    let active = dots.querySelector('[data-is-active]');
+                    let lastdot = dots.querySelector('li:last-child');
+
+                    if (active === null) {
+                        self.$el.querySelector('.dots li:first-child').setAttribute('data-is-active', true);
+                        active = dots.querySelector('[data-is-active]');
+                    }
+                    else if(active === lastdot) {
+                        clearInterval(timer);
+                        active.removeAttribute('data-is-active');
+                        self.$el.querySelector('.dots li:first-child').setAttribute('data-is-active', true);
+                        self.getOTP();
+                    }
+                    else
+                    {
+                        let sibling = active.nextSibling;
+                        if (sibling === null) return;
+
+                        active.removeAttribute('data-is-active');
+                        sibling.setAttribute('data-is-active', true);
+                    }
+                },3000);
+
+            }
+        }
+    }
+</script>
\ No newline at end of file
diff --git a/resources/js/views/Accounts.vue b/resources/js/views/Accounts.vue
index 867e4ad3..64fe14de 100644
--- a/resources/js/views/Accounts.vue
+++ b/resources/js/views/Accounts.vue
@@ -2,7 +2,7 @@
     <div>
         <div class="container">
             <div class="buttons are-large is-centered">
-                <span v-for="account in accounts" class="button is-black twofaccount" @click.stop="getOTP(account.id)">
+                <span v-for="account in accounts" class="button is-black twofaccount" @click.stop="getAccount(account.id)">
                     <img src="https://fakeimg.pl/64x64/">
                     {{ account.name }}
                     <span class="is-family-primary is-size-7 has-text-grey">{{ account.email }}</span>
@@ -10,7 +10,12 @@
             </div>
         </div>
         <modal v-model="ModalIsActive">
-            <modal-twofaccount v-bind="twofaccount"></modal-twofaccount>
+            <modal-twofaccount
+                :name='twofaccount.name'
+                :icon='twofaccount.icon'
+                :email='twofaccount.email'>
+                <one-time-password :AccountId='twofaccount.id' ></one-time-password>
+            </modal-twofaccount>
         </modal>
     </div>
 </template>
@@ -20,40 +25,35 @@
 
 <script>
     import Modal from '../components/Modal'
+    import ModalTwofaccount from '../components/ModalTwofaccount'
+    import OneTimePassword from '../components/OneTimePassword'
+
+    // const ModalTwofaccount = {
+    //     props: ['id', 'name', 'email', 'icon', 'totp'],
+    //     template: `
+    //         <section class="section">
+    //             <div class="columns is-centered">
+    //                 <div class="column is-three-quarters">
+    //                     <div class="box has-text-centered has-background-black-ter ">
+    //                         <figure class="image is-64x64" style="display: inline-block">
+    //                             <img :src="icon">
+    //                         </figure>
+    //                         <p class="is-size-4 has-text-grey-light">{{ name }}</p>
+    //                         <p class="is-size-6 has-text-grey">{{ email }}</p>
+    //                         <p id="otp" title="refresh" class="is-size-1 has-text-white">{{ totp }}</p>
+    //                         <ul class="dots">
+    //                             <li data-is-active style="display:none"></li><li></li><li></li><li></li><li></li><li></li><li></li><li></li><li></li><li></li>
+    //                         </ul>
+    //                     </div>
+    //                 </div>
+    //             </div>
+    //         </section>
+    //     `
+    // }
+
 
-    const ModalTwofaccount = {
-        props: ['id', 'name', 'email', 'uri', 'icon', 'totp'],
-        template: `
-            <section class="section">
-                <div class="columns is-centered">
-                    <div class="column is-three-quarters">
-                        <div class="box has-text-centered has-background-black-ter ">
-                            <figure class="image is-64x64" style="display: inline-block">
-                                <img :src="icon">
-                            </figure>
-                            <p class="is-size-4 has-text-grey-light">{{ name }}</p>
-                            <p class="is-size-6 has-text-grey">{{ email }}</p>
-                            <p id="otp" title="refresh" class="is-size-1 has-text-white">{{ totp }}</p>
-                            <ul class="dots">
-                                <li></li>
-                                <li></li>
-                                <li></li>
-                                <li></li>
-                                <li></li>
-                                <li></li>
-                                <li></li>
-                                <li></li>
-                                <li data-is-active></li>
-                            </ul>
-                        </div>
-                    </div>
-                </div>
-            </section>
-        `
-    }
 
     export default {
-        name: 'Accounts',
         data(){
             return {
                 accounts : [],
@@ -73,7 +73,6 @@
                         id : data.id,
                         name : data.name,
                         email : data.email,
-                        uri : data.uri,
                         icon : data.icon
                     })
                 })
@@ -82,27 +81,26 @@
         },
         components: {
             Modal,
-            ModalTwofaccount
+            ModalTwofaccount,
+            OneTimePassword
         },
         methods: {
-            getOTP: function (id) {
+            getAccount: function (id) {
                 let token = localStorage.getItem('jwt')
-                let twofa
+                let accountId = id
 
                 axios.defaults.headers.common['Content-Type'] = 'application/json'
                 axios.defaults.headers.common['Authorization'] = 'Bearer ' + token
 
                 axios.get('api/twofaccounts/' + id).then(response => {
-                    let res1 = response.data
-                    axios.get('api/twofaccounts/' + id + '/totp').then(response => {
-                        this.twofaccount = Object.assign(res1, response.data)
-                        this.twofaccount.totp = this.twofaccount.totp.substr(0, 3) + " " + this.twofaccount.totp.substr(3);
-                    })
+                    this.twofaccount.id = response.data.id
+                    this.twofaccount.name = response.data.name
+                    this.twofaccount.email = response.data.email
+                    this.twofaccount.icon = response.data.icon
+
+                    this.ModalIsActive = true;
+
                 })
-
-
-                console.log(this.twofaccount)
-                this.ModalIsActive = true
             }
         },
         beforeRouteEnter (to, from, next) {
@@ -113,5 +111,6 @@
             next()
         }
     }
+
 </script>
 
diff --git a/resources/sass/app.scss b/resources/sass/app.scss
index ea3556fd..ded32519 100644
--- a/resources/sass/app.scss
+++ b/resources/sass/app.scss
@@ -52,10 +52,10 @@ nav.level {
   background: hsl(0, 0%, 7%);  /* grey */
 }
 
-.dots li:nth-child(-n+8) {
+.dots li:nth-child(-n+9) {
   background: hsl(48, 100%, 67%); /* yellow */
 }
 
-.dots li:nth-child(-n+6) {
+.dots li:nth-child(-n+7) {
   background: hsl(141, 71%, 48%); /* green */
 }
\ No newline at end of file