Bug #28093
indexation des cellules faisant référence à "page"
Début:
16 novembre 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Dans le rendu d'une cellule on fournit une variable "page" :
def render_cell(request, cell): context = { 'page': cell.page if cell.page_id else None,
mais dans le code qui fait le rendu pour l'indexation, on ne l'a pas, et on peut se trouver avec des messages genre :
Exception while resolving variable 'page' in template 'variants/toodego/combo/json/decheterie.html'. Traceback (most recent call last): File "/usr/lib/python2.7/dist-packages/django/template/base.py", line 903, in _resolve_lookup (bit, current)) # missing attribute
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-misc-give-search-renderer-all-context-variables-2809.patch 0001-misc-give-search-renderer-all-context-variables-2809.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
À défaut d'un test que je vois pas trop comment faire, ajouter un petit commentaire qui rappelle qu'ici on doit avoir un contexte identique à ce qui est envoyé en rendu normal ? Quoique bon, on l'oubliera toujours :) Ack ainsi.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
Ouaip, on oubliera, surtout je mise sur l'idée de ne pas trop toucher à ça…
commit 139237f01e315eb65085714aa9aef0a1ce217f35 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Nov 16 17:34:51 2018 +0100 misc: give search renderer all context variables (#28093)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: give search renderer all context variables (#28093)