From 3dbdd5cb6bee2bd462770f2a5774709a54d3fb8e Mon Sep 17 00:00:00 2001
From: Sam <sam@samr1.net>
Date: Tue, 28 Sep 2021 09:10:01 +0200
Subject: [PATCH] Client - delete a workout

---
 .../src/components/Common/Modal.vue           | 113 ++++++++++++++++++
 .../WorkoutDetail/WorkoutCardTitle.vue        |  10 +-
 .../Workout/WorkoutDetail/index.vue           |  30 ++++-
 fittrackee_client/src/locales/en/api.json     |   1 +
 fittrackee_client/src/locales/en/buttons.json |   4 +-
 fittrackee_client/src/locales/en/common.json  |   1 +
 .../src/locales/en/workouts.json              |   3 +-
 fittrackee_client/src/locales/fr/api.json     |   1 +
 fittrackee_client/src/locales/fr/buttons.json |   4 +-
 fittrackee_client/src/locales/fr/common.json  |   1 +
 .../src/locales/fr/workouts.json              |   3 +-
 fittrackee_client/src/scss/base.scss          |  21 +++-
 fittrackee_client/src/scss/colors.scss        |   8 ++
 .../src/store/modules/workouts/actions.ts     |  16 +++
 .../src/store/modules/workouts/enums.ts       |   1 +
 .../src/store/modules/workouts/mutations.ts   |   8 +-
 .../src/store/modules/workouts/state.ts       |  14 +--
 .../src/store/modules/workouts/types.ts       |   4 +
 fittrackee_client/src/utils/index.ts          |   4 +
 fittrackee_client/src/views/DashBoard.vue     |   7 +-
 20 files changed, 234 insertions(+), 20 deletions(-)
 create mode 100644 fittrackee_client/src/components/Common/Modal.vue

diff --git a/fittrackee_client/src/components/Common/Modal.vue b/fittrackee_client/src/components/Common/Modal.vue
new file mode 100644
index 00000000..32cc0a5d
--- /dev/null
+++ b/fittrackee_client/src/components/Common/Modal.vue
@@ -0,0 +1,113 @@
+<template>
+  <div id="modal">
+    <div class="custom-modal">
+      <Card :without-title="false">
+        <template #title>
+          {{ title }}
+        </template>
+        <template #content>
+          <div class="modal-message">{{ message }}</div>
+          <ErrorMessage :message="errorMessages" v-if="errorMessages" />
+          <div class="modal-buttons">
+            <button class="confirm" @click="emit('confirmAction')">
+              {{ t('buttons.YES') }}
+            </button>
+            <button class="cancel" @click="emit('cancelAction')">
+              {{ t('buttons.NO') }}
+            </button>
+          </div>
+        </template>
+      </Card>
+    </div>
+  </div>
+</template>
+
+<script lang="ts">
+  import { ComputedRef, computed, defineComponent, onUnmounted } from 'vue'
+  import { useI18n } from 'vue-i18n'
+
+  import Card from '@/components/Common/Card.vue'
+  import ErrorMessage from '@/components/Common/ErrorMessage.vue'
+  import { ROOT_STORE } from '@/store/constants'
+  import { useStore } from '@/use/useStore'
+
+  export default defineComponent({
+    name: 'Modal',
+    components: {
+      Card,
+      ErrorMessage,
+    },
+    props: {
+      title: {
+        type: String,
+        required: true,
+      },
+      message: {
+        type: String,
+        required: true,
+      },
+    },
+    emits: ['cancelAction', 'confirmAction'],
+    setup(props, { emit }) {
+      const { t } = useI18n()
+      const store = useStore()
+      const errorMessages: ComputedRef<string | string[] | null> = computed(
+        () => store.getters[ROOT_STORE.GETTERS.ERROR_MESSAGES]
+      )
+      onUnmounted(() => store.commit(ROOT_STORE.MUTATIONS.EMPTY_ERROR_MESSAGES))
+      return { errorMessages, t, emit }
+    },
+  })
+</script>
+
+<style lang="scss" scoped>
+  @import '~@/scss/base';
+  #modal {
+    position: fixed;
+    top: 0;
+    bottom: 0;
+    left: 0;
+    right: 0;
+    background-color: var(--modal-background-color);
+    padding: $default-padding;
+    z-index: 1240;
+
+    .custom-modal {
+      background-color: var(--app-background-color);
+      border-radius: $border-radius;
+      max-width: 500px;
+      margin: 25% auto;
+      z-index: 1250;
+
+      ::v-deep(.card) {
+        border: 0;
+        margin: 0;
+
+        .card-content {
+          display: flex;
+          flex-direction: column;
+
+          .modal-message {
+            padding: $default-padding;
+          }
+
+          .modal-buttons {
+            display: flex;
+            justify-content: flex-end;
+
+            button {
+              margin: $default-padding * 0.5;
+            }
+          }
+        }
+      }
+    }
+
+    @media screen and (max-width: $small-limit) {
+      .custom-modal {
+        margin: 20% 0;
+        width: 100%;
+      }
+    }
+  }
+</style>
diff --git a/fittrackee_client/src/components/Workout/WorkoutDetail/WorkoutCardTitle.vue b/fittrackee_client/src/components/Workout/WorkoutDetail/WorkoutCardTitle.vue
index 67aa1312..d2eb5c70 100644
--- a/fittrackee_client/src/components/Workout/WorkoutDetail/WorkoutCardTitle.vue
+++ b/fittrackee_client/src/components/Workout/WorkoutDetail/WorkoutCardTitle.vue
@@ -23,6 +23,11 @@
       <div class="workout-title-date">
         <div class="workout-title" v-if="workoutObject.type === 'WORKOUT'">
           {{ workoutObject.title }}
