Bug #41226
ne pas inclure les cellules invalides dans le tableau de bord
Début:
01 avril 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
cf #41223; techniquement le is_visible() va être faux, ce qui va faire une 403, ce qui va faire que la cellule va s'afficher avec "undefined" comme erreur.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Fichier 0001-dashboard-skip-invalid-cells-in-dashboards-41226.patch 0001-dashboard-skip-invalid-cells-in-dashboards-41226.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
J'imagine que la technique que tu as choisie ici permet de limiter le nombre de requêtes SQL, mais avais-tu pensé à l'ajout d'une property "is_valid" sur Tile, qui irait chercher le ValidationInfo de chaque et en vérifierait le invalid_since ? Dans l'idée au final d'avoir, dans DashboardCell.render, un lisible tiles = [t for t in tiles if tiles.id_valid()]
?
Mais ce code me va bien aussi, de toute façon avec les Tile faut jouer des coudes :)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
Oui, c'est fait ainsi pour limiter le nombre de requêtes.
commit 9786d9b387df45e38df82f1794539aa6d160ddb0 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Apr 1 13:48:24 2020 +0200 dashboard: skip invalid cells in dashboards (#41226)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
dashboard: skip invalid cells in dashboards (#41226)