Project

General

Profile

Development #71992

indexation, ignorer les liens gabarits qui dépendent de la requête

Added by Frédéric Péters 2 months ago. Updated about 2 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
04 December 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision 7f057108 (diff)
Added by Frédéric Péters about 2 months ago

search: ignore links depending on cards/forms/request (#71992)

History

#3

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).

#4

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à).

#6

Updated by Frédéric Péters about 2 months ago

#7

Updated by Frédéric Péters about 2 months ago

avec une mini-correction sur le test pour avoir un gabarit qui ressemble à quelque chose.

#8

Updated by Lauréline Guérin about 2 months ago

  • Status changed from Solution proposée to Solution validée
#9

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)
#10

Updated by Transition automatique about 2 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF