Merge pull request #4320 from betagouv/perf

Perf index procedures

app/controllers/instructeurs/procedures_controller.rb

@@ -6,7 +6,10 @@ module Instructeurs
     ITEMS_PER_PAGE = 25
 
     def index
-      @procedures = current_instructeur.visible_procedures.order(archived_at: :desc, published_at: :desc, created_at: :desc)
+      @procedures = current_instructeur
+        .visible_procedures
+        .includes(:logo_attachment, :logo_active_storage_attachment, :defaut_groupe_instructeur)
+        .order(archived_at: :desc, published_at: :desc, created_at: :desc)
 
       groupe_instructeurs = current_instructeur.groupe_instructeurs.where(procedure: @procedures)
 

app/models/procedure.rb

@@ -27,6 +27,8 @@ class Procedure < ApplicationRecord
   has_one :refused_mail, class_name: "Mails::RefusedMail", dependent: :destroy
   has_one :without_continuation_mail, class_name: "Mails::WithoutContinuationMail", dependent: :destroy
 
+  has_one :defaut_groupe_instructeur, -> { where(label: GroupeInstructeur::DEFAULT_LABEL) }, class_name: 'GroupeInstructeur', inverse_of: :procedure
+
   has_one_attached :logo
   has_one_attached :logo_active_storage
   has_one_attached :notice
@@ -477,10 +479,6 @@ class Procedure < ApplicationRecord
     end
   end
 
-  def defaut_groupe_instructeur
-    groupe_instructeurs.find_by(label: GroupeInstructeur::DEFAULT_LABEL)
-  end
-
   def missing_instructeurs?
     !AssignTo.exists?(groupe_instructeur: groupe_instructeurs)
   end