Projet

Général

Profil

Bug #6366

Ne plus remplir manuellement request.user

Ajouté par Frédéric Péters il y a environ 9 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
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

Révision cb31fd9b (diff)
Ajouté par Frédéric Péters il y a environ 9 ans

misc: don't overwrite get_request().user property (#6366)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 9 ans

#2

Mis à jour par Thomas Noël il y a environ 9 ans

Ack

#3

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

Mis à jour par Thomas Noël il y a plus de 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF