Merge pull request #16037 from sgrif/sg-money-quoting

Remove unneccessary special case for money in quoting

2 files changed, 4 insertions, 11 deletions

diff --git a/activerecord/lib/active_record/connection_adapters/postgresql/quoting.rb b/activerecord/lib/active_record/connection_adapters/postgresql/quoting.rb
index 7c485eb9d0..39b36731cd 100644
--- a/activerecord/lib/active_record/connection_adapters/postgresql/quoting.rb
+++ b/activerecord/lib/active_record/connection_adapters/postgresql/quoting.rb
@@ -34,13 +34,6 @@ module ActiveRecord
             else
               super
             end
-          when Numeric
-            if sql_type == 'money' || [:string, :text].include?(column.type)
-              # Not truly string input, so doesn't require (or allow) escape string syntax.
-              "'#{value}'"
-            else
-              super
-            end
           when String
             case sql_type
             when 'xml'   then "xml '#{quote_string(value)}'"
diff --git a/activerecord/lib/active_record/type/string.rb b/activerecord/lib/active_record/type/string.rb
index 14b03dcb2d..8cc533bc41 100644
--- a/activerecord/lib/active_record/type/string.rb
+++ b/activerecord/lib/active_record/type/string.rb
@@ -16,10 +16,10 @@ module ActiveRecord
       end
 
       def type_cast_for_database(value)
-        if value.is_a?(::String)
-          ::String.new(value)
-        else
-          super
+        case value
+        when ::Numeric then value.to_s
+        when ::String then ::String.new(value)
+        else super
         end
       end