From 5df5c96f60b102c5caa908b28d45908d369efda6 Mon Sep 17 00:00:00 2001
From: k2maan <ketuman333@gmail.com>
Date: Tue, 29 Aug 2023 12:22:10 +0530
Subject: [PATCH] Improved: naming and added checks when no data is returned
 from the modal (#237)

---
 src/components/EditPickersModal.vue | 15 ++++++---------
 src/views/InProgress.vue            |  4 ++--
 2 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/src/components/EditPickersModal.vue b/src/components/EditPickersModal.vue
index 7494549c..b007145f 100644
--- a/src/components/EditPickersModal.vue
+++ b/src/components/EditPickersModal.vue
@@ -15,7 +15,7 @@
     <ion-row>
       <ion-chip v-for="picker in selectedPickers" :key="picker.id">
         <ion-label>{{ picker.name }}</ion-label>
-        <ion-icon :icon="closeCircle" @click="removePicker(picker.id)" />
+        <ion-icon :icon="closeCircle" @click="updateSelectedPickers(picker.id)" />
       </ion-chip>
     </ion-row>
 
@@ -25,7 +25,7 @@
         {{ 'No picker found' }}
       </div>
       <div v-else>
-        <ion-item v-for="(picker, index) in pickers" :key="index" @click="selectPicker(picker.id)">
+        <ion-item v-for="(picker, index) in pickers" :key="index" @click="updateSelectedPickers(picker.id)">
           <ion-label>
             {{ picker.name }}
             <p>{{ picker.externalId ? picker.externalId : picker.id }}</p>
@@ -35,7 +35,7 @@
       </div>
     </ion-list>
     <ion-fab vertical="bottom" horizontal="end" slot="fixed">
-      <ion-fab-button :disabled="evaluateSelectedPickers()" @click="confirmSave()">
+      <ion-fab-button :disabled="arePickersSelected()" @click="confirmSave()">
         <ion-icon :icon="saveOutline" />
       </ion-fab-button>
     </ion-fab>
@@ -109,8 +109,8 @@ export default defineComponent({
     isPickerSelected(id: string) {
       return this.selectedPickers.some((picker: any) => picker.id == id)
     },
-    selectPicker(id: string) {
-      const picker = this.selectedPickers.some((picker: any) => picker.id == id)
+    updateSelectedPickers(id: string) {
+      const picker = this.isPickerSelected(id)
       if (picker) {
         // if picker is already selected then removing that picker from the list on click
         this.selectedPickers = this.selectedPickers.filter((picker: any) => picker.id != id)
@@ -118,9 +118,6 @@ export default defineComponent({
         this.selectedPickers.push(this.pickers.find((picker: any) => picker.id == id))
       }
     },
-    removePicker(id: string) {
-      this.selectedPickers = this.selectedPickers.filter((picker: any) => picker.id != id)
-    },
     async findPickers() {
       let inputFields = {}
       this.pickers = []
@@ -237,7 +234,7 @@ export default defineComponent({
         logger.error('Something went wrong, could not edit picker(s)')
       }
     },
-    evaluateSelectedPickers() {
+    arePickersSelected() {
       // disable the save button if only 'System Generate' entry is there
       // or if no pickers are selected
       return (this.selectedPickers.length === 1
diff --git a/src/views/InProgress.vue b/src/views/InProgress.vue
index dfc1eaef..d2a9bc2e 100644
--- a/src/views/InProgress.vue
+++ b/src/views/InProgress.vue
@@ -171,7 +171,7 @@
       </div>
     </ion-content>
     <!-- only show footer buttons if 'All orders' is not selected -->
-    <ion-footer v-if="selectedPicklistId.length">
+    <ion-footer v-if="selectedPicklistId">
       <ion-toolbar>
         <ion-buttons slot="end">
           <ion-button fill="outline" color="primary" @click="editPickers(getPicklist(selectedPicklistId))">
@@ -911,7 +911,7 @@ export default defineComponent({
       editPickersModal.onDidDismiss().then((result) => {
         // manually updating the picklist data as UI is not updated because same data
         // is returned from solr on fetchPickersInformation API call
-        if (result.data?.editedPicklist) {
+        if (result.data?.editedPicklist && Object.keys(result.data?.editedPicklist).length) {
           const editedPicklist = result.data.editedPicklist
           this.picklists = JSON.parse(JSON.stringify(this.picklists.map((picklist: any) => picklist.id === editedPicklist.id ? picklist = editedPicklist : picklist)))
         }