diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2019-02-13 22:32:40 +0900 |
---|---|---|
committer | Ryuta Kamizono <kamipo@gmail.com> | 2019-02-13 22:45:35 +0900 |
commit | 47e3bbeb9057b37c244330cc4e745c8a8090e8c5 (patch) | |
tree | 6b92c111ee30d23a014aee9631812b4744b9d853 /activerecord/lib/active_record/associations/collection_association.rb | |
parent | 7724a6e98b904cb9d7ca0b135105c718e3b572d1 (diff) | |
download | rails-47e3bbeb9057b37c244330cc4e745c8a8090e8c5.tar.gz rails-47e3bbeb9057b37c244330cc4e745c8a8090e8c5.tar.bz2 rails-47e3bbeb9057b37c244330cc4e745c8a8090e8c5.zip |
Revert "Merge pull request #35127 from bogdan/counter-cache-loading"
This reverts commit eec3e28a1abf75676dcee58308ee5721bb53c325, reversing
changes made to 5588fb4802328a2183f4a55c36d6703ee435f85c.
Reason: Marking as loaded without actual loading is too greedy optimization.
See more context #35239.
Closes #35239.
[Edouard CHIN & Ryuta Kamizono]
Diffstat (limited to 'activerecord/lib/active_record/associations/collection_association.rb')
-rw-r--r-- | activerecord/lib/active_record/associations/collection_association.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/associations/collection_association.rb b/activerecord/lib/active_record/associations/collection_association.rb index b0c0beac0e..c3d4eab562 100644 --- a/activerecord/lib/active_record/associations/collection_association.rb +++ b/activerecord/lib/active_record/associations/collection_association.rb @@ -209,8 +209,7 @@ module ActiveRecord # This method is abstract in the sense that it relies on # +count_records+, which is a method descendants have to provide. def size - if !find_target? - loaded! unless loaded? + if !find_target? || loaded? target.size elsif @association_ids @association_ids.size |