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

refactor(instructeurs import): extract notify_instructeurs method

Browse source
This commit is contained in:
Eric Leroy-Terquem 2024-09-06 09:59:33 +02:00
parent d361ce2e38
commit 7dc853ad1c
No known key found for this signature in database
GPG key ID: 53D8FAECEF207605
1 changed files with 14 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
app/controllers/administrateurs/groupe_instructeurs_controller.rb
View file

@ -348,15 +348,7 @@ module Administrateurs
added_instructeurs_by_group.each do |groupe, added_instructeurs| added_instructeurs_by_group.each do |groupe, added_instructeurs|
if added_instructeurs.present? if added_instructeurs.present?
known_instructeurs, new_instructeurs = added_instructeurs.partition { |instructeur| instructeur.user.email_verified_at } notify_instructeurs(groupe, added_instructeurs)
new_instructeurs.each { InstructeurMailer.confirm_and_notify_added_instructeur(_1, groupe, current_administrateur.email).deliver_later }
if known_instructeurs.present?
GroupeInstructeurMailer
.notify_added_instructeurs(groupe, known_instructeurs, current_administrateur.email)
.deliver_later
end
end end
flash_message_for_import(invalid_emails) flash_message_for_import(invalid_emails)
end end
@ -366,15 +358,7 @@ module Administrateurs
added_instructeurs, invalid_emails = InstructeursImportService.import_instructeurs(procedure, instructors_emails) added_instructeurs, invalid_emails = InstructeursImportService.import_instructeurs(procedure, instructors_emails)
if added_instructeurs.present? if added_instructeurs.present?
known_instructeurs, new_instructeurs = added_instructeurs.partition { |instructeur| instructeur.user.email_verified_at } notify_instructeurs(groupe_instructeur, added_instructeurs)
new_instructeurs.each { InstructeurMailer.confirm_and_notify_added_instructeur(_1, groupe_instructeur, current_administrateur.email).deliver_later }
if known_instructeurs.present?
GroupeInstructeurMailer
.notify_added_instructeurs(groupe_instructeur, known_instructeurs, current_administrateur.email)
.deliver_later
end
end end
flash_message_for_import(invalid_emails) flash_message_for_import(invalid_emails)
else else
@ -522,5 +506,17 @@ module Administrateurs
.update(instructeurs: [current_administrateur.instructeur], routing_rule:) .update(instructeurs: [current_administrateur.instructeur], routing_rule:)
end end
end end
def notify_instructeurs(groupe, added_instructeurs)
known_instructeurs, new_instructeurs = added_instructeurs.partition { |instructeur| instructeur.user.email_verified_at }
new_instructeurs.each { InstructeurMailer.confirm_and_notify_added_instructeur(_1, groupe, current_administrateur.email).deliver_later }
if known_instructeurs.present?
GroupeInstructeurMailer
.notify_added_instructeurs(groupe, known_instructeurs, current_administrateur.email)
.deliver_later
end
end
end end
end end