Fix parseInt & parseFloat result is NaN when value is not a number, fix

thinkjs/thinkjs#873
thinkjs · Sep 27, 2017 · c2dc66f · c2dc66f
1 parent f4851d9
commit c2dc66f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lib/schema.js b/lib/schema.js
@@ -84,8 +84,8 @@ module.exports = class MysqlSchema extends Schema {
    */
   parseType(tinyType, value) {
     if (tinyType === 'enum' || tinyType === 'set' || tinyType === 'bigint') return value;
-    if (tinyType.indexOf('int') > -1) return parseInt(value, 10);
-    if (['double', 'float', 'decimal'].indexOf(tinyType) > -1) return parseFloat(value, 10);
+    if (tinyType.indexOf('int') > -1) return parseInt(value, 10) || 0;
+    if (['double', 'float', 'decimal'].indexOf(tinyType) > -1) return parseFloat(value, 10) || 0;
     if (tinyType === 'bool') return value ? 1 : 0;
     return value;
   }

diff --git a/test/lib/schema.js b/test/lib/schema.js
@@ -337,8 +337,10 @@ test('schema parse type', t => {
   t.is(schema.parseType('set', 'True'), 'True');
   t.is(schema.parseType('bigint', 'False'), 'False');
   t.is(schema.parseType('int(10)', '3'), 3);
+  t.is(schema.parseType('int(10)', 'fasdfadf'), 0);
   t.is(schema.parseType('double', '3.3'), 3.3);
   t.is(schema.parseType('float', '3.3'), 3.3);
+  t.is(schema.parseType('float', 'fasdfasdf'), 0);
   t.is(schema.parseType('decimal', '3.3'), 3.3);
   t.is(schema.parseType('bool', '0'), 1);
   t.is(schema.parseType('bool', ''), 0);