Merge pull request #4239 from frantzmiccoli/string-validation-fix

String validation fix

lib/data-types.js

@@ -83,7 +83,7 @@ STRING.prototype.toSql = function() {
   return 'VARCHAR(' + this._length + ')' + ((this._binary) ? ' BINARY' : '');
   return 'VARCHAR(' + this._length + ')' + ((this._binary) ? ' BINARY' : '');
 };
 };
 STRING.prototype.validate = function(value) {
 STRING.prototype.validate = function(value) {
-  if (typeof value !== 'string') {
+  if (Object.prototype.toString.call(value) !== '[object String]') {
     if (this.options.binary) {
     if (this.options.binary) {
       if (Buffer.isBuffer(value)) {
       if (Buffer.isBuffer(value)) {
         return true;
         return true;

test/unit/sql/data-types.test.js

@@ -63,6 +63,9 @@ suite(Support.getTestDialectTeaser('SQL'), function() {
           var type = DataTypes.STRING();
           var type = DataTypes.STRING();
 
 
           expect(type.validate('foobar')).to.equal(true);
           expect(type.validate('foobar')).to.equal(true);
+          /*jshint -W053 */
+          expect(type.validate(new String('foobar'))).to.equal(true);
+          /*jshint +W053 */
         });
         });
       });
       });
     });
     });