Développement #75187
open
custom_user : mettre à plat les usages de User.userexternalid (?)
Added by Paul Marillonnet almost 3 years ago.
Updated almost 3 years ago.
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 to Développement #75181: clean-unused-accounts : envois abusifs de notification de suppression de comptes en recette (?) added
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.
- Status changed from Nouveau to Information nécessaire
- Assignee set to Paul Marillonnet
- 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.
Also available in: Atom
PDF