+          <i
+            class="fa fa-trash"
+            aria-hidden="true"
+            @click="emit('displayModal', true)"
+          />
         </div>
         <div class="workout-title" v-else>
           {{ workoutObject.title }}
@@ -86,9 +91,10 @@
         required: true,
       },
     },
-    setup() {
+    emits: ['displayModal'],
+    setup(props, { emit }) {
       const { t } = useI18n()
-      return { t }
+      return { t, emit }
     },
   })
 </script>
diff --git a/fittrackee_client/src/components/Workout/WorkoutDetail/index.vue b/fittrackee_client/src/components/Workout/WorkoutDetail/index.vue
index 9ab509ef..afca581c 100644
--- a/fittrackee_client/src/components/Workout/WorkoutDetail/index.vue
+++ b/fittrackee_client/src/components/Workout/WorkoutDetail/index.vue
@@ -1,8 +1,19 @@
 <template>
   <div class="workout-detail">
+    <Modal
+      v-if="displayModal"
+      :title="t('common.CONFIRMATION')"
+      :message="t('workouts.WORKOUT_DELETION_CONFIRMATION')"
+      @confirmAction="deleteWorkout(workoutObject.workoutId)"
+      @cancelAction="updateDisplayModal(false)"
+    />
     <Card :without-title="false">
       <template #title>
-        <WorkoutCardTitle :sport="sport" :workoutObject="workoutObject" />
+        <WorkoutCardTitle
+          :sport="sport"
+          :workoutObject="workoutObject"
+          @displayModal="updateDisplayModal(true)"
+        />
       </template>
       <template #content>
         <WorkoutMap
@@ -29,9 +40,11 @@
   import { useRoute } from 'vue-router'
 
   import Card from '@/components/Common/Card.vue'
+  import Modal from '@/components/Common/Modal.vue'
   import WorkoutCardTitle from '@/components/Workout/WorkoutDetail/WorkoutCardTitle.vue'
   import WorkoutData from '@/components/Workout/WorkoutDetail/WorkoutData.vue'
   import WorkoutMap from '@/components/Workout/WorkoutDetail/WorkoutMap.vue'
+  import { WORKOUTS_STORE } from '@/store/constants'
   import { ISport } from '@/types/sports'
   import { IAuthUserProfile } from '@/types/user'
   import {
@@ -41,12 +54,14 @@
     IWorkoutSegment,
     TCoordinates,
   } from '@/types/workouts'
