fix: Don't parse NaNs, but serialise them

47ng · Aug 31, 2023 · 4da7786 · 4da7786
1 parent 8a50113
commit 4da7786
Showing 1 changed file with 15 additions and 9 deletions.
diff --git a/src/lib/defs.ts b/src/lib/defs.ts
@@ -98,11 +98,14 @@ export const queryTypes: QueryTypeMap = {
     }
   },
   integer: {
-    parse: v => parseInt(v) || null,
-    serialize: v => { 
-      if (isNaN(v)) return null;
-      return Math.round(v).toFixed();
+    parse: v => {
+      const int = parseInt(v)
+      if (Number.isNaN(int)) {
+        return null
+      }
+      return int
     },
+    serialize: v => Math.round(v).toFixed(),
     withDefault(defaultValue) {
       return {
         ...this,
@@ -111,11 +114,14 @@ export const queryTypes: QueryTypeMap = {
     }
   },
   float: {
-    parse: v => parseFloat(v) || null,
-    serialize: v => { 
-      if (isNaN(v)) { return null }
-      return v.toString()
-    }
+    parse: v => {
+      const float = parseFloat(v)
+      if (Number.isNaN(float)) {
+        return null
+      }
+      return float
+    },
+    serialize: v => v.toString(),
     withDefault(defaultValue) {
       return {
         ...this,