diff options
author | Arthur Neves <arthurnn@gmail.com> | 2014-02-28 17:00:38 -0500 |
---|---|---|
committer | Arthur Neves <arthurnn@gmail.com> | 2014-02-28 17:00:38 -0500 |
commit | 774160b9ad6908435bf3485e7ac98633deff76c6 (patch) | |
tree | 37c2e322ff4eae74c8d314912293b07efb46218b | |
parent | be6f51a487eb412ca29765aa7e84d2f13a688891 (diff) | |
download | rails-774160b9ad6908435bf3485e7ac98633deff76c6.tar.gz rails-774160b9ad6908435bf3485e7ac98633deff76c6.tar.bz2 rails-774160b9ad6908435bf3485e7ac98633deff76c6.zip |
Remove unnecessary db call when replacing.
When replacing a has_many association with the same one, there is no
need to do a round-trip to the db to create/and drop a new transaction.
[fixes #14220]
-rw-r--r-- | activerecord/lib/active_record/associations/collection_association.rb | 4 | ||||
-rw-r--r-- | activerecord/test/cases/associations/has_many_associations_test.rb | 10 |
2 files changed, 13 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/associations/collection_association.rb b/activerecord/lib/active_record/associations/collection_association.rb index 270871c866..9a2900843e 100644 --- a/activerecord/lib/active_record/associations/collection_association.rb +++ b/activerecord/lib/active_record/associations/collection_association.rb @@ -359,7 +359,9 @@ module ActiveRecord if owner.new_record? replace_records(other_array, original_target) else - transaction { replace_records(other_array, original_target) } + if other_array != original_target + transaction { replace_records(other_array, original_target) } + end end end diff --git a/activerecord/test/cases/associations/has_many_associations_test.rb b/activerecord/test/cases/associations/has_many_associations_test.rb index a86fb15719..49d3301044 100644 --- a/activerecord/test/cases/associations/has_many_associations_test.rb +++ b/activerecord/test/cases/associations/has_many_associations_test.rb @@ -1242,6 +1242,16 @@ class HasManyAssociationsTest < ActiveRecord::TestCase assert_equal orig_accounts, firm.accounts end + def test_replace_with_same_content + firm = Firm.first + firm.clients = [] + firm.save + + assert_queries(0, ignore_none: true) do + firm.clients = [] + end + end + def test_transactions_when_replacing_on_persisted good = Client.new(:name => "Good") bad = Client.new(:name => "Bad", :raise_on_save => true) |