From 0507bb673d20eac936da599acc9f782347bdddf7 Mon Sep 17 00:00:00 2001
From: SebastienGt <sebastien.friedberg@gmail.com>
Date: Fri, 6 Jan 2023 16:52:58 +0100
Subject: [PATCH] Simplification du code en supprimant if

---
 .../groupe_instructeurs_controller.rb         | 16 ++++++--------
 .../administrateurs/services_controller.rb    | 22 ++++++++-----------
 2 files changed, 16 insertions(+), 22 deletions(-)

diff --git a/app/controllers/administrateurs/groupe_instructeurs_controller.rb b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
index ced71b523..cf15a3636 100644
--- a/app/controllers/administrateurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
@@ -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
diff --git a/app/controllers/administrateurs/services_controller.rb b/app/controllers/administrateurs/services_controller.rb
index 2d7851d42..8c73dd591 100644
--- a/app/controllers/administrateurs/services_controller.rb
+++ b/app/controllers/administrateurs/services_controller.rb
@@ -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