rodrigo/iachat
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

chore: Fix RemoveStaleNotificationJob query (#5242)

Browse Source 
Rubocop amended the query in #5236, which led to failure as the Message model already has a default scope.
This PR fixes it.
This commit is contained in:
Sojan Jose 2022-08-10 15:08:37 +02:00 committed by GitHub
parent 74fdfffe08
commit 831a3bcd50
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
app/jobs/migration/remove_stale_notifications_job.rb
View File

@ -10,7 +10,8 @@ class Migration::RemoveStaleNotificationsJob < ApplicationJob
def remove_invalid_messages
deleted_ids = []
Message.distinct.pluck(:inbox_id).each_slice(1000) do |id_list|
Message.unscoped.distinct.pluck(:inbox_id).each_slice(1000) do |id_list|
deleted_ids << (id_list - Inbox.where(id: id_list).pluck(:id))
end