Projet

Général

Profil

Development #59922

Rendre str(user) équivalent à user.get_full_name()

Ajouté par Valentin Deniaud il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
20 décembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

La seule utilité de la méthode str de User c'est de se retrouver utilisée dans repr, à part ça je ne trouve pas d'endroit où elle est appelée.

À côté de ça, la méthode get_full_name est utilisée à mille endroits.

Mon cas d'usage derrière ça c'est que dans les vues manager où on mixe utilisateurs et rôles, on se retrouve à avoir une condition chaque fois qu'on veut afficher un objet pour savoir quelle méthode appeler et c'est pas pratique.


Fichiers

Révisions associées

Révision cdda4a98 (diff)
Ajouté par Valentin Deniaud il y a plus de 2 ans

custom_user: return get_full_name result in str (#59922)

Historique

#1

Mis à jour par Valentin Deniaud il y a plus de 2 ans

Voilà, je ne cherche pas à supprimer tous les usages de get_full_name, je le fais juste aux endroits où ça simplifie un peu le code.

je ne trouve pas d'endroit où elle est appelée

Au final j'ai trouvé un endroit dans le journal, qui oblige donc à modifier les tests, mais je pense que cet appel était plus fortuit que voulu.

#2

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

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

Ok.

#3

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit cdda4a98bf2f18d56b7578bd45750a59df2a55c7
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Mon Dec 20 14:05:26 2021 +0100

    custom_user: return get_full_name result in __str__ (#59922)
#4

Mis à jour par Frédéric Péters il y a plus de 2 ans

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

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF