From 7cb3e246a88b83f6c1d87fe47689acf5e0c9eaac Mon Sep 17 00:00:00 2001
From: Pig Fang <g-plane@hotmail.com>
Date: Tue, 25 Jun 2019 22:59:05 +0800
Subject: [PATCH] Fix possible duplicated signings (fix #68)

---
 resources/assets/src/views/user/Dashboard.vue       | 4 ++++
 resources/assets/tests/views/user/Dashboard.test.ts | 9 +++++++++
 resources/misc/changelogs/en/4.3.0.md               | 4 ++++
 resources/misc/changelogs/zh_CN/4.3.0.md            | 4 ++++
 4 files changed, 21 insertions(+)

diff --git a/resources/assets/src/views/user/Dashboard.vue b/resources/assets/src/views/user/Dashboard.vue
index b6fa2ac8..2e4d9330 100644
--- a/resources/assets/src/views/user/Dashboard.vue
+++ b/resources/assets/src/views/user/Dashboard.vue
@@ -73,6 +73,7 @@
           class="btn btn-primary pull-left"
           type="primary"
           round
+          :disabled="signing"
           @click="sign"
         >
           <i class="far fa-calendar-check" aria-hidden="true" /> &nbsp;{{ $t('user.sign') }}
@@ -114,6 +115,7 @@ export default {
     lastSignAt: new Date(),
     signAfterZero: false,
     signGap: 0,
+    signing: false,
     playersUsed: 0,
     playersTotal: 1,
     storageUsed: 0,
@@ -186,6 +188,7 @@ export default {
       this.score = data.user.score
     },
     async sign() {
+      this.signing = true
       const {
         code, message, data,
       } = await this.$http.post('/user/sign')
@@ -199,6 +202,7 @@ export default {
       } else {
         this.$message.warning(message)
       }
+      this.signing = false
     },
   },
 }
diff --git a/resources/assets/tests/views/user/Dashboard.test.ts b/resources/assets/tests/views/user/Dashboard.test.ts
index 6c841030..27e53d30 100644
--- a/resources/assets/tests/views/user/Dashboard.test.ts
+++ b/resources/assets/tests/views/user/Dashboard.test.ts
@@ -166,3 +166,12 @@ test('sign', async () => {
   expect(button.attributes('disabled')).toBe('disabled')
   expect(wrapper.text()).toContain('3 / 4 KB')
 })
+
+test('disable button when signing', () => {
+  Vue.prototype.$http.get.mockResolvedValue(scoreInfo())
+  const wrapper = mount(Dashboard)
+  const button = wrapper.find(Button)
+  expect(button.attributes('disabled')).toBeFalsy()
+  wrapper.setData({ signing: true })
+  expect(button.attributes('disabled')).toBeTruthy()
+})
diff --git a/resources/misc/changelogs/en/4.3.0.md b/resources/misc/changelogs/en/4.3.0.md
index 2f6755f5..e9bab7b6 100644
--- a/resources/misc/changelogs/en/4.3.0.md
+++ b/resources/misc/changelogs/en/4.3.0.md
@@ -7,3 +7,7 @@
 
 - Tweaked style of chart at administration panel.
 - Optimized panel of changing theme color.
+
+## Fixed
+
+- Fixed that possible duplicated signings.
diff --git a/resources/misc/changelogs/zh_CN/4.3.0.md b/resources/misc/changelogs/zh_CN/4.3.0.md
index 23651add..f6f2c298 100644
--- a/resources/misc/changelogs/zh_CN/4.3.0.md
+++ b/resources/misc/changelogs/zh_CN/4.3.0.md
@@ -7,3 +7,7 @@
 
 - 调整管理面板中的图表样式
 - 优化「更改配色」的面板
+
+## 修复
+
+- 修复可能存在的重复签到问题