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

refactor(instructeurs): add_instructeurs should add

Browse source
This commit is contained in:
Paul Chavard 2023-02-21 17:36:34 +01:00 committed by Eric Leroy-Terquem
parent 302ddb9951
commit 3c3629d151
5 changed files with 5 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -120,8 +120,6 @@ module Administrateurs
end end
if instructeurs.present? if instructeurs.present?
instructeurs.each { groupe_instructeur.add(_1) }
flash[:notice] = if procedure.routing_enabled? flash[:notice] = if procedure.routing_enabled?
t('.assignment', t('.assignment',
count: instructeurs.size, count: instructeurs.size,

3
app/graphql/mutations/groupe_instructeur_ajouter_instructeurs.rb
View file

@ -11,9 +11,8 @@ module Mutations
def resolve(groupe_instructeur:, instructeurs:) def resolve(groupe_instructeur:, instructeurs:)
ids, emails = partition_instructeurs_by(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 groupe_instructeur.reload
result = { groupe_instructeur: } result = { groupe_instructeur: }

3
app/graphql/mutations/groupe_instructeur_creer.rb
View file

@ -29,9 +29,8 @@ module Mutations
result = { groupe_instructeur: } result = { groupe_instructeur: }
if emails.present? || ids.present? 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 groupe_instructeur.reload
if invalid_emails.present? if invalid_emails.present?

1
app/models/groupe_instructeur.rb
View file

@ -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 # 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 -= instructeurs
instructeurs_to_add.each { add(_1) }
[instructeurs_to_add, invalid_emails] [instructeurs_to_add, invalid_emails]
end end

7
app/services/instructeurs_import_service.rb
View file

@ -30,8 +30,7 @@ class InstructeursImportService
.to_h .to_h
target_groupes.each do |groupe_instructeur, emails| target_groupes.each do |groupe_instructeur, emails|
instructeurs, invalid_emails = groupe_instructeur.add_instructeurs(emails:) _, invalid_emails = groupe_instructeur.add_instructeurs(emails:)
instructeurs.each { groupe_instructeur.add(_1) }
errors << invalid_emails errors << invalid_emails
end end
@ -46,9 +45,7 @@ class InstructeursImportService
groupe_instructeur = procedure.defaut_groupe_instructeur groupe_instructeur = procedure.defaut_groupe_instructeur
instructeurs, invalid_emails = groupe_instructeur.add_instructeurs(emails: instructeurs_emails) _, invalid_emails = groupe_instructeur.add_instructeurs(emails: instructeurs_emails)
instructeurs.each { groupe_instructeur.add(_1) }
invalid_emails invalid_emails
end end