Development #84325
remettre un Null('deleted_timestamp') systématique dans get_users_with_roles()
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
06 décembre 2023
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Non
Planning:
Non
Description
Commentaire posé dans https://git.entrouvert.org/entrouvert/wcs/pulls/894 qui fait
- criterias = [Null('deleted_timestamp')] ... + if not include_disabled_users: + criterias.append(And([Null('deleted_timestamp'), Equal('is_active', True)]))
Avant tout le temps il y avait Null('deleted_timestamp'), on perd ça; là j'ai l'impression qu'on peut se trouver, si include_disabled_users=True est passé, avec des utilisateurs supprimés. (peut-être que quelque chose fait ailleurs assure qu'en pratique ça n'est pas le cas mais ça serait bien plus facile d'être sûr et d'éviter le cas où ce quelque chose ailleurs bouge, si le Null('deleted_timestamp') était tout le temps assuré ici.
Demandes liées
Historique
Mis à jour par Frédéric Péters il y a 5 mois
- Lié à Development #84155: Sources de données “Utilisateurs” : discriminer directement en SQL l’inclusion des usagers désactivés ajouté