@things-factory/product-ui 4.3.738 → 4.3.741

package/client/pages/product-list.js

@@ -1086,35 +1086,62 @@ class ProductList extends localize(i18next)(PageView) {
           patch.isRequireSerialNumberScanningOutbound?.toString().toLowerCase() === 'true'
         patch.isInventoryDecimal = patch.isInventoryDecimal?.toString().toLowerCase() === 'true'
         patch.allowStackingOption = patch.allowStackingOption?.toString().toLowerCase() === 'true'
-        return {
+
+        // Helper function to safely parse integers, handling empty strings
+        const safeParseInt = val => {
+          if (val === null || val === undefined || val === '') return undefined
+          const parsed = parseInt(val)
+          return isNaN(parsed) ? undefined : parsed
+        }
+
+        // Helper function to safely parse floats, handling empty strings
+        const safeParseFloat = val => {
+          if (val === null || val === undefined || val === '') return undefined
+          const parsed = parseFloat(val)
+          return isNaN(parsed) ? undefined : parsed
+        }
+
+        const result = {
           ...patch,
           costPrice: patch?.costPrice ? parseFloat(patch.costPrice) : undefined,
-          uomValue: patch?.uomValue ? parseFloat(patch.uomValue) : undefined,
-          bufferQty: patch?.bufferQty ? parseInt(patch.bufferQty) : undefined,
-          minQty: patch?.minQty ? parseInt(patch.minQty) : undefined,
-          maxQty: patch?.maxQty ? parseInt(patch.maxQty) : undefined,
-          packingSize: patch?.packingSize ? parseInt(patch.packingSize) : undefined,
-          minInboundShelfLife: patch?.minInboundShelfLife ? parseInt(patch.minInboundShelfLife) : undefined,
-          minOutboundShelfLife: patch?.minOutboundShelfLife ? parseInt(patch.minOutboundShelfLife) : undefined,
+          uomValue: safeParseFloat(patch?.uomValue),
+          bufferQty: safeParseInt(patch?.bufferQty),
+          minQty: safeParseInt(patch?.minQty),
+          maxQty: safeParseInt(patch?.maxQty),
+          packingSize: safeParseInt(patch?.packingSize),
+          minInboundShelfLife: safeParseInt(patch?.minInboundShelfLife),
+          minOutboundShelfLife: safeParseInt(patch?.minOutboundShelfLife),
+          outboundAlert: safeParseInt(patch?.outboundAlert),
           productRef: patch.productRef?.id ? { id: patch.productRef?.id } : undefined,
           pickingStrategy: patch?.pickingStrategy ? patch.pickingStrategy.toUpperCase() : undefined,
           gtin: patch?.gtin ? patch.gtin : patch.sku,
-          commissionFee: patch?.commissionFee ? parseFloat(patch.commissionFee) : undefined,
-          platformFee: patch?.platformFee ? parseFloat(patch.platformFee) : undefined,
-          serviceFee: patch?.serviceFee ? parseFloat(patch.serviceFee) : undefined,
-          transactionFee: patch?.transactionFee ? parseFloat(patch.transactionFee) : undefined,
-          nettWeight: patch?.nettWeight ? parseFloat(patch.nettWeight) : undefined,
-          width: patch?.width ? parseFloat(patch.width) : undefined,
-          depth: patch?.depth ? parseFloat(patch.depth) : undefined,
-          height: patch?.height ? parseFloat(patch.height) : undefined,
-          grossWeight: patch?.grossWeight ? parseFloat(patch.grossWeight) : undefined,
-          volume: patch?.volume ? parseFloat(patch.volume) : undefined,
+          commissionFee: safeParseFloat(patch?.commissionFee),
+          platformFee: safeParseFloat(patch?.platformFee),
+          serviceFee: safeParseFloat(patch?.serviceFee),
+          transactionFee: safeParseFloat(patch?.transactionFee),
+          nettWeight: safeParseFloat(patch?.nettWeight),
+          width: safeParseFloat(patch?.width),
+          depth: safeParseFloat(patch?.depth),
+          height: safeParseFloat(patch?.height),
+          grossWeight: safeParseFloat(patch?.grossWeight),
+          volume: safeParseFloat(patch?.volume),
+          volumeSize: safeParseFloat(patch?.volumeSize),
+          density: safeParseFloat(patch?.density),
           isRequiredCheckExpiry: patch?.isRequiredCheckExpiry || false,
           isRequireSerialNumberScanningInbound: patch?.isRequireSerialNumberScanningInbound || false,
           isRequireSerialNumberScanningOutbound: patch?.isRequireSerialNumberScanningOutbound || false,
           isInventoryDecimal: patch?.isInventoryDecimal || false,
           allowStackingOption: patch?.allowStackingOption || false
         }
+
+        // Remove any remaining empty strings from numeric fields to prevent GraphQL errors
+        Object.keys(result).forEach(key => {
+          if (result[key] === '') {
+            delete result[key]
+          }
+        })
+
+        return result
       })
 
       this._validate(patches)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@things-factory/product-ui",
-  "version": "4.3.738",
+  "version": "4.3.741",
   "main": "dist-server/index.js",
   "browser": "client/index.js",
   "things-factory": true,
@@ -23,13 +23,13 @@
     "migration:create": "node ../../node_modules/typeorm/cli.js migration:create -d ./server/migrations"
   },
   "dependencies": {
-    "@things-factory/form-ui": "^4.3.738",
-    "@things-factory/grist-ui": "^4.3.738",
-    "@things-factory/i18n-base": "^4.3.738",
-    "@things-factory/import-ui": "^4.3.738",
-    "@things-factory/layout-base": "^4.3.738",
-    "@things-factory/product-base": "^4.3.738",
-    "@things-factory/shell": "^4.3.738"
+    "@things-factory/form-ui": "^4.3.740",
+    "@things-factory/grist-ui": "^4.3.740",
+    "@things-factory/i18n-base": "^4.3.740",
+    "@things-factory/import-ui": "^4.3.740",
+    "@things-factory/layout-base": "^4.3.740",
+    "@things-factory/product-base": "^4.3.740",
+    "@things-factory/shell": "^4.3.740"
   },
-  "gitHead": "4c0363d0c8fa24d07a827b9c884ad32755f3ee8c"
+  "gitHead": "97de9f3e63d7acdb0e59eff61f5c170457582490"
 }