From 4d7354aa352b94ec3a10ad053cff04890c534080 Mon Sep 17 00:00:00 2001 From: Ryuta Kamizono Date: Tue, 4 Dec 2018 14:33:31 +0900 Subject: Ensure that `delete_all` on collection proxy returns affected count Unlike the `Relation#delete_all`, `delete_all` on collection proxy doesn't return affected count. Since the `CollectionProxy` is a subclass of the `Relation`, this inconsistency is probably not intended, so it should return the count consistently. --- .../lib/active_record/associations/has_many_through_association.rb | 2 ++ 1 file changed, 2 insertions(+) (limited to 'activerecord/lib/active_record/associations/has_many_through_association.rb') 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 2322a49931..84a9797aa5 100644 --- a/activerecord/lib/active_record/associations/has_many_through_association.rb +++ b/activerecord/lib/active_record/associations/has_many_through_association.rb @@ -161,6 +161,8 @@ module ActiveRecord else update_counter(-count) end + + count end def difference(a, b) -- cgit v1.2.3