Development #40252
cellules dans des zones désactivées
0%
Description
À passer sur différentes traces issues de l'indexation, je me suis plusieurs fois trouvé avec des cellules posées dans des zones (genre "barre latérale") qui n'étaient plus actives.
On devrait voir pour que ces cellules soient ignorées de l'indexation (c'est quelque chose qui demande de faire le calcul réel des zones/placeholders pour chacune des pages).
Fichiers
Révisions associées
cells: better perfs on invalid cells report (#40252)
cells: exclude cells with inactive placeholders from invalid report
(#40252)
search: add a num queries test on index_site (#40252)
search: better queries for index_site (#40252)
search: don't index cells with an inactive placeholder (#40252)
Historique
Mis à jour par Thomas Noël il y a environ 4 ans
Par "qui ne sont plus actives", tu entends des cellules posées sur un placeholder d'un modèle, puis ce modèle changé le placeholder n'existe plus ?
Mis à jour par Frédéric Péters il y a environ 4 ans
Oui, et du coup pour les retirer/corriger j'ai rebasculé le modèle en "deux colonnes" (exemple), supprimé les cellules de la colonne droite, puis remis le modèle en "une colonne".
Mis à jour par Lauréline Guérin il y a environ 4 ans
note: cela touche aussi le rapport des cellules invalides, qui remonte aussi les cellules dans des placeholders non actifs.
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0007-search-don-t-index-cells-with-an-inactive-placeholde.patch 0007-search-don-t-index-cells-with-an-inactive-placeholde.patch ajouté
- Fichier 0006-search-remove-bad-test-wrong-merge-in-past.patch 0006-search-remove-bad-test-wrong-merge-in-past.patch ajouté
- Fichier 0005-search-better-queries-for-index_site-40252.patch 0005-search-better-queries-for-index_site-40252.patch ajouté
- Fichier 0004-search-add-a-num-queries-test-on-index_site-40252.patch 0004-search-add-a-num-queries-test-on-index_site-40252.patch ajouté
- Fichier 0003-cells-exclude-cells-with-inactive-placeholders-from-.patch 0003-cells-exclude-cells-with-inactive-placeholders-from-.patch ajouté
- Fichier 0002-cells-better-perfs-on-invalid-cells-report-40252.patch 0002-cells-better-perfs-on-invalid-cells-report-40252.patch ajouté
- Fichier 0001-cells-don-t-check-validity-in-is_visible-if-placehol.patch 0001-cells-don-t-check-validity-in-is_visible-if-placehol.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Fred, j'ai choisi de filtrer les cellules à placeholder invalide au moment du report, pour ne pas interférer avec le calcul de l'invalidité.
Car si on fait ça au moment de marquer une cellule invalide, se pose la question: que se passe-t-il lorsqu'on change le modèle de page, doit-on repasser sur toutes les validités ?
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 5bc694d32d542460f92289b611e6ab74ffb54d3c Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 27 17:58:23 2020 +0100 search: don't index cells with an inactive placeholder (#40252) commit ee82f0e26e817532834b60f5e5b9bfe5f43bdd1e Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 27 17:45:51 2020 +0100 search: remove bad test - wrong merge in past ? commit 3464f6d0233cffd14b9d442c83488a17397206ed Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 27 17:42:42 2020 +0100 search: better queries for index_site (#40252) commit a83ce2c5eeaa59135f16afe2eef48b237bde61f4 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 27 15:50:00 2020 +0100 search: add a num queries test on index_site (#40252) commit 50b1101297296b2876e2fd571d6dcd3dce58089b Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 27 15:18:21 2020 +0100 cells: exclude cells with inactive placeholders from invalid report (#40252) commit 453a7840073773db5181ed199bc7f4de86469bae Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 27 14:55:23 2020 +0100 cells: better perfs on invalid cells report (#40252) commit 80500d71a9945544fe7fcd25f6df89cf5374da06 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 27 14:33:59 2020 +0100 cells: don't check validity in is_visible if placeholder search (#40252)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
cells: don't check validity in is_visible if placeholder search (#40252)