From d3e5aec842572d04e29d5a0f37dd8b8121754351 Mon Sep 17 00:00:00 2001
From: puriphatt <puriphat@frappet.com>
Date: Thu, 10 Apr 2025 18:04:37 +0700
Subject: [PATCH] refactor: rename agencyFile and agencyFileList to userFile
 and userFileList for clarity

---
 .../02_personnel-management/FormByType.vue    | 188 ++++++++----------
 .../02_personnel-management/MainPage.vue      |  40 ++--
 2 files changed, 105 insertions(+), 123 deletions(-)

diff --git a/src/components/02_personnel-management/FormByType.vue b/src/components/02_personnel-management/FormByType.vue
index 1537dc07..114fd1b9 100644
--- a/src/components/02_personnel-management/FormByType.vue
+++ b/src/components/02_personnel-management/FormByType.vue
@@ -34,14 +34,13 @@ const importNationality = defineModel<string | null | undefined>(
 );
 const trainingPlace = defineModel<string | null | undefined>('trainingPlace');
 const checkpoint = defineModel<string | null | undefined>('checkpoint');
-const agencyFile = defineModel<File[]>('agencyFile');
-const agencyFileList =
-  defineModel<{ name: string; url: string }[]>('agencyFileList');
+const userFile = defineModel<File[]>('userFile');
+const userFileList =
+  defineModel<{ name: string; url: string }[]>('userFileList');
 const remark = defineModel<string | null | undefined>('remark');
 const agencyStatus = defineModel<string | null | undefined>('agencyStatus');
 
 const attachmentRef = ref();
-const checkpointENOption = ref([]);
 
 defineProps<{
   dense?: boolean;
@@ -71,18 +70,12 @@ function deleteFile(name: string) {
       userStore.deleteAttachment(userId.value, payload);
       const result = await userStore.fetchAttachment(userId.value);
       if (result) {
-        agencyFileList.value = result;
+        userFileList.value = result;
       }
     },
     cancel: () => {},
   });
 }
-
-onMounted(async () => {
-  const resultOption = await fetch('/option/option.json');
-  const rawOption = await resultOption.json();
-  checkpointENOption.value = rawOption.eng.border;
-});
 </script>
 <template>
   <div class="row col-12">
