diff options
author | Carlos Antonio da Silva <carlosantoniodasilva@gmail.com> | 2013-02-13 17:32:10 -0800 |
---|---|---|
committer | Carlos Antonio da Silva <carlosantoniodasilva@gmail.com> | 2013-02-13 17:32:10 -0800 |
commit | d1904aff6f04bcca0ee3da4aae8477f2f30b7e85 (patch) | |
tree | 61c2b013ddf8b5eb22c59e1f5ee9c1c3dfc938fd /activerecord/lib | |
parent | c0022f8d885d5cca8a7f7d3d64c60af1e04684ea (diff) | |
parent | 18ffe50581f41a4f24d791e42826c977276a0df3 (diff) | |
download | rails-d1904aff6f04bcca0ee3da4aae8477f2f30b7e85.tar.gz rails-d1904aff6f04bcca0ee3da4aae8477f2f30b7e85.tar.bz2 rails-d1904aff6f04bcca0ee3da4aae8477f2f30b7e85.zip |
Merge pull request #9278 from matthewrobertson/backport-hmt-counter-cache-bug
Backport of #8400 to fix for issue #7630, a bug in has_many :through counter caches.
Diffstat (limited to 'activerecord/lib')
-rw-r--r-- | activerecord/lib/active_record/associations/has_many_through_association.rb | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/activerecord/lib/active_record/associations/has_many_through_association.rb b/activerecord/lib/active_record/associations/has_many_through_association.rb index 53d49fef2e..2b9e7ea55d 100644 --- a/activerecord/lib/active_record/associations/has_many_through_association.rb +++ b/activerecord/lib/active_record/associations/has_many_through_association.rb @@ -139,6 +139,11 @@ module ActiveRecord delete_through_records(records) + if source_reflection.options[:counter_cache] + counter = source_reflection.counter_cache_column + klass.decrement_counter counter, records.map(&:id) + end + if through_reflection.macro == :has_many && update_through_counter?(method) update_counter(-count, through_reflection) end |