ne pas inclure la fonction attribuée au niveau du formdef dans le filtrage sur les fonctions de l'usager (#76682) #301

Merged
fpeters merged 1 commits from wip/76682-merged-roles-dict into main 2023-11-11 08:17:02 +01:00
Owner
No description provided.
fpeters added 1 commit 2023-05-09 14:27:23 +02:00
gitea/wcs/pipeline/head There was a failure building this commit Details
86a5c2740d
maybe, check tests
fpeters force-pushed wip/76682-merged-roles-dict from 86a5c2740d to 86dee5ff29 2023-11-09 15:59:49 +01:00 Compare
fpeters force-pushed wip/76682-merged-roles-dict from 86dee5ff29 to b832eae882 2023-11-09 16:22:45 +01:00 Compare
fpeters force-pushed wip/76682-merged-roles-dict from b832eae882 to cc7b9d4755 2023-11-09 17:49:02 +01:00 Compare
fpeters force-pushed wip/76682-merged-roles-dict from b8bd948661 to 29030bf0f0 2023-11-09 18:33:12 +01:00 Compare
fpeters changed title from WIP: ne pas inclure la fonction attribuée au niveau du formdef dans le filtrage sur les fonctions de l'usager (#76682) to ne pas inclure la fonction attribuée au niveau du formdef dans le filtrage sur les fonctions de l'usager (#76682) 2023-11-10 13:53:08 +01:00
Author
Owner

Il y avait un comportement incohérent entre le contrôle d'accès et ce qui sortait via une requête sur la db lors du filtrage sur les fonctions de l'usager : sur ce filtrage, une fonction attribuée au niveau du formulaire était prise en compte, même quand le formdata l'overridait, alors qu'ailleurs seul l'override était considéré.

Ce patch modifie la création de workflow_merged_roles_dict (qui est la colonne sur laquelle la requête est réalisée) pour correspondre.

Ça a un impact sur un test artificiel dans test_sql.py + le test de l'API de filtrage sur la fonction, qui sont mis à jour.

Je ne pense pas que ça ait un impact sur les usages réels.

Il y avait un comportement incohérent entre le contrôle d'accès et ce qui sortait via une requête sur la db lors du filtrage sur les fonctions de l'usager : sur ce filtrage, une fonction attribuée au niveau du formulaire était prise en compte, même quand le formdata l'overridait, alors qu'ailleurs seul l'override était considéré. Ce patch modifie la création de workflow_merged_roles_dict (qui est la colonne sur laquelle la requête est réalisée) pour correspondre. Ça a un impact sur un test artificiel dans test_sql.py + le test de l'API de filtrage sur la fonction, qui sont mis à jour. Je ne pense pas que ça ait un impact sur les usages réels.
lguerin approved these changes 2023-11-10 17:19:24 +01:00
fpeters merged commit 2ab30cd6b8 into main 2023-11-11 08:17:02 +01:00
fpeters deleted branch wip/76682-merged-roles-dict 2023-11-11 08:17:02 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 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#301
No description provided.