+  import { useStore } from '@/use/useStore'
   import { formatWorkoutDate, getDateWithTZ } from '@/utils/dates'
 
   export default defineComponent({
     name: 'WorkoutDetail',
     components: {
       Card,
+      Modal,
       WorkoutCardTitle,
       WorkoutData,
       WorkoutMap,
@@ -74,6 +89,7 @@
     },
     setup(props) {
       const route = useRoute()
+      const store = useStore()
       const { t } = useI18n()
 
       function getWorkoutObjectUrl(
@@ -137,6 +153,14 @@
           workoutTime: workoutDate.workout_time,
         }
       }
+      function updateDisplayModal(value: boolean) {
+        displayModal.value = value
+      }
+      function deleteWorkout(workoutId: string) {
+        store.dispatch(WORKOUTS_STORE.ACTIONS.DELETE_WORKOUT, {
+          workoutId: workoutId,
+        })
+      }
 
       const workout: ComputedRef<IWorkout> = computed(
         () => props.workoutData.workout
@@ -149,6 +173,7 @@
           ? workout.value.segments[+segmentId.value - 1]
           : null
       )
+      let displayModal: Ref<boolean> = ref(false)
 
       watch(
         () => route.params.segmentId,
@@ -170,7 +195,10 @@
         workoutObject: computed(() =>
           getWorkoutObject(workout.value, segment.value)
         ),
+        displayModal,
         t,
+        deleteWorkout,
+        updateDisplayModal,
       }
     },
   })
diff --git a/fittrackee_client/src/locales/en/api.json b/fittrackee_client/src/locales/en/api.json
index ad320ff2..0c70e619 100644
--- a/fittrackee_client/src/locales/en/api.json
+++ b/fittrackee_client/src/locales/en/api.json
@@ -1,6 +1,7 @@
 {
   "ERROR": {
     "UNKNOWN": "Error. Please try again or contact the administrator.",
+    "Error, Please try again or contact the administrator": "Error. Please try again or contact the administrator.",
     "Invalid credentials": "Invalid credentials.",
     "Network Error": "Network Error",
     "Password and password confirmation don't match": "Password and password confirmation don't match.",
diff --git a/fittrackee_client/src/locales/en/buttons.json b/fittrackee_client/src/locales/en/buttons.json
index 5631a0a0..a666fea0 100644
--- a/fittrackee_client/src/locales/en/buttons.json
+++ b/fittrackee_client/src/locales/en/buttons.json
@@ -1,4 +1,6 @@
 {
   "LOGIN": "Log in",
-  "REGISTER": "Register"
+  "NO": "No",
+  "REGISTER": "Register",
+  "YES": "Yes"
 }
\ No newline at end of file
diff --git a/fittrackee_client/src/locales/en/common.json b/fittrackee_client/src/locales/en/common.json
index 177ccabf..ad93e543 100644
--- a/fittrackee_client/src/locales/en/common.json
+++ b/fittrackee_client/src/locales/en/common.json
@@ -1,4 +1,5 @@
 {
+  "CONFIRMATION": "Confirmation",
   "DAY": "day | days",
   "HOME": "Home"
 }
\ No newline at end of file
diff --git a/fittrackee_client/src/locales/en/workouts.json b/fittrackee_client/src/locales/en/workouts.json
index 34d5c037..b33e69f6 100644
--- a/fittrackee_client/src/locales/en/workouts.json
+++ b/fittrackee_client/src/locales/en/workouts.json
@@ -57,5 +57,6 @@
       "wind":  "wind"
     }
   },
-  "WORKOUT": "workout | workouts"
+  "WORKOUT": "workout | workouts",
+  "WORKOUT_DELETION_CONFIRMATION": "Are you sure you want to delete this workout?"
 }
