1 files changed, 9 insertions, 10 deletions
diff --git a/activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb b/activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb
index 5623257fe8..e5c0e1690c 100644
--- a/activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb
+++ b/activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb
@@ -43,7 +43,7 @@ module ActiveRecord
       end
 
       # Returns an array of table names defined in the database.
-      def tables(name = nil)
+      def tables
         raise NotImplementedError, "#tables is not implemented"
       end
 
@@ -994,15 +994,15 @@ module ActiveRecord
       end
 
       def insert_versions_sql(versions) # :nodoc:
-        sm_table = ActiveRecord::Migrator.schema_migrations_table_name
+        sm_table = quote_table_name(ActiveRecord::Migrator.schema_migrations_table_name)
 
         if versions.is_a?(Array)
           sql = "INSERT INTO #{sm_table} (version) VALUES\n"
-          sql << versions.map { |v| "('#{v}')" }.join(",\n")
+          sql << versions.map { |v| "(#{quote(v)})" }.join(",\n")
           sql << ";\n\n"
           sql
         else
-          "INSERT INTO #{sm_table} (version) VALUES ('#{versions}');"
+          "INSERT INTO #{sm_table} (version) VALUES (#{quote(versions)});"
         end
       end
 
@@ -1032,7 +1032,7 @@ module ActiveRecord
         end
 
         unless migrated.include?(version)
-          execute "INSERT INTO #{sm_table} (version) VALUES ('#{version}')"
+          execute "INSERT INTO #{sm_table} (version) VALUES (#{quote(version)})"
         end
 
         inserting = (versions - migrated).select { |v| v < version }
@@ -1169,7 +1169,7 @@ module ActiveRecord
         raise NotImplementedError, "#{self.class} does not support changing column comments"
       end
 
-      protected
+      private
 
         def add_index_sort_order(quoted_columns, **options)
           if order = options[:order]
@@ -1253,7 +1253,6 @@ module ActiveRecord
           end
         end
 
-      private
         def create_table_definition(*args)
           TableDefinition.new(*args)
         end
@@ -1262,7 +1261,7 @@ module ActiveRecord
           AlterTable.new create_table_definition(name)
         end
 
-        def index_name_options(column_names) # :nodoc:
+        def index_name_options(column_names)
           if column_names.is_a?(String)
             column_names = column_names.scan(/\w+/).join("_")
           end
@@ -1270,7 +1269,7 @@ module ActiveRecord
           { column: column_names }
         end
 
-        def foreign_key_name(table_name, options) # :nodoc:
+        def foreign_key_name(table_name, options)
           identifier = "#{table_name}_#{options.fetch(:column)}_fk"
           hashed_identifier = Digest::SHA256.hexdigest(identifier).first(10)
           options.fetch(:name) do
@@ -1278,7 +1277,7 @@ module ActiveRecord
           end
         end
 
-        def validate_index_length!(table_name, new_name, internal = false) # :nodoc:
+        def validate_index_length!(table_name, new_name, internal = false)
           max_index_length = internal ? index_name_length : allowed_index_name_length
 
           if new_name.length > max_index_length