From 105e853f01b0b35452e9faad90763edf3e4c17a4 Mon Sep 17 00:00:00 2001
From: Methapon2001 <61303214+Methapon2001@users.noreply.github.com>
Date: Tue, 3 Dec 2024 11:22:46 +0700
Subject: [PATCH] refactor: store data in field instead of attribute

---
 src/pages/08_request-list/FormDuty.vue        |  2 +-
 src/pages/08_request-list/FormExpansion.vue   | 19 ++++++++--------
 ...rmEmployee.vue => FormResponsibleUser.vue} | 22 +++++++++++--------
 src/pages/08_request-list/RequestListView.vue |  3 +--
 src/stores/request-list/index.ts              | 10 ++++++++-
 src/stores/request-list/types.ts              | 18 ++++++++++-----
 6 files changed, 47 insertions(+), 27 deletions(-)
 rename src/pages/08_request-list/{FormEmployee.vue => FormResponsibleUser.vue} (64%)

diff --git a/src/pages/08_request-list/FormDuty.vue b/src/pages/08_request-list/FormDuty.vue
index e785b926..25a421f4 100644
--- a/src/pages/08_request-list/FormDuty.vue
+++ b/src/pages/08_request-list/FormDuty.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-const duty = defineModel<boolean>('duty', { required: true });
+const duty = defineModel<boolean>('duty', { required: false });
 const dutyCost = defineModel<number>('dutyCost', { required: false });
 
 withDefaults(
diff --git a/src/pages/08_request-list/FormExpansion.vue b/src/pages/08_request-list/FormExpansion.vue
index 3cedab00..5de34c12 100644
--- a/src/pages/08_request-list/FormExpansion.vue
+++ b/src/pages/08_request-list/FormExpansion.vue
@@ -3,7 +3,7 @@ import { reactive, ref } from 'vue';
 import FormGroupHead from './FormGroupHead.vue';
 import FormDuty from './FormDuty.vue';
 import FormIssue from './FormIssue.vue';
-import FormEmployee from './FormEmployee.vue';
+import FormResponsibleUser from './FormResponsibleUser.vue';
 
 import { UndoButton, SaveButton, EditButton } from 'src/components/button';
 import { AttributesForm, Step } from 'src/stores/request-list/types';
@@ -25,8 +25,8 @@ const defaultForm = {
   customerDutyCost: 30,
   companyDuty: false,
   companyDutyCost: 30,
-  localEmployee: true,
-  employeeId: '',
+  responsibleUserLocal: true,
+  responsibleUserId: '',
   individualDuty: false,
   individualDutyCost: 10,
 };
@@ -37,8 +37,8 @@ const attributesForm = defineModel<AttributesForm>('attributesForm', {
     customerDutyCost: 30,
     companyDuty: false,
     companyDutyCost: 30,
-    localEmployee: true,
-    employeeId: '',
+    responsibleUserLocal: true,
+    responsibleUserId: '',
     individualDuty: false,
     individualDutyCost: 10,
   },
@@ -55,7 +55,8 @@ function triggerUndo() {
 async function triggerSubmit() {
   const payload = {
     ...props.step,
-    attributes: { form: formData.value },
+    ...formData.value,
+    attributes: {},
   };
   const res = await requestListStore.editStatusRequestWork(payload);
 
@@ -150,11 +151,11 @@ function assignToForm() {
             $t('general.select', { msg: $t('requestList.employeeMessenger') })
           }}
         </FormGroupHead>
-        <FormEmployee
+        <FormResponsibleUser
           :readonly="!state.isEdit"
           class="surface-1"
-          v-model:local-employee="formData.localEmployee"
-          v-model:employee-id="formData.employeeId"
+          v-model:responsible-user-local="formData.responsibleUserLocal"
+          v-model:responsible-user-id="formData.responsibleUserId"
           :district-id="responsibleAreaDistrictId"
         />
       </section>
diff --git a/src/pages/08_request-list/FormEmployee.vue b/src/pages/08_request-list/FormResponsibleUser.vue
similarity index 64%
rename from src/pages/08_request-list/FormEmployee.vue
rename to src/pages/08_request-list/FormResponsibleUser.vue
index 3cdcc378..31f6e209 100644
--- a/src/pages/08_request-list/FormEmployee.vue
+++ b/src/pages/08_request-list/FormResponsibleUser.vue
@@ -2,8 +2,12 @@
 import SelectUser from 'src/components/shared/select/SelectUser.vue';
 import { watch } from 'vue';
 
-const localEmployee = defineModel<boolean>('localEmployee', { required: true });
-const employeeId = defineModel<string>('employeeId', { required: false });
+const responsibleUserLocal = defineModel<boolean>('responsibleUserLocal', {
+  required: false,
+});
+const responsibleUserId = defineModel<string>('responsibleUserId', {
+  required: false,
+});
 
 defineProps<{
   readonly?: boolean;
@@ -11,22 +15,22 @@ defineProps<{
   cost?: boolean;
 }>();
 
-watch(localEmployee, (lhs, rhs) => {
-  if (lhs && !rhs) employeeId.value = '';
+watch(responsibleUserLocal, (lhs, rhs) => {
+  if (lhs && !rhs) responsibleUserId.value = '';
 });
 </script>
 <template>
   <div class="row items-center q-px-md q-py-xs">
     <div class="col-12 row">
       <q-radio
-        v-model="localEmployee"
+        v-model="responsibleUserLocal"
         :val="true"
         :label="$t('requestList.localEmployee')"
         :disable="readonly"
         class="col"
       />
       <q-radio
-        v-model="localEmployee"
+        v-model="responsibleUserLocal"
         :val="false"
         :label="$t('requestList.nonLocalEmployee')"
         :disable="readonly"
@@ -38,12 +42,12 @@ watch(localEmployee, (lhs, rhs) => {
 
     <div class="col-12 row">
       <SelectUser
-        :key="localEmployee.toString()"
+        :key="responsibleUserLocal.toString()"
         class="col-md-5 col-12"
-        v-model:value="employeeId"
+        v-model:value="responsibleUserId"
         :params="{
           userType: 'MESSENGER',
-          responsibleDistrictId: localEmployee ? districtId : undefined,
+          responsibleDistrictId: responsibleUserLocal ? districtId : undefined,
         }"
         :readonly
         :label="$t('general.select', { msg: $t('personnel.MESSENGER') })"
diff --git a/src/pages/08_request-list/RequestListView.vue b/src/pages/08_request-list/RequestListView.vue
index 0367939f..457c2eaf 100644
--- a/src/pages/08_request-list/RequestListView.vue
+++ b/src/pages/08_request-list/RequestListView.vue
@@ -722,8 +722,7 @@ function isInstallmentPaySuccess(installmentNo: number) {
                     }"
                     :id="value.id"
                     :attributes-form="
-                      value.stepStatus?.[pageState.currentStep - 1]?.attributes
-                        ?.form
+                      value.stepStatus?.[pageState.currentStep - 1]
                     "
                     :responsible-area-district-id="
                       data.quotation.customerBranch.districtId
diff --git a/src/stores/request-list/index.ts b/src/stores/request-list/index.ts
index a9cb7807..acb6a92d 100644
--- a/src/stores/request-list/index.ts
+++ b/src/stores/request-list/index.ts
@@ -240,8 +240,16 @@ export const useRequestList = defineStore('request-list', () => {
     const res = await api.put<Step>(
       `/request-work/${body.requestWorkId}/step-status/${body.step}`,
       {
+        customerDuty: body.customerDuty,
+        customerDutyCost: body.customerDutyCost,
+        companyDuty: body.customerDuty,
+        companyDutyCost: body.companyDutyCost,
+        individualDuty: body.individualDuty,
+        individualDutyCost: body.individualDutyCost,
+        responsibleUserLocal: body.responsibleUserLocal,
+        responsibleUserId: body.responsibleUserId,
         attributes: body.attributes,
-        requestWorkStatus: body.workStatus,
+        workStatus: body.workStatus,
         requestWorkId: undefined,
         step: undefined,
       },
diff --git a/src/stores/request-list/types.ts b/src/stores/request-list/types.ts
index 81d4fb02..dea7f38d 100644
--- a/src/stores/request-list/types.ts
+++ b/src/stores/request-list/types.ts
@@ -72,14 +72,14 @@ export type Attributes = {
 };
 
 export type AttributesForm = {
-  customerDuty: boolean;
+  customerDuty?: boolean;
   customerDutyCost?: number;
-  companyDuty: boolean;
+  companyDuty?: boolean;
   companyDutyCost?: number;
-  localEmployee: boolean;
-  employeeId?: string;
-  individualDuty: boolean;
+  individualDuty?: boolean;
   individualDutyCost?: number;
+  responsibleUserLocal?: boolean;
+  responsibleUserId?: string;
 };
 
 export type Step = {
@@ -87,4 +87,12 @@ export type Step = {
   requestWorkId: string;
   workStatus?: RequestWorkStatus;
   step: number;
+  customerDuty?: boolean;
+  customerDutyCost?: number;
+  companyDuty?: boolean;
+  companyDutyCost?: number;
+  individualDuty?: boolean;
+  individualDutyCost?: number;
+  responsibleUserLocal?: boolean;
+  responsibleUserId?: string;
 };