diff --git a/app/controllers/administrateurs/groupe_instructeurs_controller.rb b/app/controllers/administrateurs/groupe_instructeurs_controller.rb index b701254eb..a410afef8 100644 --- a/app/controllers/administrateurs/groupe_instructeurs_controller.rb +++ b/app/controllers/administrateurs/groupe_instructeurs_controller.rb @@ -120,8 +120,6 @@ module Administrateurs end if instructeurs.present? - instructeurs.each { groupe_instructeur.add(_1) } - flash[:notice] = if procedure.routing_enabled? t('.assignment', count: instructeurs.size, diff --git a/app/graphql/mutations/groupe_instructeur_ajouter_instructeurs.rb b/app/graphql/mutations/groupe_instructeur_ajouter_instructeurs.rb index c536667ef..a6e6a6c1f 100644 --- a/app/graphql/mutations/groupe_instructeur_ajouter_instructeurs.rb +++ b/app/graphql/mutations/groupe_instructeur_ajouter_instructeurs.rb @@ -11,9 +11,8 @@ module Mutations def resolve(groupe_instructeur:, instructeurs:) ids, emails = partition_instructeurs_by(instructeurs) - instructeurs, invalid_emails = groupe_instructeur.add_instructeurs(ids:, emails:) + _, invalid_emails = groupe_instructeur.add_instructeurs(ids:, emails:) - instructeurs.each { groupe_instructeur.add(_1) } groupe_instructeur.reload result = { groupe_instructeur: } diff --git a/app/graphql/mutations/groupe_instructeur_creer.rb b/app/graphql/mutations/groupe_instructeur_creer.rb index d7a0a6331..465606fe1 100644 --- a/app/graphql/mutations/groupe_instructeur_creer.rb +++ b/app/graphql/mutations/groupe_instructeur_creer.rb @@ -29,9 +29,8 @@ module Mutations result = { groupe_instructeur: } if emails.present? || ids.present? - instructeurs, invalid_emails = groupe_instructeur.add_instructeurs(ids:, emails:) + _, invalid_emails = groupe_instructeur.add_instructeurs(ids:, emails:) - instructeurs.each { groupe_instructeur.add(_1) } groupe_instructeur.reload if invalid_emails.present? diff --git a/app/models/groupe_instructeur.rb b/app/models/groupe_instructeur.rb index 30efb9956..3ea1e9334 100644 --- a/app/models/groupe_instructeur.rb +++ b/app/models/groupe_instructeur.rb @@ -66,6 +66,7 @@ class GroupeInstructeur < ApplicationRecord # We dont't want to assign a user to a groupe_instructeur if they are already assigned to it instructeurs_to_add -= instructeurs + instructeurs_to_add.each { add(_1) } [instructeurs_to_add, invalid_emails] end diff --git a/app/services/instructeurs_import_service.rb b/app/services/instructeurs_import_service.rb index dffd27836..eb471fa5b 100644 --- a/app/services/instructeurs_import_service.rb +++ b/app/services/instructeurs_import_service.rb @@ -30,8 +30,7 @@ class InstructeursImportService .to_h target_groupes.each do |groupe_instructeur, emails| - instructeurs, invalid_emails = groupe_instructeur.add_instructeurs(emails:) - instructeurs.each { groupe_instructeur.add(_1) } + _, invalid_emails = groupe_instructeur.add_instructeurs(emails:) errors << invalid_emails end @@ -46,9 +45,7 @@ class InstructeursImportService groupe_instructeur = procedure.defaut_groupe_instructeur - instructeurs, invalid_emails = groupe_instructeur.add_instructeurs(emails: instructeurs_emails) - - instructeurs.each { groupe_instructeur.add(_1) } + _, invalid_emails = groupe_instructeur.add_instructeurs(emails: instructeurs_emails) invalid_emails end