Fix #11: Use null for blank integer value

Author: bhch

src/components/form.js

@@ -11,6 +11,9 @@ export function FormInput({label, help_text, error, inputRef, ...props}) {
     if (inputRef)
         props.ref = inputRef;
 
+    if (props.type === 'number' && props.value === null)
+        props.value = '';
+
     return (
         <div>
             {label && <label>{label}</label>}

src/data.js

@@ -18,7 +18,9 @@ export function getBlankObject(schema) {
             keys[key] = getBlankObject(value);
         else if (type === 'boolean')
             keys[key] = false;
-        else // string, numbe, integer, etc.
+        else if (type === 'integer' || type === 'number')
+            keys[key] = null;
+        else // string etc.
             keys[key] = '';
     }
 
@@ -41,7 +43,9 @@ export function getBlankArray(schema) {
         items.push(getBlankObject(schema.items));
     else if (type === 'boolean')
         items.push(false);
-    else // string, number, integer, etc.
+    else if (type === 'integer' || type === 'number')
+        items.push(null);
+    else // string, etc.
         items.push('');
 
     return items;
@@ -62,7 +66,9 @@ export function getBlankData(schema) {
         return getBlankObject(schema);
     else if (type === 'boolean')
         return false;
-    else // string, number, integer, etc.
+    else if (type === 'integer' || type === 'number')
+        return null;
+    else // string, etc.
         return '';
 }
 
@@ -86,6 +92,10 @@ function getSyncedArray(data, schema) {
         } else if (type === 'object') {
             newData[i] = getSyncedObject(item, schema.items);
         }
+        else {
+            if ((type === 'integer' || type === 'number') && item === '')
+                newData[i] = null;
+        }
     }
 
     return newData;
@@ -116,15 +126,21 @@ function getSyncedObject(data, schema) {
                 newData[key] = getSyncedObject({}, schemaValue);
             else if (type === 'boolean')
                 newData[key] = false;
+            else if (type === 'integer' || type === 'number')
+                newData[key] = null;
             else
                 newData[key] = '';
         } else {
             if (type === 'array')
                 newData[key] = getSyncedArray(data[key], schemaValue);
             else if (type === 'object')
                 newData[key] = getSyncedObject(data[key], schemaValue);
-            else
-                newData[key] = data[key];
+            else {
+                if ((type === 'integer' || type === 'number') && data[key] === '')
+                    newData[key] = null;
+                else
+                    newData[key] = data[key];
+            }
         }
 
     }

src/ui.js

@@ -16,7 +16,9 @@ function handleChange(e, fieldType, callback) {
 
     if (fieldType === 'number' || fieldType === 'integer') {
         value = value.trim();
-        if (value !== '' && !isNaN(Number(value)))
+        if (value === '')
+            value = null;
+        else if (!isNaN(Number(value)))
             value = Number(value);
     } else if (fieldType === 'boolean') {
         if (value === 'false' || value === false)