Project

General

Profile

Développement #59922

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

Added by Valentin Deniaud almost 3 years ago. Updated almost 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
20 December 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files

Associated revisions

Revision cdda4a98 (diff)
Added by Valentin Deniaud almost 3 years ago

custom_user: return get_full_name result in str (#59922)

History

#1

Updated by Valentin Deniaud almost 3 years ago

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

Updated by Benjamin Dauvergne almost 3 years ago

  • Status changed from Solution proposée to Solution validée

Ok.

#3

Updated by Valentin Deniaud almost 3 years ago

  • Status changed from Solution validée to 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

Updated by Frédéric Péters almost 3 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#5

Updated by Transition automatique almost 3 years ago

Automatic expiration

Also available in: Atom PDF