Development #49422
Pagination & limit
0%
Description
Dans la pagination utilisée pour le listing des demandes, des cards, et bientôt des erreurs (#48926), le user peut changer la limit. Il a le choix entre 10, 20, 50 et 100.
Dans la pratique, rien de l'empêche de bidouiller l'url et mettre limit=100000.
Pour éviter des pb de perf et de chargement de page, je propose de contrôler que limit est <= 100, et de reset à 20 (ou un default) si ça dépasse.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
J'avais souvenir de Vincennes où on avait monté la valeur par défaut (default-page-size) mais 1/ le site n'est plus utilisé, 2/ on n'avait monté qu'à 200.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0001-misc-be-sure-that-pagination-is-not-too-high-49422.patch 0001-misc-be-sure-that-pagination-is-not-too-high-49422.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
si default-page-size est > à 100, alors c'est le nouveau max
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Le "or 20" pourrait être "or 100" ça m'aurait évité de me demander de quoi que c'est que ce 20, mais allez hop ça marche ainsi.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
J'avais mis 20 parce que c'est le default que j'ai trouvé dans le code.
Mais c'est plus lisible avec 100, et c'est équivalent, je change
Mis à jour par Thomas Noël il y a plus de 3 ans
- Fichier branch.jpeg branch.jpeg ajouté
Mon ack "Solution validée" reste bien valable avec la nouvelle branche.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9497da3b92eec0c36fedcdbe296c438dbb102343
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Thu Dec 17 11:22:03 2020 +0100
misc: be sure that pagination is not too high (#49422)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: be sure that pagination is not too high (#49422)