DGNum/demarches-normaliennes
13
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Avoid n+1 because of gestionnaire.follows?(dossier)

Browse source
This commit is contained in:
Mathieu Magnin 2017-07-31 18:35:34 +02:00
parent d780ab1c90
commit f3f2b6acc2
3 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
app/controllers/new_gestionnaire/procedures_controller.rb
View file

@ -37,6 +37,11 @@ module NewGestionnaire
.where(procedure: @procedure) .where(procedure: @procedure)
.en_cours .en_cours
@followed_dossiers_id = current_gestionnaire
.followed_dossiers
.where(procedure: @procedure)
.pluck(:id)
@termines_dossiers = procedure.dossiers.includes(:user).termine @termines_dossiers = procedure.dossiers.includes(:user).termine
@all_state_dossiers = procedure.dossiers.includes(:user).all_state @all_state_dossiers = procedure.dossiers.includes(:user).all_state

2
app/views/new_gestionnaire/procedures/_dossier_actions.html.haml
View file

@ -1,5 +1,5 @@
- if dossier.en_construction_ou_instruction? - if dossier.en_construction_ou_instruction?
- if current_gestionnaire.follow?(dossier) - if dossier_is_followed
= link_to unfollow_dossier_path(procedure, dossier), method: :patch, class: 'button' do = link_to unfollow_dossier_path(procedure, dossier), method: :patch, class: 'button' do
%i.unfollow> %i.unfollow>
ne plus suivre ne plus suivre

2
app/views/new_gestionnaire/procedures/show.html.haml
View file

@ -47,6 +47,6 @@
%td.status-col %td.status-col
= link_to(dossier_path(@procedure, dossier), class: 'big-link') do = link_to(dossier_path(@procedure, dossier), class: 'big-link') do
= render partial: 'status', locals: { dossier: dossier } = render partial: 'status', locals: { dossier: dossier }
%td.follow-col= render partial: 'dossier_actions', locals: { procedure: @procedure, dossier: dossier } %td.follow-col= render partial: 'dossier_actions', locals: { procedure: @procedure, dossier: dossier, dossier_is_followed: @followed_dossiers_id.include?(dossier.id) }
- else - else
%h2 Aucun dossier %h2 Aucun dossier