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

statistics: keep field items order in filter options (#80523) #703

Merged
vdeniaud merged 1 commits from wip/80523-stats-respecter-l-ordre-des-choi into main 2023-10-30 11:05:28 +01:00
Conversation 0 Commits 1 Files Changed 4 +117 -20
vdeniaud commented 2023-09-21 14:32:01 +02:00
Owner
Copy Link

Voilà il faut juste faire attention à ce que l'astuce d'avoir un champ item simple en donnée de traitement pour y mettre n'importe quoi et que ça apparaisse dans les stats continue à fonctionner (c'est le test que j'ajoute).

Voilà il faut juste faire attention à ce que l'astuce d'avoir un champ item simple en donnée de traitement pour y mettre n'importe quoi et que ça apparaisse dans les stats continue à fonctionner (c'est le test que j'ajoute).
vdeniaud added 1 commit 2023-09-21 14:32:02 +02:00
vdeniaud changed title from WIP: statistics: keep field items order in filter options (#80523) to statistics: keep field items order in filter options (#80523) 2023-09-21 14:51:33 +02:00
fpeters reviewed 2023-10-05 22:29:16 +02:00
wcs/statistics/views.py Outdated
@ -451,0 +450,4 @@
if field.data_source or not field.items:
options = form_page.get_item_filter_options(field, selected_filter='all', anonymised=True)
else:
options = [(x, x) for x in field.items]
fpeters commented 2023-10-05 22:29:16 +02:00
Owner
Copy Link

Je me demande si ça doit vraiment se passer ici, spécifique aux statistiques, je m'imagine que ça aurait sa place dans get_item_filter_options, appliquer le tri en fonctione de l'ordre de field.items quand c'est défini, plutôt que le tri alphabétique.

Je me demande si ça doit vraiment se passer ici, spécifique aux statistiques, je m'imagine que ça aurait sa place dans get_item_filter_options, appliquer le tri en fonctione de l'ordre de field.items quand c'est défini, plutôt que le tri alphabétique.
vdeniaud commented 2023-10-11 12:20:23 +02:00
Author
Owner
Copy Link

Ouep visiblement #35703 qui a introduit l'extraction des options depuis la base réfléchissait uniquement à propos des sources de données, revenir là dessus pour le cas field.items défini ne devrait pas avoir d’inconvénient

Ouep visiblement #35703 qui a introduit l'extraction des options depuis la base réfléchissait uniquement à propos des sources de données, revenir là dessus pour le cas field.items défini ne devrait pas avoir d’inconvénient
vdeniaud force-pushed wip/80523-stats-respecter-l-ordre-des-choi from 31c965afd6 to 247af215d1 2023-10-11 10:37:15 +02:00 Compare
vdeniaud force-pushed wip/80523-stats-respecter-l-ordre-des-choi from 247af215d1 to 1da14109bd 2023-10-11 12:12:31 +02:00 Compare
vdeniaud requested review from fpeters 2023-10-11 12:30:13 +02:00
fpeters approved these changes 2023-10-26 09:37:42 +02:00
vdeniaud force-pushed wip/80523-stats-respecter-l-ordre-des-choi from 1da14109bd to e554c04bf1 2023-10-30 09:29:19 +01:00 Compare
vdeniaud merged commit e554c04bf1 into main 2023-10-30 11:05:28 +01:00
vdeniaud deleted branch wip/80523-stats-respecter-l-ordre-des-choi 2023-10-30 11:05:28 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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#703
No description provided.
Delete Branch "wip/80523-stats-respecter-l-ordre-des-choi"

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?