tech(clean): better naming

app/controllers/administrateurs/administrateur_controller.rb

@@ -1,7 +1,7 @@
 module Administrateurs
   class AdministrateurController < ApplicationController
     before_action :authenticate_administrateur!
-    helper_method :is_administrateur_through_procedure_administration_as_manager?
+    helper_method :administrateur_as_manager?
 
     def retrieve_procedure
       id = params[:procedure_id] || params[:id]
@@ -33,14 +33,14 @@ module Administrateurs
     end
 
     def ensure_not_super_admin!
-      if is_administrateur_through_procedure_administration_as_manager?
+      if administrateur_as_manager?
         redirect_back fallback_location: root_url, alert: "Interdit aux super admins", status: 403
       end
     end
 
     private
 
-    def is_administrateur_through_procedure_administration_as_manager?
+    def administrateur_as_manager?
       id = params[:procedure_id] || params[:id]
 
       current_administrateur.administrateurs_procedures

app/controllers/instructeurs/instructeur_controller.rb

@@ -7,14 +7,14 @@ module Instructeurs
     end
 
     def ensure_not_super_admin!
-      if is_instructeur_through_assign_tos_as_manager?
+      if instructeur_as_manager?
         redirect_back fallback_location: root_url, alert: "Interdit aux super admins", status: 403
       end
     end
 
     private
 
-    def is_instructeur_through_assign_tos_as_manager?
+    def instructeur_as_manager?
       procedure_id = params[:procedure_id]
 
       current_instructeur.assign_to

app/controllers/instructeurs/procedures_controller.rb

@@ -61,7 +61,7 @@ module Instructeurs
       @counts = current_instructeur
         .dossiers_count_summary(groupe_instructeur_ids)
         .symbolize_keys
-      @can_download_dossiers = (@counts[:tous] + @counts[:archives]) > 0 && !is_instructeur_through_assign_tos_as_manager?
+      @can_download_dossiers = (@counts[:tous] + @counts[:archives]) > 0 && !instructeur_as_manager?
 
       dossiers = Dossier.where(groupe_instructeur_id: groupe_instructeur_ids)
       dossiers_count = @counts[statut.underscore.to_sym]
@@ -102,7 +102,7 @@ module Instructeurs
       @a_suivre_count, @suivis_count, @traites_count, @tous_count, @archives_count, @supprimes_recemment_count, @expirant_count = current_instructeur
         .dossiers_count_summary(groupe_instructeur_ids)
         .fetch_values('a_suivre', 'suivis', 'traites', 'tous', 'archives', 'supprimes_recemment', 'expirant')
-      @can_download_dossiers = (@tous_count + @archives_count) > 0 && !is_instructeur_through_assign_tos_as_manager?
+      @can_download_dossiers = (@tous_count + @archives_count) > 0 && !instructeur_as_manager?
 
       notifications = current_instructeur.notifications_for_groupe_instructeurs(groupe_instructeur_ids)
       @has_en_cours_notifications = notifications[:en_cours].present?
@@ -146,7 +146,7 @@ module Instructeurs
       @can_download_dossiers = current_instructeur
         .dossiers
         .visible_by_administration
-        .exists?(groupe_instructeur_id: groupe_instructeur_ids) && !is_instructeur_through_assign_tos_as_manager?
+        .exists?(groupe_instructeur_id: groupe_instructeur_ids) && !instructeur_as_manager?
 
       export = Export.find_or_create_export(export_format, groupe_instructeurs, **export_options)
 

app/views/administrateurs/groupe_instructeurs/index.html.haml

@@ -22,4 +22,4 @@
         groupe_instructeur: @procedure.defaut_groupe_instructeur,
         instructeurs: @instructeurs,
         available_instructeur_emails: @available_instructeur_emails,
-        disabled_as_super_admin: is_administrateur_through_procedure_administration_as_manager? }
+        disabled_as_super_admin: administrateur_as_manager? }

app/views/administrateurs/groupe_instructeurs/show.html.haml

@@ -11,4 +11,4 @@
       groupe_instructeur: @groupe_instructeur,
       instructeurs: @instructeurs,
       available_instructeur_emails: @available_instructeur_emails,
-      disabled_as_super_admin: is_administrateur_through_procedure_administration_as_manager? }
+      disabled_as_super_admin: administrateur_as_manager? }

app/views/administrateurs/procedure_administrateurs/create.turbo_stream.haml

@@ -1,3 +1,3 @@
 - if @administrateur.present?
   = turbo_stream.append "administrateurs", partial: 'administrateur', locals: { procedure: @procedure, administrateur: @administrateur }
-  = turbo_stream.replace "new_administrateur", partial: 'add_admin_form', locals: { procedure: @procedure, disabled_as_super_admin: is_administrateur_through_procedure_administration_as_manager? }
+  = turbo_stream.replace "new_administrateur", partial: 'add_admin_form', locals: { procedure: @procedure, disabled_as_super_admin: administrateur_as_manager? }

app/views/administrateurs/procedure_administrateurs/index.html.haml

@@ -16,4 +16,4 @@
     %tfoot
       %tr
         %th{ colspan: 4 }
-          = render 'add_admin_form', procedure: @procedure, disabled_as_super_admin: is_administrateur_through_procedure_administration_as_manager?
+          = render 'add_admin_form', procedure: @procedure, disabled_as_super_admin: administrateur_as_manager?