From 5239c70d6ff1d57df16ee85e82ce5a734be8a80e Mon Sep 17 00:00:00 2001
From: Hironsan <light.tree.1.13@gmail.com>
Date: Mon, 28 Oct 2019 11:32:09 +0900
Subject: [PATCH] Move statistics logic to statistics store

---
 .../pages/projects/_id/statistics/index.vue   | 56 ++++-----------
 frontend/services/statistics.service.js       |  2 +-
 frontend/store/statistics.js                  | 68 +++++++++++++++++++
 3 files changed, 84 insertions(+), 42 deletions(-)
 create mode 100644 frontend/store/statistics.js

diff --git a/frontend/pages/projects/_id/statistics/index.vue b/frontend/pages/projects/_id/statistics/index.vue
index 9f011ff7..e4ebef25 100644
--- a/frontend/pages/projects/_id/statistics/index.vue
+++ b/frontend/pages/projects/_id/statistics/index.vue
@@ -1,5 +1,8 @@
 <template>
-  <v-container fluid>
+  <v-container
+    v-if="stats"
+    fluid
+  >
     <v-row>
       <v-col
         cols="12"
@@ -7,7 +10,7 @@
       >
         <v-card>
           <doughnut-chart
-            :chart-data="progressStat"
+            :chart-data="progress"
           />
         </v-card>
       </v-col>
@@ -17,7 +20,7 @@
       >
         <v-card>
           <bar-chart
-            :chart-data="labelStat"
+            :chart-data="labelStats"
           />
         </v-card>
       </v-col>
@@ -27,7 +30,7 @@
       >
         <v-card>
           <bar-chart
-            :chart-data="userStat"
+            :chart-data="userStats"
           />
         </v-card>
       </v-col>
@@ -36,9 +39,9 @@
 </template>
 
 <script>
+import { mapActions, mapGetters, mapState } from 'vuex'
 import DoughnutChart from '@/components/molecules/DoughnutChart'
 import BarChart from '@/components/molecules/BarChart'
-import StatisticsService from '@/services/statistics.service'
 
 export default {
   layout: 'project',
@@ -50,48 +53,19 @@ export default {
     BarChart
   },
 
-  data() {
-    return {
-      progressStat: {},
-      userStat: {},
-      labelStat: {}
-    }
+  computed: {
+    ...mapGetters('statistics', ['userStats', 'labelStats', 'progress']),
+    ...mapState('statistics', ['stats'])
   },
 
-  created() {
-    StatisticsService.getStatistics(this.$route.params.id).then((response) => {
-      this.labelStat = this.makeData(response.label, 'Label stats')
-      this.userStat = this.makeData(response.user, 'User stats')
-      const complete = response.total - response.remaining
-      const incomplete = response.remaining
-      this.progressStat = {
-        datasets: [{
-          data: [complete, incomplete],
-          backgroundColor: ['#00d1b2', '#ffdd57']
-        }],
-
-        labels: [
-          'Completed',
-          'Incomplete'
-        ]
-      }
+  async created() {
+    await this.fetchStatistics({
+      projectId: this.$route.params.id
     })
   },
 
   methods: {
-    makeData(object, label) {
-      const labels = Object.keys(object)
-      const counts = Object.values(object)
-      const res = {
-        labels: labels,
-        datasets: [{
-          label: label,
-          backgroundColor: '#00d1b2',
-          data: counts
-        }]
-      }
-      return res
-    }
+    ...mapActions('statistics', ['fetchStatistics'])
   },
 
   validate({ params }) {
diff --git a/frontend/services/statistics.service.js b/frontend/services/statistics.service.js
index 838e6dfe..dd731324 100644
--- a/frontend/services/statistics.service.js
+++ b/frontend/services/statistics.service.js
@@ -5,7 +5,7 @@ class StatisticsService {
     this.request = ApiService
   }
 
-  getStatistics(projectId) {
+  getStatistics({ projectId }) {
     return this.request.get(`/projects/${projectId}/statistics`)
   }
 }
diff --git a/frontend/store/statistics.js b/frontend/store/statistics.js
new file mode 100644
index 00000000..c84481ae
--- /dev/null
+++ b/frontend/store/statistics.js
@@ -0,0 +1,68 @@
+import StatisticsService from '@/services/statistics.service'
+
+function makeData(object, label) {
+  const labels = object ? Object.keys(object) : []
+  const counts = object ? Object.values(object) : []
+  return {
+    labels: labels,
+    datasets: [{
+      label: label,
+      backgroundColor: '#00d1b2',
+      data: counts
+    }]
+  }
+}
+
+export const state = () => ({
+  loading: false,
+  stats: {}
+})
+
+export const mutations = {
+  setLoading(state, payload) {
+    state.loading = payload
+  },
+  setStatistics(state, payload) {
+    state.stats = payload
+  }
+}
+
+export const getters = {
+  progress(state) {
+    const complete = state.stats.total - state.stats.remaining
+    const incomplete = state.stats.remaining
+    return {
+      datasets: [{
+        data: [complete, incomplete],
+        backgroundColor: ['#00d1b2', '#ffdd57']
+      }],
+
+      labels: [
+        'Completed',
+        'Incomplete'
+      ]
+    }
+  },
+  labelStats(state) {
+    return makeData(state.stats.label, 'Label stats')
+  },
+  userStats(state) {
+    return makeData(state.stats.user, 'User stats')
+  }
+}
+
+export const actions = {
+  fetchStatistics({ commit }, payload) {
+    commit('setLoading', true)
+    StatisticsService.getStatistics(payload)
+      .then((response) => {
+        commit('setStatistics', response)
+      })
+      .catch((error) => {
+        alert(error)
+      })
+      .finally(() => {
+        commit('setLoading', false)
+      })
+  }
+}