Projet

Général

Profil

Development #84325

remettre un Null('deleted_timestamp') systématique dans get_users_with_roles()

Ajouté par Frédéric Péters il y a 5 mois. Mis à jour il y a 5 mois.

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

Lié à w.c.s. - Development #84155: Sources de données “Utilisateurs” : discriminer directement en SQL l’inclusion des usagers désactivésFermé01 décembre 2023

Actions

Historique

#1

Mis à jour par Frédéric Péters il y a 5 mois

  • Description mis à jour (diff)
#2

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é
#3

Mis à jour par Frédéric Péters il y a 5 mois

  • Statut changé de Nouveau à Fermé

Répété en #84326.

Formats disponibles : Atom PDF