Merge pull request #1427 from testn/fix-field-length

Fix field.length to be number
sidorares · Oct 17, 2021 · 29f2217 · 29f2217
2 parents 11dd45b + 0c7ab95
commit 29f2217
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/parsers/text_parser.js b/lib/parsers/text_parser.js
@@ -105,7 +105,7 @@ function compile(fields, options, config) {
       );
       parserFn(`this.wrap${i} = {
         type: ${helpers.srcEscape(typeNames[field.columnType])},
-        length: ${helpers.srcEscape(field.columnLength)},
+        length: ${field.columnLength},
         db: ${helpers.srcEscape(field.schema)},
         table: ${helpers.srcEscape(field.table)},
         name: ${helpers.srcEscape(field.name)},

diff --git a/test/integration/connection/test-typecast.js b/test/integration/connection/test-typecast.js
@@ -11,7 +11,9 @@ connection.query(
   {
     sql: 'select "foo uppercase" as foo',
     typeCast: function(field, next) {
+      assert.equal("number", typeof field.length);
       if (field.type === 'VAR_STRING') {
+
         return field.string().toUpperCase();
       }
       return next();