Merge pull request #500 from jcoleman/fix_incorrect_typecasting_of_raw_sql_strings

Type-castable attributes should not try to cast SqlLiteral nodes

Author: sgrif

lib/arel/nodes/casted.rb

@@ -30,7 +30,7 @@ def nil?; val.nil?; end
 
     def self.build_quoted other, attribute = nil
       case other
-        when Arel::Nodes::Node, Arel::Attributes::Attribute, Arel::Table, Arel::Nodes::BindParam, Arel::SelectManager, Arel::Nodes::Quoted
+        when Arel::Nodes::Node, Arel::Attributes::Attribute, Arel::Table, Arel::Nodes::BindParam, Arel::SelectManager, Arel::Nodes::Quoted, Arel::Nodes::SqlLiteral
           other
         else
           case attribute

test/attributes/test_attribute.rb

@@ -997,6 +997,18 @@ def fake_caster.type_cast_for_database(attr_name, value)
         assert table.able_to_type_cast?
         condition.to_sql.must_equal %("foo"."id" = 1 AND "foo"."other_id" = '2')
       end
+
+      it 'does not type cast SqlLiteral nodes' do
+        fake_caster = Object.new
+        def fake_caster.type_cast_for_database(attr_name, value)
+          value.to_i
+        end
+        table = Table.new(:foo, type_caster: fake_caster)
+        condition = table["id"].eq(Arel.sql("(select 1)"))
+
+        assert table.able_to_type_cast?
+        condition.to_sql.must_equal %("foo"."id" = (select 1))
+      end
     end
   end
 end