Développement #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é.
Related issues
History
Updated by Paul Marillonnet about 2 years ago
- Related to Développement #75181: clean-unused-accounts : envois abusifs de notification de suppression de comptes en recette (?) added
Updated by Benjamin Dauvergne about 2 years ago
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.
Updated by Benjamin Dauvergne about 2 years ago
- Status changed from Nouveau to Information nécessaire
- Assignee set to Paul Marillonnet
Updated by Paul Marillonnet about 2 years ago
- Status changed from Information nécessaire to 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.