mirror of
https://github.com/scinote-eln/scinote-web.git
synced 2024-12-27 02:04:33 +08:00
Fix user deletion [SCI-9868]
This commit is contained in:
parent
36596b3931
commit
6c9d128b97
1 changed files with 0 additions and 20 deletions
|
@ -321,7 +321,6 @@ class User < ApplicationRecord
|
|||
has_many :hidden_repository_cell_reminders, dependent: :destroy
|
||||
|
||||
before_validation :downcase_email!
|
||||
before_destroy :destroy_notifications
|
||||
|
||||
def name
|
||||
full_name
|
||||
|
@ -635,25 +634,6 @@ class User < ApplicationRecord
|
|||
self.email = email.downcase
|
||||
end
|
||||
|
||||
def destroy_notifications
|
||||
# Find all notifications where user is the only reference
|
||||
# on the notification, and destroy all such notifications
|
||||
# (user_notifications are destroyed when notification is
|
||||
# destroyed). We try to do this efficiently (hence in_groups_of).
|
||||
nids_all = notifications.pluck(:id)
|
||||
nids_all.in_groups_of(1000, false) do |nids|
|
||||
Notification
|
||||
.where(id: nids)
|
||||
.joins(:user_notifications)
|
||||
.group('notifications.id')
|
||||
.having('count(notification_id) <= 1')
|
||||
.destroy_all
|
||||
end
|
||||
|
||||
# Now, simply destroy all user notification relations left
|
||||
user_notifications.destroy_all
|
||||
end
|
||||
|
||||
def clear_view_cache
|
||||
Rails.cache.delete_matched(%r{^views\/users\/#{id}-})
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue