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

backoffice: allow role specific custom view to be default (#77193) #633

Merged
fpeters merged 1 commits from wip/77193-custom-default-view-by-role into main 2023-09-05 19:49:00 +02:00
Conversation 0 Commits 1 Files Changed 2 +50 -10
fpeters commented 2023-09-04 20:42:12 +02:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/77193-custom-default-view-by-role from 68e875bbd2 to 29e5648585 2023-09-04 21:04:55 +02:00 Compare
fpeters changed title from WIP: backoffice: allow role specific custom view to be default (#77193) to backoffice: allow role specific custom view to be default (#77193) 2023-09-04 21:39:46 +02:00
lguerin reviewed 2023-09-05 08:54:05 +02:00
wcs/backoffice/management.py Outdated
@ -829,0 +829,4 @@
# search for first default role custom view
user = get_request().user
if user:
user_role_ids = user.get_roles()
lguerin commented 2023-09-05 08:54:02 +02:00
Owner
Copy Link

Les rôles d'un user sont toujours ordonnés de la même façon ? Si un user a plusieurs rôles, et qu'il existe une vue par défaut pour plus d'un de ses rôles, le résultat sera toujours le même ?

Les rôles d'un user sont toujours ordonnés de la même façon ? Si un user a plusieurs rôles, et qu'il existe une vue par défaut pour plus d'un de ses rôles, le résultat sera toujours le même ?
fpeters commented 2023-09-05 09:47:58 +02:00
Author
Owner
Copy Link

Il n'y a pas de problème par rapport à la liste des rôles, l'itération principale se fait sur la liste des vues, mais si jamais l'ordre de celle-ci changeait, on pourrait en effet voir une vue qui change, j'ai ajouté cette modification au patch :

-        for view in get_publisher().custom_view_class.select(clause=criterias):
+        for view in get_publisher().custom_view_class.select(clause=criterias, order_by='id'):

pour garantir qu'on récupère toujours les listes dans le même ordre.

Il n'y a pas de problème par rapport à la liste des rôles, l'itération principale se fait sur la liste des vues, mais si jamais l'ordre de celle-ci changeait, on pourrait en effet voir une vue qui change, j'ai ajouté cette modification au patch : ``` - for view in get_publisher().custom_view_class.select(clause=criterias): + for view in get_publisher().custom_view_class.select(clause=criterias, order_by='id'): ``` pour garantir qu'on récupère toujours les listes dans le même ordre.
fpeters force-pushed wip/77193-custom-default-view-by-role from 29e5648585 to 8c3da9d46e 2023-09-05 09:46:24 +02:00 Compare
lguerin approved these changes 2023-09-05 09:53:31 +02:00
fpeters merged commit 71e6af5a4e into main 2023-09-05 19:49:00 +02:00
fpeters deleted branch wip/77193-custom-default-view-by-role 2023-09-05 19:49:00 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#633
No description provided.
Delete Branch "wip/77193-custom-default-view-by-role"

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?