Development #75187
custom_user : mettre à plat les usages de User.userexternalid (?)
0%
Description
On y fait référence à plusieurs endroits dans le code, mais il semble que soit l’accesseur inverse de la FK UserExternalId.user ait été retiré, soit que le bout de code qui initialise expressément User.userexternalid ait disparu (?)
Toujours est-il que, à jouer dans les tests, je n’arrive pas à tomber sur un user pour lequel cet attribut userexternalid est initialisé.
Demandes liées
Historique
Mis à jour par Paul Marillonnet il y a environ un an
- Lié à Development #75181: clean-unused-accounts : envois abusifs de notification de suppression de comptes en recette (?) ajouté
Mis à jour par Benjamin Dauvergne il y a environ un an
Cet accesseur n'existe pas, UserExternalId a une FK vers User donc c'est userexternalid_set le seul accesseur qui existe et c'est un Manager pas une instance de modèle. Je ne sais pas d'où ça sort.
Mis à jour par Benjamin Dauvergne il y a environ un an
- Statut changé de Nouveau à Information nécessaire
- Assigné à mis à Paul Marillonnet
Mis à jour par Paul Marillonnet il y a environ un an
- Statut changé de Information nécessaire à Nouveau
Dang, j’ai fait un git blame, et c’est moi, et c’est récent en plus…
Dans src/authentic/manager/user_views.py
:
947c325cd (Paul Marillonnet 2023-02-13 14:49:19 +0100 457) getattr(self.object, 'userexternalid', None) 947c325cd (Paul Marillonnet 2023-02-13 14:49:19 +0100 458) and getattr(self.object.userexternalid, 'source', None) 947c325cd (Paul Marillonnet 2023-02-13 14:49:19 +0100 459) and self.object.userexternalid.source in realms
Je ne sais pas à quel moment j’ai cru qu’un tel bout de code pourrait marcher, mais il faudrait le remplacer par la version correcte, et en profiter pour ajouter un test de cas positif, qui passe dans cet embranchement.