FIX: don't enqueue TruncateUserFlagStats job when not needed

2019-04-01 15:53:55 +02:00 · 2019-04-01 15:53:55 +02:00 · a6596662dc
parent 6e0e36f790
commit a6596662dc
1 changed files with 4 additions and 4 deletions
--- a/app/models/reviewable.rb
+++ b/app/models/reviewable.rb
@ -431,10 +431,10 @@ private
      RETURNING user_id, flags_agreed + flags_disagreed + flags_ignored AS total
    SQL

-    Jobs.enqueue(
-      :truncate_user_flag_stats,
-      user_ids: result.select { |r| r.total > Jobs::TruncateUserFlagStats.truncate_to }.map(&:user_id)
-    )
+    user_ids = result.select { |r| r.total > Jobs::TruncateUserFlagStats.truncate_to }.map(&:user_id)
+    return if user_ids.blank?
+
+    Jobs.enqueue(:truncate_user_flag_stats, user_ids: user_ids)
  end
 end