From 8b656070cad0a6825b13beacf0ef933093028a26 Mon Sep 17 00:00:00 2001
From: Wolkenfarmer <jomaxw2@aol.com>
Date: Sun, 1 Sep 2024 15:04:01 +0200
Subject: [PATCH] #330 frontend: use enum for the exercise status

---
 .../screensPatient/ScreenWaiting.vue          | 20 ++++++----------
 .../widgets/ExerciseControlPanel.vue          | 24 +++++++++----------
 frontend/src/sockets/MessageData.ts           | 10 +++++++-
 frontend/src/sockets/SocketPatient.ts         |  8 +++----
 frontend/src/sockets/SocketTrainer.ts         | 12 +++++-----
 frontend/src/stores/Exercise.ts               |  2 +-
 6 files changed, 39 insertions(+), 37 deletions(-)

diff --git a/frontend/src/components/screensPatient/ScreenWaiting.vue b/frontend/src/components/screensPatient/ScreenWaiting.vue
index 59cd5bf3..c42cfeeb 100644
--- a/frontend/src/components/screensPatient/ScreenWaiting.vue
+++ b/frontend/src/components/screensPatient/ScreenWaiting.vue
@@ -8,13 +8,13 @@
 
 	const title = computed(() => {
 		switch (exerciseStore.status) {
-			case 'running':
+			case ExerciseStatus.RUNNING:
 				return patientStore.relocatingInfo
-			case 'not-started':
+			case ExerciseStatus.NOT_STARTED:
 				return 'Warte, bis die Übung beginnt'
-			case 'paused':
+			case ExerciseStatus.PAUSED:
 				return 'Übung pausiert'
-			case 'ended':
+			case ExerciseStatus.ENDED:
 				return 'Übung beendet'
 			default:
 				return 'Warte auf Übung'
@@ -22,15 +22,9 @@
 	})
 
 	const info = computed(() => {
-		switch (exerciseStore.status) {
-			case 'running':
-				return new Date(new Date(0).setSeconds(patientStore.timeUntilBack)).toISOString().substring(12, 19)
-			case 'not-started':
-			case 'paused':
-			case 'ended':
-			default:
-				return 'Bereich: '+ exerciseStore.getAreaName(patientStore.areaId)
-		}
+		if (exerciseStore.status == ExerciseStatus.RUNNING) {
+			return new Date(new Date(0).setSeconds(patientStore.timeUntilBack)).toISOString().substring(12, 19)
+		} else return 'Bereich: '+ exerciseStore.getAreaName(patientStore.areaId)
 	})
 </script>
 
diff --git a/frontend/src/components/widgets/ExerciseControlPanel.vue b/frontend/src/components/widgets/ExerciseControlPanel.vue
index d200226b..3a087693 100644
--- a/frontend/src/components/widgets/ExerciseControlPanel.vue
+++ b/frontend/src/components/widgets/ExerciseControlPanel.vue
@@ -1,18 +1,18 @@
 <script setup lang="ts">
-	import { svg } from "@/assets/Svg"
-	import { ref, computed, watch } from "vue"
+	import {svg} from "@/assets/Svg"
+	import {computed, ref, watch} from "vue"
 	import socketTrainer from "@/sockets/SocketTrainer"
-	import { useExerciseStore } from "@/stores/Exercise"
+	import {useExerciseStore} from "@/stores/Exercise"
 	import ButtonPopup from "./ButtonPopup.vue"
 	import SpeedSelectorPopup from "./SpeedSelectorPopup.vue"
-	import { useAvailablesStore } from "@/stores/Availables"
-	import { useLogStore } from "@/stores/Log"
-	import { Modules, setModule } from "@/App.vue"
-	import {CustomList, ListItem, ListItemButton, ListItemName, ListItemAddButton, ListItemRight, ListItemLeft} from "@/components/widgets/List"
+	import {useAvailablesStore} from "@/stores/Availables"
+	import {useLogStore} from "@/stores/Log"
+	import {Modules, setModule} from "@/App.vue"
+	import {ListItem, ListItemLeft, ListItemName, ListItemRight} from "@/components/widgets/List"
 
 	const exerciseStore = useExerciseStore()
 
-	const status = ref('')
+	const status = ref(ExerciseStatus.UNKNOWN)
 	const speed = ref(1)
 
 	status.value = exerciseStore.status
@@ -28,13 +28,13 @@
 
 	const info = computed(() => {
 		switch (status.value) {
-			case "not-started":
+			case ExerciseStatus.NOT_STARTED:
 				return "Die Übung wurde noch nicht gestartet"
-			case "running":
+			case ExerciseStatus.RUNNING:
 				return "Die Übung läuft"
-			case "paused":
+			case ExerciseStatus.PAUSED:
 				return "Die Übung ist pausiert"
-			case "ended":
+			case ExerciseStatus.ENDED:
 				return "Die Übung ist beendet"
 			default:
 				return "Fehler"
diff --git a/frontend/src/sockets/MessageData.ts b/frontend/src/sockets/MessageData.ts
index 49906e5e..fd891cfb 100644
--- a/frontend/src/sockets/MessageData.ts
+++ b/frontend/src/sockets/MessageData.ts
@@ -33,11 +33,19 @@ interface MessageData {
 
 interface Exercise {
 	exerciseId: string
-	status: string
+	status: ExerciseStatus
 	speed: number
 	areas: Area[]
 }
 
+enum ExerciseStatus {
+	RUNNING = "running",
+	PAUSED = "paused",
+	ENDED = "ended",
+	NOT_STARTED = "not-started",
+	UNKNOWN = "unknown",
+}
+
 interface Area {
 	areaId: number
 	areaName: string
diff --git a/frontend/src/sockets/SocketPatient.ts b/frontend/src/sockets/SocketPatient.ts
index b2a7f7c6..5e8d2c7b 100644
--- a/frontend/src/sockets/SocketPatient.ts
+++ b/frontend/src/sockets/SocketPatient.ts
@@ -80,21 +80,21 @@ class SocketPatient {
 					patientStore.initializePatientFromExercise()
 					break
 				case 'exercise-start':
-					exerciseStore.status = 'running'
+					exerciseStore.status = ExerciseStatus.RUNNING
 					setScreen(Screens.STATUS, ScreenPosition.LEFT)
 					setScreen(Screens.ACTIONS, ScreenPosition.RIGHT)
 					break
 				case 'exercise-pause':
-					exerciseStore.status = 'paused'
+					exerciseStore.status = ExerciseStatus.PAUSED
 					setScreen(Screens.WAITING, ScreenPosition.FULL)
 					break
 				case 'exercise-resume':
-					exerciseStore.status = 'running'
+					exerciseStore.status = ExerciseStatus.RUNNING
 					setScreen(Screens.STATUS, ScreenPosition.LEFT)
 					setScreen(Screens.ACTIONS, ScreenPosition.RIGHT)
 					break
 				case 'exercise-end':
-					exerciseStore.status = 'ended'
+					exerciseStore.status = ExerciseStatus.ENDED
 					setScreen(Screens.ENDED, ScreenPosition.FULL)
 					break
 				case 'delete':
diff --git a/frontend/src/sockets/SocketTrainer.ts b/frontend/src/sockets/SocketTrainer.ts
index cc7650f4..d3c81b02 100644
--- a/frontend/src/sockets/SocketTrainer.ts
+++ b/frontend/src/sockets/SocketTrainer.ts
@@ -65,22 +65,22 @@ class SocketTrainer {
 					break
 				case 'exercise':
 					useExerciseStore().createFromJSON(data.exercise as Exercise)
-					if (exerciseStore.status == '') {
-						exerciseStore.status = 'not-started'
+					if (exerciseStore.status == ExerciseStatus.UNKNOWN) {
+						exerciseStore.status = ExerciseStatus.NOT_STARTED
 						setModule(Modules.TRAINER)
 					}
 					break
 				case 'exercise-start':
-					exerciseStore.status = 'running'
+					exerciseStore.status = ExerciseStatus.RUNNING
 					break
 				case 'exercise-pause':
-					exerciseStore.status = 'paused'
+					exerciseStore.status = ExerciseStatus.PAUSED
 					break
 				case 'exercise-resume':
-					exerciseStore.status = 'running'
+					exerciseStore.status = ExerciseStatus.RUNNING
 					break
 				case 'exercise-end':
-					exerciseStore.status = 'ended'
+					exerciseStore.status = ExerciseStatus.ENDED
 					break
 				case 'log-update':
 					useLogStore().addLogEntries(data.logEntries as LogEntry[])
diff --git a/frontend/src/stores/Exercise.ts b/frontend/src/stores/Exercise.ts
index 5b04ce1e..1c764dab 100644
--- a/frontend/src/stores/Exercise.ts
+++ b/frontend/src/stores/Exercise.ts
@@ -3,7 +3,7 @@ import {defineStore} from 'pinia'
 export const useExerciseStore = defineStore('exercise', {
 	state: (): Exercise => ({
 		exerciseId: "",
-		status: "",
+		status: ExerciseStatus.RUNNING,
 		speed: 1,
 		areas: [],
 	}),