fix: if user passes integer/float as string for integer/float fields to the save data api - before throwing error we are trying to parse it

Author: yaroslav8765

custom/VisionAction.vue

@@ -830,19 +830,18 @@ async function saveData() {
     const saveResults = await Promise.all(saveTasks);
     const failedResult = saveResults.find(res => res?.ok === false || res?.error);
 
-    if (!failedResult) {
-      confirmDialog.value.close();
-      props.updateList();
-      props.clearCheckboxes?.();
-    } else if (failedResult.ok === false) {
-      adminforth.alert({
-        message: failedResult.error,
-        variant: 'danger',
-        timeout: 'unlimited',
+     if (failedResult && failedResult.ok === false) {
+      saveResults.filter(res => res?.ok === false).forEach(res => {
+        adminforth.alert({
+          message: res.error || t(errorText),
+          variant: 'danger',
+          timeout: 'unlimited',
+        });
+        console.error('Error saving data:', res.error);
       });
       isError.value = true;
       errorMessage.value = t(errorText);
-    } else {
+    } else if ( failedResult ) {
       console.error('Error saving data:', failedResult);
       isError.value = true;
       errorMessage.value = t(errorText);
@@ -852,6 +851,9 @@ async function saveData() {
     isError.value = true;
     errorMessage.value = t(errorText);
   } finally {
+    confirmDialog.value.close();
+    props.updateList();
+    props.clearCheckboxes?.();
     isLoading.value = false;
     isDataSaved.value = true;
     window.removeEventListener('beforeunload', beforeUnloadHandler);

index.ts

@@ -825,7 +825,11 @@ export default class  BulkAiFlowPlugin extends AdminForthPlugin {
             }
           }
           const primaryKeyColumn = this.resourceConfig.columns.find((col) => col.primaryKey);
+
           const decimalFieldsArray = this.resourceConfig.columns.filter(c => c.type === 'decimal').map(c => c.name);
+          const integerFieldsArray = this.resourceConfig.columns.filter(c => c.type === 'integer').map(c => c.name);
+          const floatFieldsArray = this.resourceConfig.columns.filter(c => c.type === 'float').map(c => c.name);
+
           const updates = selectedIds.map(async (ID, idx) => {
             const oldRecord = await this.adminforth.resource(this.resourceConfig.resourceId).get( [Filters.EQ(primaryKeyColumn.name, ID)] );
             for (const [key, value] of Object.entries(outputImageFields)) {
@@ -869,6 +873,21 @@ export default class  BulkAiFlowPlugin extends AdminForthPlugin {
                 }
               }
             }
+            if (integerFieldsArray.length > 0) {
+              for (const fieldName of integerFieldsArray) {
+                if (fieldsToUpdate[idx].hasOwnProperty(fieldName)) {
+                  fieldsToUpdate[idx][fieldName] = parseInt(fieldsToUpdate[idx][fieldName], 10);
+                }
+              }
+            }
+            if (floatFieldsArray.length > 0) {
+              for (const fieldName of floatFieldsArray) {
+                if (fieldsToUpdate[idx].hasOwnProperty(fieldName)) {
+                  fieldsToUpdate[idx][fieldName] = parseFloat(fieldsToUpdate[idx][fieldName]);
+                }
+              }
+            }
+
             const newRecord = {
               ...oldRecord,
               ...fieldsToUpdate[idx]
@@ -881,7 +900,11 @@ export default class  BulkAiFlowPlugin extends AdminForthPlugin {
               adminUser: adminUser,
             })
           });
-          await Promise.all(updates);
+          try {
+            await Promise.all(updates);
+          } catch (error) {
+            return { ok: false, error: `Error updating records, because of unprocesseble data for record ID ${selectedIds}` };
+          }
           return { ok: true };
         } else {
           return { ok: false, error: isAllowedToSave.error };