Development #71992
indexation, ignorer les liens gabarits qui dépendent de la requête
0%
Description
Via #69330, on a des liens type http://example.net/?foo={{ cards|objects:"foo"|filter_by_user:request.user|first|get:"id" }} et pendant l'indexation des liens pour la recherche le rendu du gabarit échoue.
Il y a deux choses ici, 1/ le request.user, que de toute façon on n'aura pas, 2/ le cards qui vient via un context_processor qu'on n'a pas (parce qu'on a un simple dictionnaire comme contexte).
J'imagine des situations légitimes où on pourrait avoir un lien {{ cards|objects:"foo"|filter_by:"x"|filter_value:page.slug|first|get:"y" }} (sur le coup on n'a pas non plus page dans le contexte, mais c'est pour l'idée).
Après avoir hésité je serais pour oublier ça, juste zapper ces liens.
Files
Associated revisions
History
Updated by Guillaume Baffoin 2 months ago
À mon avis, c'est aussi le problème de #71701 (le user qui n'est pas défini en cron).
Updated by Frédéric Péters 2 months ago
À mon avis, c'est aussi le problème de #71701 (le user qui n'est pas défini en cron).
Non c'est autre chose là. (je commente là).
Updated by Frédéric Péters about 2 months ago
- File 0001-search-ignore-links-depending-on-cards-forms-request.patch 0001-search-ignore-links-depending-on-cards-forms-request.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Frédéric Péters about 2 months ago
- File 0001-search-ignore-links-depending-on-cards-forms-request.patch 0001-search-ignore-links-depending-on-cards-forms-request.patch added
avec une mini-correction sur le test pour avoir un gabarit qui ressemble à quelque chose.
Updated by Lauréline Guérin about 2 months ago
- Status changed from Solution proposée to Solution validée
Updated by Frédéric Péters about 2 months ago
- Status changed from Solution validée to Résolu (à déployer)
commit 7f05710818c3a3fe97cb21d63740b22bfb02df0e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Dec 4 10:02:56 2022 +0100
search: ignore links depending on cards/forms/request (#71992)
Updated by Transition automatique about 2 months ago
- Status changed from Résolu (à déployer) to Solution déployée
search: ignore links depending on cards/forms/request (#71992)