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

perf(usager): don't query for all dossiers tabs when we already know the tab

Browse source 
Fait passer le controller de 20 à 13 requêtes lorsqu'on est sur l'onglet en cours
This commit is contained in:
Colin Darie 2023-11-30 15:34:34 +01:00
parent 5635ab1a1b
commit 9ac3775207
No known key found for this signature in database
GPG key ID: 8C76CADD40253590
1 changed files with 11 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
app/controllers/users/dossiers_controller.rb
View file

@ -408,21 +408,20 @@ module Users
# else en-cours # else en-cours
def statut(mes_dossiers, dossiers_traites, dossiers_invites, dossiers_supprimes_recemment, dossiers_supprimes_definitivement, dossier_transferes, dossiers_close_to_expiration, params_statut) def statut(mes_dossiers, dossiers_traites, dossiers_invites, dossiers_supprimes_recemment, dossiers_supprimes_definitivement, dossier_transferes, dossiers_close_to_expiration, params_statut)
tabs = { tabs = {
'en-cours' => mes_dossiers.present?, 'en-cours' => mes_dossiers,
'traites' => dossiers_traites.present?, 'traites' => dossiers_traites,
'dossiers-invites' => dossiers_invites.present?, 'dossiers-invites' => dossiers_invites,
'dossiers-supprimes-recemment' => dossiers_supprimes_recemment.present?, 'dossiers-supprimes-recemment' => dossiers_supprimes_recemment,
'dossiers-supprimes-definitivement' => dossiers_supprimes_definitivement.present?, 'dossiers-supprimes-definitivement' => dossiers_supprimes_definitivement,
'dossiers-transferes' => dossier_transferes.present?, 'dossiers-transferes' => dossier_transferes,
'dossiers-expirant' => dossiers_close_to_expiration.present? 'dossiers-expirant' => dossiers_close_to_expiration
} }
if tabs[params_statut]
if tabs[params_statut]&.present?
params_statut params_statut
else else
tabs tab = tabs.find { |_tab, scope| scope.present? }
.filter { |_tab, filled| filled } tab&.first || 'en-cours'
.map { |tab, _| tab }
.first || 'en-cours'
end end
end end