diff options
author | David Yip <yipdw@member.fsf.org> | 2018-06-02 16:08:31 -0500 |
---|---|---|
committer | David Yip <yipdw@member.fsf.org> | 2018-06-02 16:08:31 -0500 |
commit | a641d1b5b8e9f20104ab16e5419e6dee4e5af37f (patch) | |
tree | 30eeb0f208b0ff314d71f2f90935fc3ab5fc282f /app/models/favourite.rb | |
parent | 9ad5de8a3a5a14760f83f3d0f95eddca1fc4b106 (diff) | |
parent | 165b5dc7f5711efd07adb73316e94d89b9e7e3a3 (diff) |
Merge remote-tracking branch 'personal/merge/tootsuite/master' into gs-master
Diffstat (limited to 'app/models/favourite.rb')
-rw-r--r-- | app/models/favourite.rb | 25 |
1 files changed, 24 insertions, 1 deletions
diff --git a/app/models/favourite.rb b/app/models/favourite.rb index c998a67eb..0fce82f6f 100644 --- a/app/models/favourite.rb +++ b/app/models/favourite.rb @@ -16,7 +16,7 @@ class Favourite < ApplicationRecord update_index('statuses#status', :status) if Chewy.enabled? belongs_to :account, inverse_of: :favourites - belongs_to :status, inverse_of: :favourites, counter_cache: true + belongs_to :status, inverse_of: :favourites has_one :notification, as: :activity, dependent: :destroy @@ -25,4 +25,27 @@ class Favourite < ApplicationRecord before_validation do self.status = status.reblog if status&.reblog? end + + after_create :increment_cache_counters + after_destroy :decrement_cache_counters + + private + + def increment_cache_counters + if association(:status).loaded? + status.update_attribute(:favourites_count, status.favourites_count + 1) + else + Status.where(id: status_id).update_all('favourites_count = COALESCE(favourites_count, 0) + 1') + end + end + + def decrement_cache_counters + return if association(:status).loaded? && (status.marked_for_destruction? || status.marked_for_mass_destruction?) + + if association(:status).loaded? + status.update_attribute(:favourites_count, [status.favourites_count - 1, 0].max) + else + Status.where(id: status_id).update_all('favourites_count = GREATEST(COALESCE(favourites_count, 0) - 1, 0)') + end + end end |