Merge pull request #1115 from mulderr/master

use uuid type for uuids by default, let the dialect specify whats best - fixes #1112

lib/data-types.js

@@ -188,7 +188,7 @@ module.exports = {
   FLOAT: FLOAT,
   NOW: 'NOW',
   BLOB: BLOB,
-  UUID: 'CHAR(36)',
+  UUID: 'UUID',
   UUIDV1: 'UUIDV1',
   UUIDV4: 'UUIDV4',
 

lib/dialects/mysql/query-generator.js

@@ -50,7 +50,7 @@ module.exports = (function() {
 
       for (var attr in attributes) {
         if (attributes.hasOwnProperty(attr)) {
-          var dataType = attributes[attr]
+          var dataType = this.mysqlDataTypeMapping(tableName, attr, attributes[attr])
 
           if (Utils._.includes(dataType, 'PRIMARY KEY')) {
             primaryKeys.push(attr)
@@ -117,7 +117,7 @@ module.exports = (function() {
 
         attrString.push(Utils._.template('`<%= attrName %>` <%= definition %>')({
           attrName: attrName,
-          definition: definition
+          definition: this.mysqlDataTypeMapping(tableName, attrName, definition)
         }))
       }
 
@@ -496,6 +496,14 @@ module.exports = (function() {
      */
     dropForeignKeyQuery: function(tableName, foreignKey) {
       return 'ALTER TABLE ' + this.quoteIdentifier(tableName) + ' DROP FOREIGN KEY ' + this.quoteIdentifier(foreignKey) + ';'
+    },
+
+    mysqlDataTypeMapping: function(tableName, attr, dataType) {
+      if (Utils._.includes(dataType, 'UUID')) {
+        dataType = dataType.replace(/UUID/, 'CHAR(36) BINARY')
+      }
+
+      return dataType
     }
   }
 

test/data-types.test.js

@@ -25,7 +25,7 @@ describe(Support.getTestDialectTeaser('DataTypes'), function() {
     [Sequelize.TEXT, 'TEXT', 'TEXT'],
     [Sequelize.DATE, 'DATE', 'DATETIME'],
     [Sequelize.NOW, 'NOW', 'NOW'],
-    [Sequelize.UUID, 'UUID', 'CHAR(36)'],
+    [Sequelize.UUID, 'UUID', 'UUID'],
     [Sequelize.BOOLEAN, 'BOOLEAN', 'TINYINT(1)'],
 
     [Sequelize.BLOB, 'BLOB', 'BLOB'],