Remove deprecated #quote_bound_value

Author: rafaelfranca

activerecord/CHANGELOG.md

@@ -1,3 +1,7 @@
+*   Remove deprecated `#quote_bound_value`.
+
+    *Rafael Mendonça França*
+
 *   Remove deprecated `ActiveRecord::ConnectionAdapters::ConnectionPool#connection_klass`.
 
     *Rafael Mendonça França*

activerecord/lib/active_record/connection_adapters/abstract/quoting.rb

@@ -48,21 +48,6 @@ def type_cast(value)
         end
       end
 
-      # Quote a value to be used as a bound parameter of unknown type. For example,
-      # MySQL might perform dangerous castings when comparing a string to a number,
-      # so this method will cast numbers to string.
-      #
-      # Deprecated: Consider `Arel.sql("... ? ...", value)` or
-      # +sanitize_sql+ instead.
-      def quote_bound_value(value)
-        ActiveRecord.deprecator.warn(<<~MSG.squish)
-          #quote_bound_value is deprecated and will be removed in Rails 7.2.
-          Consider Arel.sql(".. ? ..", value) or #sanitize_sql instead.
-        MSG
-
-        quote(cast_bound_value(value))
-      end
-
       # Cast a value to be used as a bound parameter of unknown type. For example,
       # MySQL might perform dangerous castings when comparing a string to a number,
       # so this method will cast numbers to string.

activerecord/test/cases/adapters/abstract_mysql_adapter/quoting_test.rb

@@ -20,46 +20,11 @@ def test_cast_bound_rational
     assert_equal "0.75", @conn.cast_bound_value(Rational(3, 4))
   end
 
-  def test_cast_bound_duration
-    expected = assert_deprecated(ActiveRecord.deprecator) { @conn.cast_bound_value(42.seconds) }
-    assert_equal "42", expected
-  end
-
   def test_cast_bound_true
     assert_equal "1", @conn.cast_bound_value(true)
   end
 
   def test_cast_bound_false
     assert_equal "0", @conn.cast_bound_value(false)
   end
-
-  def test_quote_bound_integer
-    expected = assert_deprecated(ActiveRecord.deprecator) { @conn.quote_bound_value(42) }
-    assert_equal "'42'", expected
-  end
-
-  def test_quote_bound_big_decimal
-    expected = assert_deprecated(ActiveRecord.deprecator) { @conn.quote_bound_value(BigDecimal("4.2")) }
-    assert_equal "'4.2'", expected
-  end
-
-  def test_quote_bound_rational
-    expected = assert_deprecated(ActiveRecord.deprecator) { @conn.quote_bound_value(Rational(3, 4)) }
-    assert_equal "'0.75'", expected
-  end
-
-  def test_quote_bound_duration
-    expected = assert_deprecated(ActiveRecord.deprecator) { @conn.quote_bound_value(42.seconds) }
-    assert_equal "'42'", expected
-  end
-
-  def test_quote_bound_true
-    expected = assert_deprecated(ActiveRecord.deprecator) { @conn.quote_bound_value(true) }
-    assert_equal "'1'", expected
-  end
-
-  def test_quote_bound_false
-    expected = assert_deprecated(ActiveRecord.deprecator) { @conn.quote_bound_value(false) }
-    assert_equal "'0'", expected
-  end
 end

guides/source/7_2_release_notes.md

@@ -156,6 +156,8 @@ Please refer to the [Changelog][active-record] for detailed changes.
 
 *   Remove deprecated `ActiveRecord::ConnectionAdapters::ConnectionPool#connection_klass`.
 
+*   Remove deprecated `#quote_bound_value`.
+
 ### Deprecations
 
 *   Deprecate `Rails.application.config.active_record.allow_deprecated_singular_associations_name`