diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2017-01-24 21:40:41 +0100 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2017-01-24 21:40:41 +0100 |
commit | 8a880a3d464daf486a10d85b8ee49305aa6b1e5b (patch) | |
tree | cc143c4fc9b5e5fd359c2ae51a78ab78297ca627 /app/services/after_block_service.rb | |
parent | 326c75b35b826aa9e807fe7d1972ade41d789663 (diff) |
Make blocks create entries and unfollows instantly, but do the clean up
in the background instead. Should fix delay where blocked person can interact with blocker for a short time before background job gets processed
Diffstat (limited to 'app/services/after_block_service.rb')
-rw-r--r-- | app/services/after_block_service.rb | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/app/services/after_block_service.rb b/app/services/after_block_service.rb new file mode 100644 index 000000000..8c6197f2c --- /dev/null +++ b/app/services/after_block_service.rb @@ -0,0 +1,31 @@ +# frozen_string_literal: true + +class AfterBlockService < BaseService + def call(account, target_account) + clear_timelines(account, target_account) + clear_notifications(account, target_account) + end + + private + + def clear_timelines(account, target_account) + mentions_key = FeedManager.instance.key(:mentions, account.id) + home_key = FeedManager.instance.key(:home, account.id) + + target_account.statuses.select('id').find_each do |status| + redis.zrem(mentions_key, status.id) + redis.zrem(home_key, status.id) + end + end + + def clear_notifications(account, target_account) + Notification.where(account: account).joins(:follow).where(activity_type: 'Follow', follows: { account_id: target_account.id }).destroy_all + Notification.where(account: account).joins(mention: :status).where(activity_type: 'Mention', statuses: { account_id: target_account.id }).destroy_all + Notification.where(account: account).joins(:favourite).where(activity_type: 'Favourite', favourites: { account_id: target_account.id }).destroy_all + Notification.where(account: account).joins(:status).where(activity_type: 'Status', statuses: { account_id: target_account.id }).destroy_all + end + + def redis + Redis.current + end +end |