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

Simplification du code en supprimant if

Browse source
This commit is contained in:
SebastienGt 2023-01-06 16:52:58 +01:00 committed by Paul Chavard
parent c37ed7a7a1
commit 0507bb673d
2 changed files with 16 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -32,11 +32,10 @@ module Administrateurs
.new({ instructeurs: [current_administrateur.instructeur] }.merge(groupe_instructeur_params))
begin
if @groupe_instructeur.save!
routing_notice = " et le routage a été activé" if procedure.groupe_instructeurs.active.size == 2
redirect_to admin_procedure_groupe_instructeur_path(procedure, @groupe_instructeur),
notice: "Le groupe d’instructeurs « #{@groupe_instructeur.label} » a été créé#{routing_notice}."
end
@groupe_instructeur.save!
routing_notice = " et le routage a été activé" if procedure.groupe_instructeurs.active.size == 2
redirect_to admin_procedure_groupe_instructeur_path(procedure, @groupe_instructeur),
notice: "Le groupe d’instructeurs « #{@groupe_instructeur.label} » a été créé#{routing_notice}."
rescue StandardError => e
Rails.logger.error e.message
@procedure = procedure
@ -52,10 +51,9 @@ module Administrateurs
@groupe_instructeur = groupe_instructeur
begin
if @groupe_instructeur.update!(groupe_instructeur_params)
redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur),
notice: "Le nom est à présent « #{@groupe_instructeur.label} »."
end
@groupe_instructeur.update!(groupe_instructeur_params)
redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur),
notice: "Le nom est à présent « #{@groupe_instructeur.label} »."
rescue StandardError => e
Rails.logger.error e.message
@procedure = procedure

22
app/controllers/administrateurs/services_controller.rb
View file

@ -15,12 +15,10 @@ module Administrateurs
@service.administrateur = current_administrateur
begin
if @service.save!
@service.enqueue_api_entreprise
redirect_to admin_services_path(procedure_id: params[:procedure_id]),
notice: "#{@service.nom} créé"
end
@service.save!
@service.enqueue_api_entreprise
redirect_to admin_services_path(procedure_id: params[:procedure_id]),
notice: "#{@service.nom} créé"
rescue StandardError => e
Rails.logger.error e.message
@procedure = procedure
@ -38,14 +36,12 @@ module Administrateurs
@service = service
begin
if @service.update!(service_params)
if @service.siret_previously_changed?
@service.enqueue_api_entreprise
end
redirect_to admin_services_path(procedure_id: params[:procedure_id]),
notice: "#{@service.nom} modifié"
@service.update!(service_params)
if @service.siret_previously_changed?
@service.enqueue_api_entreprise
end
redirect_to admin_services_path(procedure_id: params[:procedure_id]),
notice: "#{@service.nom} modifié"
rescue StandardError => e
Rails.logger.error e.message
@procedure = procedure