From b987688d194c7457b7183528dd6e591f5553159b Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Mon, 9 Sep 2024 16:48:07 +0200
Subject: [PATCH] refactor: disable auto hide instructeurs toggle by admin

---
 .../instructeurs_options_component.html.haml  | 11 ------
 .../groupe_instructeurs_controller.rb         |  7 ----
 config/routes.rb                              |  1 -
 .../groupe_instructeurs_controller_spec.rb    | 37 -------------------
 4 files changed, 56 deletions(-)

diff --git a/app/components/procedure/instructeurs_options_component/instructeurs_options_component.html.haml b/app/components/procedure/instructeurs_options_component/instructeurs_options_component.html.haml
index 244b4bc98..903cf6e64 100644
--- a/app/components/procedure/instructeurs_options_component/instructeurs_options_component.html.haml
+++ b/app/components/procedure/instructeurs_options_component/instructeurs_options_component.html.haml
@@ -15,17 +15,6 @@
           hint: "L’autogestion des instructeurs permet aux instructeurs de gérer eux-mêmes la liste des instructeurs de la démarche.#{ 'Nous recommandons de laisser l’autogestion des instructeurs activée.' if @procedure.routing_enabled? }",
           disabled: false)
 
-    %li
-      = form_for @procedure,
-        method: :patch,
-        url: update_hide_instructeurs_email_admin_procedure_groupe_instructeurs_path(@procedure),
-        data: { controller: 'autosubmit', turbo: 'true' } do |f|
-
-        = render Dsfr::ToggleComponent.new(form: f,
-          target: :hide_instructeurs_email,
-          title: "Anonymat des instructeurs",
-          hint: "Permet de cacher l'adresse mail des instructeurs aux usagers lors de leurs interactions par le biais de la messagerie. Cette option est à activer pour les démarches sensibles.",
-          disabled: false)
   %hr
   %p.fr-mt-2w Routage
   %p.fr-mt-2w= t('.routing_configuration_notice_1')
diff --git a/app/controllers/administrateurs/groupe_instructeurs_controller.rb b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
index f7bdc59e7..8bc9fd900 100644
--- a/app/controllers/administrateurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
@@ -320,13 +320,6 @@ module Administrateurs
       notice: "L’autogestion des instructeurs est #{procedure.instructeurs_self_management_enabled? ? "activée" : "désactivée"}."
     end
 
-    def update_hide_instructeurs_email
-      procedure.update!(hide_instructeurs_email_params)
-
-      redirect_to options_admin_procedure_groupe_instructeurs_path(procedure),
-      notice: "L'anonymisation des instructeurs est #{procedure.hide_instructeurs_email? ? "activée" : "désactivée"}."
-    end
-
     def import
       if procedure.publiee_or_close?
         if !CSV_ACCEPTED_CONTENT_TYPES.include?(csv_file.content_type) && !CSV_ACCEPTED_CONTENT_TYPES.include?(marcel_content_type)
diff --git a/config/routes.rb b/config/routes.rb
index 367471c18..57031dc7c 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -672,7 +672,6 @@ Rails.application.routes.draw do
           post 'create_simple_routing'
           delete 'destroy_all_groups_but_defaut'
           patch 'update_instructeurs_self_management_enabled'
-          patch 'update_hide_instructeurs_email'
           post 'import'
           get 'export_groupe_instructeurs'
         end
diff --git a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
index 2ab7037b1..1269856da 100644
--- a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
@@ -995,41 +995,4 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
       expect(gi_1_1.signature).to be_attached
     end
   end
-
-  describe '#update_hide_instructeurs_email' do
-    let(:administrateur) { administrateurs(:default_admin) }
-    let(:procedure) { create(:procedure, administrateurs: [administrateur]) }
-
-    before do
-      sign_in(administrateur.user)
-    end
-
-    context 'when activating hide_instructeurs_email' do
-      it 'updates the procedure and redirects with correct notice' do
-        patch :update_hide_instructeurs_email, params: {
-          procedure_id: procedure.id,
-          procedure: { hide_instructeurs_email: "1" }
-        }
-
-        expect(procedure.reload.hide_instructeurs_email).to be true
-        expect(response).to redirect_to(options_admin_procedure_groupe_instructeurs_path(procedure))
-        expect(flash[:notice]).to eq("L'anonymisation des instructeurs est activée.")
-      end
-    end
-
-    context 'when deactivating hide_instructeurs_email' do
-      let(:procedure) { create(:procedure, hide_instructeurs_email: true, administrateurs: [administrateur]) }
-
-      it 'updates the procedure and redirects with correct notice' do
-        patch :update_hide_instructeurs_email, params: {
-          procedure_id: procedure.id,
-          procedure: { hide_instructeurs_email: "0" }
-        }
-
-        expect(procedure.reload.hide_instructeurs_email).to be false
-        expect(response).to redirect_to(options_admin_procedure_groupe_instructeurs_path(procedure))
-        expect(flash[:notice]).to eq("L'anonymisation des instructeurs est désactivée.")
-      end
-    end
-  end
 end