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

refactor(groupe_instructeurs): extract method for flash message at import

Browse source
This commit is contained in:
Eric Leroy-Terquem 2023-01-18 17:48:42 +01:00
parent 25df48f255
commit d45efa99bc
1 changed files with 12 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -219,13 +219,8 @@ module Administrateurs
if groupes_emails_has_keys.blank? if groupes_emails_has_keys.blank?
flash[:alert] = "Importation impossible, veuillez importer un csv #{view_context.link_to('suivant ce modèle', "/csv/#{I18n.locale}/import-groupe-test.csv")}" flash[:alert] = "Importation impossible, veuillez importer un csv #{view_context.link_to('suivant ce modèle', "/csv/#{I18n.locale}/import-groupe-test.csv")}"
else else
add_instructeurs_and_get_errors = InstructeursImportService.import_groupes(procedure, groupes_emails) result = InstructeursImportService.import_groupes(procedure, groupes_emails)
flash_message_for_import(result)
if add_instructeurs_and_get_errors.blank?
flash[:notice] = "La liste des instructeurs a été importée avec succès"
else
flash[:alert] = "Import terminé. Cependant les emails suivants ne sont pas pris en compte: #{add_instructeurs_and_get_errors.join(', ')}"
end
end end
elsif params[:instructeurs_csv_file] elsif params[:instructeurs_csv_file]
@ -237,13 +232,8 @@ module Administrateurs
if instructors_emails_has_key.blank? if instructors_emails_has_key.blank?
flash[:alert] = "Importation impossible, veuillez importer un csv #{view_context.link_to('suivant ce modèle', "/csv/import-instructeurs-test.csv")}" flash[:alert] = "Importation impossible, veuillez importer un csv #{view_context.link_to('suivant ce modèle', "/csv/import-instructeurs-test.csv")}"
else else
add_instructeurs_and_get_errors = InstructeursImportService.import_instructeurs(procedure, instructors_emails) result = InstructeursImportService.import_instructeurs(procedure, instructors_emails)
flash_message_for_import(result)
if add_instructeurs_and_get_errors.blank?
flash[:notice] = "La liste des instructeurs a été importée avec succès"
else
flash[:alert] = "Import terminé. Cependant les emails suivants ne sont pas pris en compte: #{add_instructeurs_and_get_errors.join(', ')}"
end
end end
end end
redirect_to admin_procedure_groupe_instructeurs_path(procedure) redirect_to admin_procedure_groupe_instructeurs_path(procedure)
@ -334,5 +324,13 @@ module Administrateurs
def routing_enabled_params def routing_enabled_params
{ routing_enabled: params.require(:routing) == 'enable' } { routing_enabled: params.require(:routing) == 'enable' }
end end
def flash_message_for_import(result)
if result.blank?
flash[:notice] = "La liste des instructeurs a été importée avec succès"
else
flash[:alert] = "Import terminé. Cependant les emails suivants ne sont pas pris en compte: #{result.join(', ')}"
end
end
end end
end end