From 48601c27b7942a18482bc817321a1ce9effece20 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rafael=20Mendon=C3=A7a=20Fran=C3=A7a?=
 <rafaelmfranca@gmail.com>
Date: Wed, 27 Jun 2012 11:20:23 -0700
Subject: Merge pull request #6878 from masarakki/master

fix bug in limit of enum columns of mysql

Closes #6432
---
 .../connection_adapters/abstract_mysql_adapter.rb              |  2 ++
 activerecord/test/cases/adapters/mysql/enum_test.rb            | 10 ++++++++++
 activerecord/test/cases/adapters/mysql2/enum_test.rb           | 10 ++++++++++
 activerecord/test/schema/mysql2_specific_schema.rb             |  9 +++++++++
 activerecord/test/schema/mysql_specific_schema.rb              | 10 ++++++++++
 5 files changed, 41 insertions(+)
 create mode 100644 activerecord/test/cases/adapters/mysql/enum_test.rb
 create mode 100644 activerecord/test/cases/adapters/mysql2/enum_test.rb

(limited to 'activerecord')

diff --git a/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb b/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb
index 88f82b5e35..e3cda4bf88 100644
--- a/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb
+++ b/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb
@@ -72,6 +72,8 @@ module ActiveRecord
           when /^mediumint/i; 3
           when /^smallint/i;  2
           when /^tinyint/i;   1
+          when /^enum\((.+)\)/i
+            $1.split(',').map{|enum| enum.strip.length - 2}.max
           else
             super
           end
diff --git a/activerecord/test/cases/adapters/mysql/enum_test.rb b/activerecord/test/cases/adapters/mysql/enum_test.rb
new file mode 100644
index 0000000000..40af317ad1
--- /dev/null
+++ b/activerecord/test/cases/adapters/mysql/enum_test.rb
@@ -0,0 +1,10 @@
+require "cases/helper"
+
+class MysqlEnumTest < ActiveRecord::TestCase
+  class EnumTest < ActiveRecord::Base
+  end
+
+  def test_enum_limit
+    assert_equal 5, EnumTest.columns.first.limit
+  end
+end
diff --git a/activerecord/test/cases/adapters/mysql2/enum_test.rb b/activerecord/test/cases/adapters/mysql2/enum_test.rb
new file mode 100644
index 0000000000..f3a05e48ad
--- /dev/null
+++ b/activerecord/test/cases/adapters/mysql2/enum_test.rb
@@ -0,0 +1,10 @@
+require "cases/helper"
+
+class Mysql2EnumTest < ActiveRecord::TestCase
+  class EnumTest < ActiveRecord::Base
+  end
+
+  def test_enum_limit
+    assert_equal 5, EnumTest.columns.first.limit
+  end
+end
diff --git a/activerecord/test/schema/mysql2_specific_schema.rb b/activerecord/test/schema/mysql2_specific_schema.rb
index 65b6f9f227..24a43d7ece 100644
--- a/activerecord/test/schema/mysql2_specific_schema.rb
+++ b/activerecord/test/schema/mysql2_specific_schema.rb
@@ -32,4 +32,13 @@ CREATE TABLE collation_tests (
 ) CHARACTER SET utf8 COLLATE utf8_general_ci
 SQL
 
+  ActiveRecord::Base.connection.execute <<-SQL
+DROP TABLE IF EXISTS enum_tests;
+SQL
+
+  ActiveRecord::Base.connection.execute <<-SQL
+CREATE TABLE enum_tests (
+  enum_column ENUM('true','false')
+)
+SQL
 end
diff --git a/activerecord/test/schema/mysql_specific_schema.rb b/activerecord/test/schema/mysql_specific_schema.rb
index 7d324f98c4..802c08b819 100644
--- a/activerecord/test/schema/mysql_specific_schema.rb
+++ b/activerecord/test/schema/mysql_specific_schema.rb
@@ -41,6 +41,16 @@ CREATE TABLE collation_tests (
   string_cs_column VARCHAR(1) COLLATE utf8_bin,
   string_ci_column VARCHAR(1) COLLATE utf8_general_ci
 ) CHARACTER SET utf8 COLLATE utf8_general_ci
+SQL
+
+  ActiveRecord::Base.connection.execute <<-SQL
+DROP TABLE IF EXISTS enum_tests;
+SQL
+
+  ActiveRecord::Base.connection.execute <<-SQL
+CREATE TABLE enum_tests (
+  enum_column ENUM('true','false')
+)
 SQL
 
 end
-- 
cgit v1.2.3