Bug #16460
ajouter context_processor.request par défaut dans TEMPLATE_CONTEXT_PROCESSORS
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
23 mai 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
On a plusieurs endroits qui font :
if not (getattr(context['request'], 'user', None) and context['request'].user.is_authenticated()):
C'est-à-dire qui pensent que le contexte a toujours une entrée 'request'. On l'ajoute effectivement la plupart du temps, mais je me dis qu'on gagnerait à l'ajouter explicitement une bonne fois pour toutes, via django.template.context_processors.request dans TEMPLATE_CONTEXT_PROCESSORS
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 7 ans
Surtout que dans debian_config_common.py on met déjà systématiquement celui-ci.
Mis à jour par Thomas Noël il y a presque 7 ans
- Fichier 0001-add-django-context_processors.request-in-context-pro.patch 0001-add-django-context_processors.request-in-context-pro.patch ajouté
- Patch proposed changé de Non à Oui
Voilà, en première étape d'un travail plus général de nettoyage des contextes.
Mis à jour par Frédéric Péters il y a presque 7 ans
warnings.warn(
"django.core.context_processors is deprecated in favor of "
"django.template.context_processors.",
RemovedInDjango110Warning, stacklevel=2)
Plutôt donc prendre celui de django.template.etc.
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Nouveau à Résolu (à déployer)
Actualisé et poussé,
commit 925e2efd445922cc8bd7cb873750d661e7b06c9b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Aug 15 16:29:07 2019 +0200
settings: add django context_processors.request to context processors (#16460)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
settings: add django context_processors.request to context processors (#16460)