Permalink
Browse files

Merge pull request #11120 from awilliams/ar_mysql2_boolean_quoting

Fixes AR mysql2 adapter incorrectly casting boolean values
  • Loading branch information...
2 parents 754a373 + 41bd94b commit cb1d07e43926bcec95cb8b4a663ca9889173395a @senny senny committed Jul 16, 2013
View
@@ -1,3 +1,25 @@
+* Fix bug when using Mysql2 adapter where in some cases, boolean values were
+ being output in sql as `t` or `f` instead of `1` or `0`. Example:
+
+ class Model < ActiveRecord::Base
+ validates_uniqueness_of :boolean_col
+ end
+ Model.first.valid?
+
+ Previously generated sql:
+
+ SELECT 1 AS one FROM `models` WHERE
+ `models`.`boolean_col` = BINARY 'f' LIMIT 1
+
+ With fix:
+
+ SELECT 1 AS one FROM `models` WHERE
+ `models`.`boolean_col` = BINARY 0 LIMIT 1
+
+ Fixes: #11119
+
+ *Adam Williams*
+
* `change_column` for PostgreSQL adapter respects the `:array` option.
*Yves Senn*
@@ -272,6 +272,12 @@ def quoted_false
QUOTED_FALSE
end
+ def type_cast(value, column)
+ return super unless value == true || value == false
+
+ value ? 1 : 0
+ end
+
# REFERENTIAL INTEGRITY ====================================
def disable_referential_integrity(&block) #:nodoc:
@@ -160,12 +160,6 @@ def error_number(exception) # :nodoc:
# QUOTING ==================================================
- def type_cast(value, column)
- return super unless value == true || value == false
-
- value ? 1 : 0
- end
-
def quote_string(string) #:nodoc:
@connection.quote(string)
end
@@ -0,0 +1,25 @@
+require "cases/helper"
+
+module ActiveRecord
+ module ConnectionAdapters
+ class Mysql2Adapter
+ class QuotingTest < ActiveRecord::TestCase
+ def setup
+ @conn = ActiveRecord::Base.connection
+ end
+
+ def test_type_cast_true
+ c = Column.new(nil, 1, 'boolean')
+ assert_equal 1, @conn.type_cast(true, nil)
+ assert_equal 1, @conn.type_cast(true, c)
+ end
+
+ def test_type_cast_false
+ c = Column.new(nil, 1, 'boolean')
+ assert_equal 0, @conn.type_cast(false, nil)
+ assert_equal 0, @conn.type_cast(false, c)
+ end
+ end
+ end
+ end
+end

0 comments on commit cb1d07e

Please sign in to comment.