From d66c922b6e97108da658597f5f0dfd713d791f83 Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Fri, 25 Jul 2025 10:01:51 -0400 Subject: [PATCH] Fix `Style/GuardClause` in process status update service --- .../process_status_update_service.rb | 26 ++++++++++++------- 1 file changed, 17 insertions(+), 9 deletions(-) diff --git a/app/services/activitypub/process_status_update_service.rb b/app/services/activitypub/process_status_update_service.rb index 0da1335b5cc..282439ca32a 100644 --- a/app/services/activitypub/process_status_update_service.rb +++ b/app/services/activitypub/process_status_update_service.rb @@ -204,21 +204,29 @@ class ActivityPub::ProcessStatusUpdateService < BaseService added_tags = current_tags - previous_tags - unless added_tags.empty? - @account.featured_tags.where(tag_id: added_tags.pluck(:id)).find_each do |featured_tag| - featured_tag.increment(@status.created_at) - end - end + increment_featured_tags(added_tags) unless added_tags.empty? removed_tags = previous_tags - current_tags - unless removed_tags.empty? - @account.featured_tags.where(tag_id: removed_tags.pluck(:id)).find_each do |featured_tag| - featured_tag.decrement(@status) - end + decrement_featured_tags(removed_tags) unless removed_tags.empty? + end + + def increment_featured_tags(added_tags) + featured_tags_for(added_tags).find_each do |featured_tag| + featured_tag.increment(@status.created_at) end end + def decrement_featured_tags(removed_tags) + featured_tags_for(removed_tags).find_each do |featured_tag| + featured_tag.decrement(@status) + end + end + + def featured_tags_for(tags) + @account.featured_tags.where(tag_id: tags.pluck(:id)) + end + def update_mentions! unresolved_mentions = []