From 0a353a97869b2af256d4253533beeb38303cf753 Mon Sep 17 00:00:00 2001 From: eileencodes Date: Fri, 6 Apr 2018 09:36:07 -0400 Subject: Fix structure:dump for multiple databases ccea4cf broke multiple database structure:dump, the current_config line should have been deleted instead. I'm struggling to write a test for this since the confings are passed from rake to the structure_dump method rather than the other way around. Hoping to come up with a test while I work on structure:load commands for multiple databases. --- activerecord/lib/active_record/railties/databases.rake | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'activerecord/lib/active_record/railties') diff --git a/activerecord/lib/active_record/railties/databases.rake b/activerecord/lib/active_record/railties/databases.rake index fe557b5d2b..24449e8df3 100644 --- a/activerecord/lib/active_record/railties/databases.rake +++ b/activerecord/lib/active_record/railties/databases.rake @@ -317,8 +317,7 @@ db_namespace = namespace :db do ActiveRecord::DatabaseConfigurations.configs_for(Rails.env) do |spec_name, config| ActiveRecord::Base.establish_connection(config) filename = ActiveRecord::Tasks::DatabaseTasks.dump_filename(spec_name, :sql) - current_config = ActiveRecord::Tasks::DatabaseTasks.current_config - ActiveRecord::Tasks::DatabaseTasks.structure_dump(current_config, filename) + ActiveRecord::Tasks::DatabaseTasks.structure_dump(config, filename) if ActiveRecord::SchemaMigration.table_exists? File.open(filename, "a") do |f| -- cgit v1.2.3