aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/migration.rb
diff options
context:
space:
mode:
authoryui-knk <spiketeika@gmail.com>2015-09-12 23:00:57 +0900
committeryui-knk <spiketeika@gmail.com>2015-11-09 23:13:23 +0900
commit7429633b824cf8aaeea52d11876e7bd2b8d2f256 (patch)
treeaad38b660d637c8d9578955eb136337e60fb2d03 /activerecord/lib/active_record/migration.rb
parent16214d1108c31174c94503caced3855b0f6bad95 (diff)
downloadrails-7429633b824cf8aaeea52d11876e7bd2b8d2f256.tar.gz
rails-7429633b824cf8aaeea52d11876e7bd2b8d2f256.tar.bz2
rails-7429633b824cf8aaeea52d11876e7bd2b8d2f256.zip
Deprecate `#table_exists?`, `#tables` and passing arguments to `#talbes`
Reported on #21509, how views is treated by `#tables` are differ by each adapters. To fix this different behavior, after Rails 5.0 is released, deprecate `#tables`. And `#table_exists?` would check both tables and views. To make their behavior consistent with `#tables`, after Rails 5.0 is released, deprecate `#table_exists?`.
Diffstat (limited to 'activerecord/lib/active_record/migration.rb')
-rw-r--r--activerecord/lib/active_record/migration.rb10
1 files changed, 6 insertions, 4 deletions
diff --git a/activerecord/lib/active_record/migration.rb b/activerecord/lib/active_record/migration.rb
index 1e870d4c44..ffd243a517 100644
--- a/activerecord/lib/active_record/migration.rb
+++ b/activerecord/lib/active_record/migration.rb
@@ -967,10 +967,12 @@ module ActiveRecord
end
def get_all_versions(connection = Base.connection)
- if connection.table_exists?(schema_migrations_table_name)
- SchemaMigration.all.map { |x| x.version.to_i }.sort
- else
- []
+ ActiveSupport::Deprecation.silence do
+ if connection.table_exists?(schema_migrations_table_name)
+ SchemaMigration.all.map { |x| x.version.to_i }.sort
+ else
+ []
+ end
end
end