Projet

Général

Profil

Development #31026

enregistrement de l'auteur d'une modif de page

Ajouté par Frédéric Péters il y a environ 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
01 mars 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Ça fonctionne mais c'est parce que django fait un object CallableBool un peu magique.


Fichiers

Révisions associées

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

misc: don't rely on is_anonymous being a CallableBool (#31026)

Historique

#1

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

#2

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

  • Statut changé de Solution proposée à Solution validée
#3

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 5e5d3d4f7276280dc56634f81ba67098d47b705c
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Mar 1 11:23:12 2019 +0100

    misc: don't rely on is_anonymous being a CallableBool (#31026)
#4

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

Juste pour dire que c'est Django 1.8 incompatible, je ne sais pas si c'est notre politique de commencer à se restreindre à Django 1.11.

#5

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

Ça marche très bien en 1.8 aussi; et maintenant que je regarde la direction prise par django est en fait l'inverse, de is_anonymous() méthode à is_anonymous propriété/booléen.

#6

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

Ça marchait parce que bool(x.method) renvoie toujours vrai, donc ça ne crashe pas mais en 1.8 avec ça on ne doit pas logger l'auteur; et oui pardon ma remarque est à l'envers, tu as rendu du code incompatible Django 1.8 compatible.

Coté a2 j'envisage de backporer le CallableBool pour commencer à migrer vers du code Django 1.11 pure, mais bon c'est plus facile quand on a sa propre classe User.

#7

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF