DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

fix(users.update_email): optimistic

Browse source
This commit is contained in:
mfo 2024-07-12 14:54:32 +02:00
parent 6dc8496d3d
commit d5c8b7464c
No known key found for this signature in database
GPG key ID: 7CE3E1F5B794A8EC
2 changed files with 14 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
app/controllers/users/profil_controller.rb
View file

@ -9,11 +9,11 @@ module Users
def update_email
requested_user = User.find_by(email: requested_email)
if requested_user.present? && current_user.ask_for_merge(requested_user)
current_user.update(unconfirmed_email: nil)
if requested_user.present?
current_user.ask_for_merge(requested_user)
flash.notice = t('devise.registrations.update_needs_confirmation')
elsif current_user.update(update_email_params)
elsif update_with_lock(update_email_params)
current_user.update(requested_merge_into: nil)
flash.notice = t('devise.registrations.update_needs_confirmation')
@ -22,6 +22,15 @@ module Users
end
redirect_to profil_path
rescue ActiveRecord::RecordInvalid => e
flash.alert = e.record.errors.full_messages
redirect_to profil_path
end
def update_with_lock(params)
current_user.with_lock do
current_user.update!(params)
end
end
def transfer_all_dossiers

8
app/models/user.rb
View file

@ -257,12 +257,8 @@ class User < ApplicationRecord
end
def ask_for_merge(requested_user)
if update(requested_merge_into: requested_user)
UserMailer.ask_for_merge(self, requested_user.email).deliver_later
return true
else
return false
end
update!(requested_merge_into: requested_user, unconfirmed_email: nil)
UserMailer.ask_for_merge(self, requested_user.email).deliver_later
end
def send_devise_notification(notification, *args)