From 8a0e9072c10f8da09c4b72df49870fce485977ba Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Matej=20Glasn=C3=A1k?= <glasnak@wezeo.com>
Date: Thu, 17 Oct 2024 08:36:08 +0200
Subject: [PATCH] [Fix] Handling of deleted employee in master task

---
 .../master-task/_components/a-tasks-table.vue       |  4 ++--
 .../tm@modal/master-task/z-master-task-modal.vue    | 13 +++++++++----
 2 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/app/src/plugins/tm@modal/master-task/_components/a-tasks-table.vue b/app/src/plugins/tm@modal/master-task/_components/a-tasks-table.vue
index 187e9ef..3d71c97 100644
--- a/app/src/plugins/tm@modal/master-task/_components/a-tasks-table.vue
+++ b/app/src/plugins/tm@modal/master-task/_components/a-tasks-table.vue
@@ -47,8 +47,8 @@ export default {
 
 				return {
 					...subtask,
-					_employeeInitials: employee._initials,
-					_employeeFullName: employee._fullName
+					_employeeInitials: employee ? employee._initials : '?',
+					_employeeFullName: employee ? employee._fullName : ''
 				}
 			})
 		}
diff --git a/app/src/plugins/tm@modal/master-task/z-master-task-modal.vue b/app/src/plugins/tm@modal/master-task/z-master-task-modal.vue
index d125133..b44d7c9 100644
--- a/app/src/plugins/tm@modal/master-task/z-master-task-modal.vue
+++ b/app/src/plugins/tm@modal/master-task/z-master-task-modal.vue
@@ -23,7 +23,7 @@
 
 					<v-text-field v-model="mastertask.duration" @focus="onInputFocus" class="modal-input" variant="solo" type="number" :label="`${mastertask.duration} h`" :disabled="true"></v-text-field>
 
-					<v-select v-model="mastertask.team_id" @focus="onInputFocus" class="modal-input border-select" :class="{error: v$.mastertask.employees.$errors.length}" label="Tím" variant="solo" :items="teams" item-value="id" item-title="name" :disabled="mastertask.id" :clearable="true" />
+					<v-select v-model="mastertask.team_id" @focus="onInputFocus" class="modal-input border-select" :class="{error: v$.mastertask.employees.$errors.length}" label="Tím" variant="solo" :items="teams" item-value="id" item-title="name" :disabled="!!mastertask.id" :clearable="true" />
 
 					<template v-if="!mastertask.id">
 						<v-text-field v-for="(employee, index) in mastertask.employees" :key="employee.id" @focus="onInputFocus" class="modal-input" :class="{error: employee.invalid}" :label="`${employee.name} ${employee.surname}`" variant="solo" placeholder="h" v-model="mastertask.employees[index].duration" type="number" min="1" @input="onEmployeeDurationChanged">
@@ -50,7 +50,7 @@
 
 					<!-- <Z-autocomplete label="Vyhľadať zamestnanca"  :options="employeesOptions" class="modal-input" @option-selected="onEmployeeSelected" /> -->
 
-					<v-btn :disabled="mastertask.id" class="w-100 mb-5 button secondary" :class="{error: v$.mastertask.project_id.$errors.length}" @click="showAddEmployeeInput = true"> <v-icon class="mx-1" icon="mdi-plus" />Pridať zamestnanca </v-btn>
+					<v-btn :disabled="!!mastertask.id" class="w-100 mb-5 button secondary" :class="{error: v$.mastertask.project_id.$errors.length}" @click="showAddEmployeeInput = true"> <v-icon class="mx-1" icon="mdi-plus" />Pridať zamestnanca </v-btn>
 
 					<div class="w-100 modal-input" :class="{ error: v$.mastertask.description.$errors.length }">
 						<QuillEditor ref="description" v-model="mastertask.description" placeholder="Popis" theme="snow" class="rich-text-editor" @keyup="onQuillEditorInput" />
@@ -170,7 +170,7 @@ export default {
 					_initials: employee ? employee.name.charAt(0) + employee.surname.charAt(0) : '?',
 					_fullName: employee ? `${employee.name} ${employee.surname}` : ''
 				}
-			})
+			}).filter(subtask => subtask !== null)
 		}
 	},
 
@@ -275,10 +275,15 @@ export default {
 
 					this.mastertask._allEmployees = this.mastertask.subtasks.map(({ employee_id, duration}) => {
 						const employee = this.enrichedEmployees.find(({ id }) => id == employee_id)
+
+						if (!employee) {
+							return null
+						}
+
 						employee.duration = duration
 
 						return employee
-					})
+					}).filter(employee => employee !== null)
 
 					// this.mastertask.project_code 	= project.code
 				} else {