diff --git a/fittrackee_client/src/locales/fr/api.json b/fittrackee_client/src/locales/fr/api.json
index 9f8fc5ff..3014a728 100644
--- a/fittrackee_client/src/locales/fr/api.json
+++ b/fittrackee_client/src/locales/fr/api.json
@@ -1,6 +1,7 @@
 {
   "ERROR": {
     "UNKNOWN": "Erreur. Veuillez réessayer ou contacter l'administrateur.",
+    "Error, Please try again or contact the administrator": "Erreur. Veuillez réessayer ou contacter l'administrateur.",
     "Invalid credentials": "Identifiants invalides.",
     "Network Error": "Erreur Réseau",
     "Password and password confirmation don't match": "Les mots de passe saisis sont différents.",
diff --git a/fittrackee_client/src/locales/fr/buttons.json b/fittrackee_client/src/locales/fr/buttons.json
index 7496ec9d..6b0f7ae9 100644
--- a/fittrackee_client/src/locales/fr/buttons.json
+++ b/fittrackee_client/src/locales/fr/buttons.json
@@ -1,4 +1,6 @@
 {
   "LOGIN": "Se connecter",
-  "REGISTER": "S'inscrire"
+  "NO": "Non",
+  "REGISTER": "S'inscrire",
+  "YES": "Oui"
 }
\ No newline at end of file
diff --git a/fittrackee_client/src/locales/fr/common.json b/fittrackee_client/src/locales/fr/common.json
index 7c78f4d2..a47e2d40 100644
--- a/fittrackee_client/src/locales/fr/common.json
+++ b/fittrackee_client/src/locales/fr/common.json
@@ -1,4 +1,5 @@
 {
+  "CONFIRMATION": "Confirmation",
   "DAY": "jour | jours",
   "HOME": "Accueil"
 }
\ No newline at end of file
diff --git a/fittrackee_client/src/locales/fr/workouts.json b/fittrackee_client/src/locales/fr/workouts.json
index d7e4e296..80c485f3 100644
--- a/fittrackee_client/src/locales/fr/workouts.json
+++ b/fittrackee_client/src/locales/fr/workouts.json
@@ -57,5 +57,6 @@
       "wind":  "venteux"
     }
   },
-  "WORKOUT": "séance | séances"
+  "WORKOUT": "séance | séances",
+  "WORKOUT_DELETION_CONFIRMATION": "Etes-vous sûr de vouloir supprimer cette séance ?"
 }
diff --git a/fittrackee_client/src/scss/base.scss b/fittrackee_client/src/scss/base.scss
index 6671647b..f7371f17 100644
--- a/fittrackee_client/src/scss/base.scss
+++ b/fittrackee_client/src/scss/base.scss
@@ -51,7 +51,7 @@ button {
 
   &:hover {
     background: var(--app-color);
-    color: #FFFFFF;
+    color: var(--button-hover-color);
   }
 
   &:enabled:active {
@@ -64,6 +64,25 @@ button {
     border-color: var(--disabled-color);
     color: var(--disabled-color);
   }
+
+  &.cancel {
+    background: var(--button-cancel-bg-color);
+    color: var(--button-cancel-color);
+    &:hover {
+    background: var(--app-color);
+    color: var(--button-hover-color);
+    }
+  }
+
+  &.confirm {
+    background: var(--button-confirm-bg-color);
+    color: var(--button-confirm-color);
+    &:hover {
+    background: var(--app-color);
+    color: var(--button-hover-color);
+    }
+  }
+
 }
 
 .container {
diff --git a/fittrackee_client/src/scss/colors.scss b/fittrackee_client/src/scss/colors.scss
index 57aba3e0..b04df8e9 100644
--- a/fittrackee_client/src/scss/colors.scss
+++ b/fittrackee_client/src/scss/colors.scss
@@ -7,6 +7,12 @@
   --app-loading-color:  #f3f3f3;
   --app-loading-top-color: var(--app-color);
 
+  --button-hover-color: #FFFFFF;
+  --button-cancel-bg-color: #FFFFFF;
+  --button-cancel-color: var(--app-color);
+  --button-confirm-bg-color: #FFFFFF;
+  --button-confirm-color: var(--app-color);
+
   --card-border-color: #c4c7cf;
   --input-border-color: #9da3af;
 
@@ -14,6 +20,8 @@
   --calendar-week-end-color: #f5f5f5;
   --calendar-today-color: #eff1f3;
 
+  --modal-background-color: rgba(0, 0, 0, 0.3);
+
   --nav-bar-background-color: #FFFFFF;
   --nav-bar-link-active: #485b6e;
   --nav-border-color: #c5ccdb;
diff --git a/fittrackee_client/src/store/modules/workouts/actions.ts b/fittrackee_client/src/store/modules/workouts/actions.ts
index 7ace6bdd..37170239 100644
--- a/fittrackee_client/src/store/modules/workouts/actions.ts
+++ b/fittrackee_client/src/store/modules/workouts/actions.ts
@@ -1,6 +1,7 @@
 import { ActionContext, ActionTree } from 'vuex'
 
 import authApi from '@/api/authApi'
+import router from '@/router'
 import { ROOT_STORE, WORKOUTS_STORE } from '@/store/constants'
 import { IRootState } from '@/store/modules/root/types'
 import {
@@ -107,4 +108,19 @@ export const actions: ActionTree<IWorkoutsState, IRootState> &
         context.commit(WORKOUTS_STORE.MUTATIONS.SET_WORKOUT_LOADING, false)
       )
   },
+  [WORKOUTS_STORE.ACTIONS.DELETE_WORKOUT](
+    context: ActionContext<IWorkoutsState, IRootState>,
+    payload: IWorkoutPayload
+  ): void {
+    context.commit(ROOT_STORE.MUTATIONS.EMPTY_ERROR_MESSAGES)
+    authApi
+      .delete(`workouts/${payload.workoutId}`)
+      .then(() => {
+        context.commit(WORKOUTS_STORE.MUTATIONS.EMPTY_WORKOUT)
+        router.push('/')
+      })
+      .catch((error) => {
+        handleError(context, error)
+      })
+  },
 }
