From cfc584f496f7e2cc94f8ca99a43a5ea70f14d9a3 Mon Sep 17 00:00:00 2001
From: clemkeirua <clement@keiruaprod.fr>
Date: Wed, 26 Feb 2020 15:21:36 +0100
Subject: [PATCH] suppression des preconditions sur sorted_ids et filtered_ids
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Ces preconditions induisent un probleme de n+1. On peut résoudre le problème en supprimant la précondition, ou en incluant les procedures dans les dossiers
J'ai opté pour la suppression de la précondition, qui avait fait débat lors de son introduction:

https://github.com/betagouv/demarches-simplifiees.fr/pull/2770/commits/667deae5ccc5a4bf1d5386440bb4cd5f8086b305#diff-b6be6196cd739f8b31e0b195677e46d6R176
---
 app/models/procedure_presentation.rb | 2 --
 1 file changed, 2 deletions(-)

diff --git a/app/models/procedure_presentation.rb b/app/models/procedure_presentation.rb
index 75872f175..944b2a07b 100644
--- a/app/models/procedure_presentation.rb
+++ b/app/models/procedure_presentation.rb
@@ -75,7 +75,6 @@ class ProcedurePresentation < ApplicationRecord
   end
 
   def sorted_ids(dossiers, instructeur)
-    dossiers.each { |dossier| assert_matching_procedure(dossier) }
     table, column, order = sort.values_at('table', 'column', 'order')
 
     case table
@@ -110,7 +109,6 @@ class ProcedurePresentation < ApplicationRecord
   end
 
   def filtered_ids(dossiers, statut)
-    dossiers.each { |dossier| assert_matching_procedure(dossier) }
     filters[statut].group_by { |filter| filter.values_at('table', 'column') } .map do |(table, column), filters|
       values = filters.pluck('value')
       case table