diff --git a/app/controllers/administrateurs/groupe_instructeurs_controller.rb b/app/controllers/administrateurs/groupe_instructeurs_controller.rb index 193641eba..09948a5e5 100644 --- a/app/controllers/administrateurs/groupe_instructeurs_controller.rb +++ b/app/controllers/administrateurs/groupe_instructeurs_controller.rb @@ -348,15 +348,7 @@ module Administrateurs added_instructeurs_by_group.each do |groupe, added_instructeurs| if added_instructeurs.present? - 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 + notify_instructeurs(groupe, added_instructeurs) end flash_message_for_import(invalid_emails) end @@ -366,15 +358,7 @@ module Administrateurs added_instructeurs, invalid_emails = InstructeursImportService.import_instructeurs(procedure, instructors_emails) if added_instructeurs.present? - known_instructeurs, new_instructeurs = added_instructeurs.partition { |instructeur| instructeur.user.email_verified_at } - - 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 + notify_instructeurs(groupe_instructeur, added_instructeurs) end flash_message_for_import(invalid_emails) else @@ -522,5 +506,17 @@ module Administrateurs .update(instructeurs: [current_administrateur.instructeur], routing_rule:) 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