diff --git a/fittrackee_client/src/store/modules/workouts/enums.ts b/fittrackee_client/src/store/modules/workouts/enums.ts
index 877e6418..a178d3aa 100644
--- a/fittrackee_client/src/store/modules/workouts/enums.ts
+++ b/fittrackee_client/src/store/modules/workouts/enums.ts
@@ -1,4 +1,5 @@
 export enum WorkoutsActions {
+  DELETE_WORKOUT = 'DELETE_WORKOUT',
   GET_CALENDAR_WORKOUTS = 'GET_CALENDAR_WORKOUTS',
   GET_USER_WORKOUTS = 'GET_USER_WORKOUTS',
   GET_WORKOUT_DATA = 'GET_WORKOUT_DATA',
diff --git a/fittrackee_client/src/store/modules/workouts/mutations.ts b/fittrackee_client/src/store/modules/workouts/mutations.ts
index d866ad68..f695bd9d 100644
--- a/fittrackee_client/src/store/modules/workouts/mutations.ts
+++ b/fittrackee_client/src/store/modules/workouts/mutations.ts
@@ -1,7 +1,6 @@
 import { MutationTree } from 'vuex'
 
 import { WORKOUTS_STORE } from '@/store/constants'
-import { initialWorkoutValue } from '@/store/modules/workouts/state'
 import {
   IWorkoutsState,
   TWorkoutsMutations,
@@ -50,6 +49,11 @@ export const mutations: MutationTree<IWorkoutsState> & TWorkoutsMutations = {
     state.user_workouts = []
   },
   [WORKOUTS_STORE.MUTATIONS.EMPTY_WORKOUT](state: IWorkoutsState) {
-    state.workoutData = initialWorkoutValue
+    state.workoutData = {
+      gpx: '',
+      loading: false,
+      workout: <IWorkout>{},
+      chartData: [],
+    }
   },
 }
diff --git a/fittrackee_client/src/store/modules/workouts/state.ts b/fittrackee_client/src/store/modules/workouts/state.ts
index 9972b669..2015c098 100644
--- a/fittrackee_client/src/store/modules/workouts/state.ts
+++ b/fittrackee_client/src/store/modules/workouts/state.ts
@@ -1,15 +1,13 @@
 import { IWorkoutsState } from '@/store/modules/workouts/types'
 import { IWorkout } from '@/types/workouts'
 
-export const initialWorkoutValue = {
-  gpx: '',
-  loading: false,
-  workout: <IWorkout>{},
-  chartData: [],
-}
-
 export const workoutsState: IWorkoutsState = {
   calendar_workouts: [],
   user_workouts: [],
-  workoutData: initialWorkoutValue,
+  workoutData: {
+    gpx: '',
+    loading: false,
+    workout: <IWorkout>{},
+    chartData: [],
+  },
 }
diff --git a/fittrackee_client/src/store/modules/workouts/types.ts b/fittrackee_client/src/store/modules/workouts/types.ts
index 9bc76f4e..cfb24d33 100644
--- a/fittrackee_client/src/store/modules/workouts/types.ts
+++ b/fittrackee_client/src/store/modules/workouts/types.ts
@@ -34,6 +34,10 @@ export interface IWorkoutsActions {
     context: ActionContext<IWorkoutsState, IRootState>,
     payload: IWorkoutPayload
   ): void
+  [WORKOUTS_STORE.ACTIONS.DELETE_WORKOUT](
+    context: ActionContext<IWorkoutsState, IRootState>,
+    payload: IWorkoutPayload
+  ): void
 }
 
 export interface IWorkoutsGetters {
diff --git a/fittrackee_client/src/utils/index.ts b/fittrackee_client/src/utils/index.ts
index 0edf17fa..83084044 100644
--- a/fittrackee_client/src/utils/index.ts
+++ b/fittrackee_client/src/utils/index.ts
@@ -17,6 +17,9 @@ export const getApiUrl = (): string => {
 // TODO: update api error messages to remove these workarounds
 const removeLastEndOfLine = (text: string): string => text.replace(/\n$/gm, '')
 const removeLastDot = (text: string): string => text.replace(/\.$/gm, '')
+const removeErrorDot = (text: string): string =>
+  text.replace(/^Error\./gm, 'Error,')
+
 export const handleError = (
   context:
     | ActionContext<IRootState, IRootState>
@@ -37,6 +40,7 @@ export const handleError = (
     ? error.message
     : msg
   errorMessages = removeLastEndOfLine(errorMessages)
+  errorMessages = removeErrorDot(errorMessages)
   context.commit(
     ROOT_STORE.MUTATIONS.SET_ERROR_MESSAGES,
     errorMessages.includes('\n')
diff --git a/fittrackee_client/src/views/DashBoard.vue b/fittrackee_client/src/views/DashBoard.vue
index 1f7c813b..a2bf555f 100644
--- a/fittrackee_client/src/views/DashBoard.vue
+++ b/fittrackee_client/src/views/DashBoard.vue
@@ -17,14 +17,14 @@
 </template>
 
 <script lang="ts">
-  import { computed, ComputedRef, defineComponent } from 'vue'
+  import { computed, ComputedRef, defineComponent, onUnmounted } from 'vue'
 
   import Timeline from '@/components/Dashboard/Timeline/index.vue'
   import UserCalendar from '@/components/Dashboard/UserCalendar/index.vue'
   import UserMonthStats from '@/components/Dashboard/UserMonthStats.vue'
   import UserRecords from '@/components/Dashboard/UserRecords/index.vue'
   import UserStatsCards from '@/components/Dashboard/UserStatsCards/index.vue'
-  import { SPORTS_STORE, USER_STORE } from '@/store/constants'
+  import { SPORTS_STORE, USER_STORE, WORKOUTS_STORE } from '@/store/constants'
   import { ISport } from '@/types/sports'
   import { IAuthUserProfile } from '@/types/user'
   import { useStore } from '@/use/useStore'
@@ -46,6 +46,9 @@
       const sports: ComputedRef<ISport[]> = computed(
         () => store.getters[SPORTS_STORE.GETTERS.SPORTS]
       )
+      onUnmounted(() => {
+        store.commit(WORKOUTS_STORE.MUTATIONS.EMPTY_WORKOUTS)
+      })
       return { authUser, sports }
     },
   })