@@ -140,11 +133,12 @@ onMounted(async () => {
         />
 
         <SelectOffice
+          v-if="userType === 'MESSENGER'"
           for="input-responsible-area"
           v-model:value="responsibleArea"
-          v-if="userType === 'MESSENGER'"
           :readonly="readonly"
           :label="$t('personnel.form.responsibleArea')"
+          class="col"
         />
       </div>
       <div
@@ -198,25 +192,13 @@ onMounted(async () => {
             (v) => (typeof v === 'string' ? (importNationality = v) : '')
           "
         />
-        <SelectInput
-          :model-value="readonly ? trainingPlace || '-' : trainingPlace"
-          id="select-trainig-place"
-          for="select-trainig-place"
-          :option="optionStore.globalOption.training"
-          class="col-md-6 col-12"
-          :readonly
-          :label="$t('personnel.form.trainingPlace')"
-          clearable
-          @update:model-value="
-            (v) => (typeof v === 'string' ? (trainingPlace = v) : '')
-          "
-        />
+
         <SelectInput
           :model-value="readonly ? checkpoint || '-' : checkpoint"
           id="select-checkpoint"
           for="select-checkpoint"
           :option="optionStore.globalOption.border"
-          class="col-6"
+          class="col-md-6 col-12"
           :readonly
           :label="$t('personnel.form.checkpoint')"
           clearable
@@ -224,17 +206,18 @@ onMounted(async () => {
             (v) => (typeof v === 'string' ? (checkpoint = v) : '')
           "
         />
+
         <SelectInput
-          :model-value="readonly ? checkpoint || '-' : checkpoint"
-          id="select-checkpoint-en"
-          for="select-checkpoint-en"
-          :option="checkpointENOption"
-          class="col-6"
+          :model-value="readonly ? trainingPlace || '-' : trainingPlace"
+          id="select-trainig-place"
+          for="select-trainig-place"
+          :option="optionStore.globalOption.training"
+          class="col-md-8 col-12"
           :readonly
-          :label="$t('personnel.form.checkpointEN')"
+          :label="$t('personnel.form.trainingPlace')"
           clearable
           @update:model-value="
-            (v) => (typeof v === 'string' ? (checkpoint = v) : '')
+            (v) => (typeof v === 'string' ? (trainingPlace = v) : '')
           "
         />
 
@@ -253,7 +236,7 @@ onMounted(async () => {
               value: AgencyStatus.Blacklist,
             },
           ]"
-          class="col-md-6 col-12"
+          class="col-md-4 col-12"
           :readonly
           :label="$t('personnel.form.agencyStatus')"
           clearable
@@ -275,76 +258,77 @@ onMounted(async () => {
           "
           @clear="remark = ''"
         />
-        <q-file
-          ref="attachmentRef"
-          for="input-attachment"
-          :dense="dense"
-          outlined
-          :readonly="readonly"
-          multiple
-          append
-          :label="$t('personnel.form.attachment')"
-          class="col"
-          v-model="agencyFile"
-        >
-          <template v-slot:prepend>
-            <Icon
-              icon="material-symbols:attach-file"
-              width="20px"
-              style="color: var(--brand-1)"
-            />
-          </template>
-          <template v-slot:file="file">
-            <div class="row full-width items-center">
-              <span class="col ellipsis">
-                {{ file.file.name }}
-              </span>
-              <q-btn
-                dense
-                rounded
-                flat
-                padding="2 2"
-                class="app-text-muted"
-                icon="mdi-close-circle"
-                @click.stop="attachmentRef.removeAtIndex(file.index)"
-              />
-            </div>
-          </template>
-        </q-file>
+      </div>
 
-        <div v-if="agencyFileList && agencyFileList?.length > 0" class="col-12">
-          <q-list bordered separator class="rounded" style="padding: 0">
-            <q-item
-              id="attachment-file"
-              for="attachment-file"
-              v-for="item in agencyFileList"
-              clickable
-              :key="item.url"
-              class="items-center row"
-              @click="() => openNewTab(item.url)"
-            >
-              <q-item-section>
-                <div class="row items-center justify-between">
-                  <div class="col">
-                    {{ item.name }}
-                  </div>
-                  <q-btn
-                    id="delete-file"
-                    v-if="!readonly && userId"
-                    rounded
-                    flat
-                    dense
-                    unelevated
-                    size="md"
-                    icon="mdi-trash-can-outline"
-                    class="app-text-negative"
-                    @click.stop="deleteFile(item.name)"
-                  />
+      <q-file
+        ref="attachmentRef"
+        for="input-attachment"
+        :dense="dense"
+        outlined
+        :readonly="readonly"
+        multiple
+        append
+        :label="$t('personnel.form.attachment')"
+        class="col"
+        v-model="userFile"
+      >
+        <template v-slot:prepend>
+          <Icon
+            icon="material-symbols:attach-file"
+            width="20px"
+            style="color: var(--brand-1)"
+          />
+        </template>
+        <template v-slot:file="file">
+          <div class="row full-width items-center">
+            <span class="col ellipsis">
+              {{ file.file.name }}
+            </span>
+            <q-btn
+              dense
+              rounded
+              flat
+              padding="2 2"
+              class="app-text-muted"
+              icon="mdi-close-circle"
+              @click.stop="attachmentRef.removeAtIndex(file.index)"
+            />
+          </div>
+        </template>
+      </q-file>
+
+      <div v-if="userFileList && userFileList?.length > 0" class="col-12">
+        <q-list bordered separator class="rounded" style="padding: 0">
+          <q-item
+            id="attachment-file"
+            for="attachment-file"
+            v-for="item in userFileList"
+            clickable
+            :key="item.url"
+            class="items-center row"
+            @click="() => openNewTab(item.url)"
+          >
+            <q-item-section>
+              <div class="row items-center justify-between">
+                <div class="col">
+                  {{ item.name }}
                 </div>
-              </q-item-section>
-            </q-item>
-          </q-list>
-        </div>
+                <q-btn
+                  id="delete-file"
+                  v-if="!readonly && userId"
+                  rounded
+                  flat
+                  dense
+                  unelevated
+                  size="md"
+                  icon="mdi-trash-can-outline"
+                  class="app-text-negative"
+                  @click.stop="deleteFile(item.name)"
+                />
+              </div>
+            </q-item-section>
+          </q-item>
+        </q-list>
       </div>
     </div>
   </div>
diff --git a/src/pages/02_personnel-management/MainPage.vue b/src/pages/02_personnel-management/MainPage.vue
index b580508c..f71b2ec1 100644
--- a/src/pages/02_personnel-management/MainPage.vue
+++ b/src/pages/02_personnel-management/MainPage.vue
@@ -93,8 +93,8 @@ const currentUser = ref<User>();
 const userCode = ref<string>();
 
 const statusToggle = ref(true);
-const agencyFile = ref<File[]>([]);
-const agencyFileList = ref<{ name: string; url: string }[]>([]);
+const userFile = ref<File[]>([]);
+const userFileList = ref<{ name: string; url: string }[]>([]);
 
 const typeStats = ref<UserTypeStats>();
 const userStats = ref<BranchUserStats[]>();
@@ -152,7 +152,7 @@ const defaultFormData = {
   citizenIssue: null,
   citizenId: '',
   remark: '',
-  agencyStatus: null,
+  agencyStatus: '',
 };
 
 const formData = ref<UserCreate>({
@@ -202,7 +202,7 @@ const formData = ref<UserCreate>({
   citizenIssue: null,
   citizenId: '',
   remark: '',
-  agencyStatus: null,
+  agencyStatus: '',
 });
 
 const fieldSelectedOption = ref<{ label: string; value: string }[]>([
@@ -331,7 +331,7 @@ function onClose(excludeDialog?: boolean) {
   urlProfile.value = '';
   profileFileImg.value = null;
   infoDrawerEdit.value = false;
-  agencyFile.value = [];
+  userFile.value = [];
   isEdit.value = false;
   statusToggle.value = true;
   isImageEdit.value = false;
@@ -340,6 +340,8 @@ function onClose(excludeDialog?: boolean) {
   mapUserType(currentTab.value);
   imageList.value = { selectedImage: '', list: [] };
   onCreateImageList.value = { selectedImage: '', list: [] };
+  userFileList.value = [];
+  userFile.value = [];
   flowStore.rotate();
 }
 
@@ -360,12 +362,10 @@ async function openDialog(
     isEdit.value = true;
     await assignFormData(id);
 
-    if (formData.value.userType === 'AGENCY') {
-      const result = await userStore.fetchAttachment(id);
+    const result = await userStore.fetchAttachment(id);
 
-      if (result) {
-        agencyFileList.value = result;
-      }
+    if (result) {
+      userFileList.value = result;
     }
   }
   if (userStore.userOption.hqOpts.length !== 0 && !id) {
@@ -429,10 +429,9 @@ async function onSubmit(excludeDialog?: boolean) {
 
     await userStore.editById(currentUser.value.id, formDataEdit);
 
-    if (currentUser.value.id && formDataEdit.userType === 'AGENCY') {
-      if (!agencyFile.value) return;
+    if (userFile.value) {
       const payload: UserAttachmentCreate = {
-        file: agencyFile.value,
+        file: userFile.value,
       };
 
       if (payload?.file) {
@@ -461,11 +460,9 @@ async function onSubmit(excludeDialog?: boolean) {
       onCreateImageList.value,
     );
 
-    if (result && formData.value.userType === 'AGENCY') {
-      if (!agencyFile.value) return;
-
+    if (userFile.value && result) {
       const payload: UserAttachmentCreate = {
-        file: agencyFile.value,
+        file: userFile.value,
       };
 
       if (payload?.file) {
@@ -747,7 +744,7 @@ watch(
     formData.value.trainingPlace = null;
     formData.value.checkpoint = null;
     formData.value.checkpointEN = null;
-    agencyFile.value = [];
+    userFile.value = [];
   },
 );
 
@@ -1800,8 +1797,8 @@ watch(
               v-model:import-nationality="formData.importNationality"
               v-model:training-place="formData.trainingPlace"
               v-model:checkpoint="formData.checkpoint"
-              v-model:agency-file="agencyFile"
-              v-model:agency-file-list="agencyFileList"
+              v-model:user-file="userFile"
+              v-model:user-file-list="userFileList"
               v-model:user-id="currentUser.id"
               v-model:remark="formData.remark"
               v-model:agency-status="formData.agencyStatus"
@@ -2025,7 +2022,8 @@ watch(
             v-model:checkpoint="formData.checkpoint"
             v-model:agency-status="formData.agencyStatus"
             v-model:remark="formData.remark"
-            v-model:agency-file="agencyFile"
+            v-model:user-file="userFile"
+            v-model:user-file-list="userFileList"
           />
         </div>
       </div>