Merge pull request #351 from sushantdhiman/fix-347

enabled use of global typeCast

Author: sidorares

lib/compile_text_parser.js

@@ -35,6 +35,11 @@ function compile (fields, options, config) {
     result.push('  var result = new Array(' + fields.length + ')');
   }
 
+  // use global typeCast if current query doesn't specify one
+  if (typeof config.typeCast === 'function' && typeof options.typeCast !== 'function') {
+    options.typeCast = config.typeCast;
+  }
+
   if (typeof options.typeCast === 'function') {
     result.push('  var wrap = ' + wrap.toString());
   }

test/common.js

@@ -72,7 +72,8 @@ module.exports.createConnection = function (args, callback) {
     compress: (args && args.compress) || config.compress,
     decimalNumbers: args && args.decimalNumbers,
     dateStrings: args && args.dateStrings,
-    authSwitchHandler: args && args.authSwitchHandler
+    authSwitchHandler: args && args.authSwitchHandler,
+    typeCast: args && args.typeCast
   };
 
   //console.log('cc params', params);

test/integration/connection/test-typecast-global-option.js

@@ -0,0 +1,34 @@
+var typeCastWrapper = function (stringMethod) {
+  return function (field, next) {
+    if (field.type == 'VAR_STRING') {
+      return field.string()[stringMethod]();
+    }
+    return next();
+  };
+};
+
+var common = require('../../common');
+var connection = common.createConnection({
+  typeCast: typeCastWrapper('toUpperCase')
+});
+
+var assert = require('assert');
+
+// query option override global typeCast
+connection.query({
+  sql: 'select "FOOBAR" as foo',
+  typeCast: typeCastWrapper('toLowerCase')
+}, function(err, res) {
+  assert.ifError(err);
+  assert.equal(res[0].foo, 'foobar')
+});
+
+// global typecast works
+connection.query({
+  sql: 'select "foobar" as foo',
+}, function(err, res) {
+  assert.ifError(err);
+  assert.equal(res[0].foo, 'FOOBAR');
+});
+
+connection.end();