Merge pull request #26689 from kamipo/deprecate_passing_name_to_indexes

Deprecate passing `name` to `indexes` like `tables`

activerecord/CHANGELOG.md

@@ -1,9 +1,13 @@
+*   Deprecate passing `name` to `indexes`.
+
+    *Ryuta Kamizono*
+
 *   Remove deprecated tasks: `db:test:clone`, `db:test:clone_schema`, `db:test:clone_structure`.
 
     *Rafel Mendonça França*
 
 *   Compare deserialized values for `PostgreSQL::OID::Hstore` types when
-    calling `ActiveRecord::Dirty#changed_in_place?`
+    calling `ActiveRecord::Dirty#changed_in_place?`.
 
     Fixes #27502.
 

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

@@ -69,7 +69,9 @@ def view_exists?(view_name)
       end
 
       # Returns an array of indexes for the given table.
-      # def indexes(table_name, name = nil) end
+      def indexes(table_name, name = nil)
+        raise NotImplementedError, "#indexes is not implemented"
+      end
 
       # Checks to see if an index exists on a table for a given index definition.
       #

activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb

@@ -367,6 +367,12 @@ def truncate(table_name, name = nil)
 
       # Returns an array of indexes for the given table.
       def indexes(table_name, name = nil) #:nodoc:
+        if name
+          ActiveSupport::Deprecation.warn(<<-MSG.squish)
+            Passing name to #indexes is deprecated without replacement.
+          MSG
+        end
+
         indexes = []
         current_index = nil
         execute_and_free("SHOW KEYS FROM #{quote_table_name(table_name)}", "SCHEMA") do |result|

activerecord/lib/active_record/connection_adapters/postgresql/schema_statements.rb

@@ -166,7 +166,13 @@ def index_name_exists?(table_name, index_name, default)
         end
 
         # Returns an array of indexes for the given table.
-        def indexes(table_name, name = nil)
+        def indexes(table_name, name = nil) # :nodoc:
+          if name
+            ActiveSupport::Deprecation.warn(<<-MSG.squish)
+              Passing name to #indexes is deprecated without replacement.
+            MSG
+          end
+
           table = Utils.extract_schema_qualified_name(table_name.to_s)
 
           result = query(<<-SQL, "SCHEMA")

activerecord/lib/active_record/connection_adapters/sqlite3_adapter.rb

@@ -316,6 +316,12 @@ def new_column_from_field(table_name, field) # :nondoc:
 
       # Returns an array of indexes for the given table.
       def indexes(table_name, name = nil) #:nodoc:
+        if name
+          ActiveSupport::Deprecation.warn(<<-MSG.squish)
+            Passing name to #indexes is deprecated without replacement.
+          MSG
+        end
+
         exec_query("PRAGMA index_list(#{quote_table_name(table_name)})", "SCHEMA").map do |row|
           sql = <<-SQL
             SELECT sql

activerecord/test/cases/adapters/postgresql/connection_test.rb

@@ -95,7 +95,7 @@ def test_tables_logs_name
     end
 
     def test_indexes_logs_name
-      @connection.indexes("items", "hello")
+      assert_deprecated { @connection.indexes("items", "hello") }
       assert_equal "SCHEMA", @subscriber.logged[0][1]
     end
 

activerecord/test/cases/adapters/sqlite3/sqlite3_adapter_test.rb

@@ -287,7 +287,7 @@ def test_tables_logs_name
       def test_indexes_logs_name
         with_example_table do
           assert_logged [["PRAGMA index_list(\"ex\")", "SCHEMA", []]] do
-            @conn.indexes("ex", "hello")
+            assert_deprecated { @conn.indexes("ex", "hello") }
           end
         end
       end