entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

data_sources: express user activation filter as sql criteria (#84155) #894

Merged
pmarillonnet merged 1 commits from wip/84155-users-data-sources-activation-criteria-as-sql into main 2023-12-06 11:16:52 +01:00
Conversation 0 Commits 1 Files Changed 2 +8 -5
pmarillonnet commented 2023-12-06 10:00:25 +01:00
Owner
Copy Link
No description provided.
pmarillonnet added 1 commit 2023-12-06 10:00:26 +01:00
pmarillonnet changed title from WIP: data_sources: express user activation filter as sql criteria (#84155) to data_sources: express user activation filter as sql criteria (#84155) 2023-12-06 10:34:32 +01:00
pducroquet approved these changes 2023-12-06 10:48:04 +01:00
pmarillonnet merged commit dae7146dc1 into main 2023-12-06 11:16:52 +01:00
pmarillonnet deleted branch wip/84155-users-data-sources-activation-criteria-as-sql 2023-12-06 11:16:52 +01:00
fpeters reviewed 2023-12-06 11:29:57 +01:00
wcs/users.py
@ -188,2 +190,4 @@
if excluded_roles:
criterias.append(Not(Intersects('roles', [str(r) for r in excluded_roles])))
if not include_disabled_users:
criterias.append(And([Null('deleted_timestamp'), Equal('is_active', True)]))
fpeters commented 2023-12-06 11:29:57 +01:00
Owner
Copy Link

Mais 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.

Mais 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.
pmarillonnet commented 2023-12-06 11:35:33 +01:00
Author
Owner
Copy Link

Mais 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.

Ah ok, mauvaise compréhension de ma part sur la différence entre désactivation et suppression. Je vais faire un nouveau ticket.

> Mais 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. Ah ok, mauvaise compréhension de ma part sur la différence entre désactivation et suppression. Je vais faire un nouveau ticket.
pmarillonnet commented 2023-12-06 11:42:04 +01:00
Author
Owner
Copy Link

Mais 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.

Ah ok, mauvaise compréhension de ma part sur la différence entre désactivation et suppression. Je vais faire un nouveau ticket.

#84326.

> > Mais 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. > > Ah ok, mauvaise compréhension de ma part sur la différence entre désactivation et suppression. Je vais faire un nouveau ticket. #84326.
Sign in to join this conversation.
Reviewers
No reviewers
pducroquet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#894
No description provided.
Delete Branch "wip/84155-users-data-sources-activation-criteria-as-sql"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?