Bug #6366
Ne plus remplir manuellement request.user
Début:
28 janvier 2015
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Il y a au début du traitement d'une requête dans auquotidien ce code,
def _q_traverse(self, path): session = get_session() if session: get_request().user = session.get_user() else: get_request().user = None
Ça a comme résultat de supprimer le getter qu'est get_request().user, ce qui n'est pas l'effet recherché. Comme de nos jours le getter, justement, existe, ces lignes pourraient être supprimées. (à noter que si HTTPRequest était une new-style class ça provoquerait une erreur (AttributeError: can't set attribute)).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 9 ans
- Fichier 0001-misc-don-t-overwrite-get_request-.user-property-6366.patch 0001-misc-don-t-overwrite-get_request-.user-property-6366.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 9 ans
- Statut changé de En cours à Résolu (à déployer)
commit cb31fd9b7b4fe01a270d5a19ed7aff9ec0cfef54 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Jan 28 14:50:03 2015 +0100 misc: don't overwrite get_request().user property (#6366)
misc: don't overwrite get_request().user property (#6366)