backoffice: use current card values in filter option labels (#84145) #887

Merged
fpeters merged 1 commits from wip/84145-card-item-filter-labels into main 2023-12-04 16:50:48 +01:00
Owner
No description provided.
fpeters force-pushed wip/84145-card-item-filter-labels from ae84d61546 to 7f81974b77 2023-12-03 18:56:01 +01:00 Compare
fpeters changed title from WIP: backoffice: use current card values in filter option labels (#84145) to backoffice: use current card values in filter option labels (#84145) 2023-12-03 19:12:22 +01:00
fpeters reviewed 2023-12-03 19:15:39 +01:00
@ -1066,2 +1066,2 @@
if filter_field.key == 'items':
options = list(sorted(filter_field.get_exploded_options(options), key=lambda x: x[1]))
if filter_field.key == 'item':
options = list(sorted(filter_field.get_filter_options(options), key=lambda x: (x[1] or '')))
Author
Owner

Il y a dans les tests des libellés qui sont à None, ça ne devrait pas arriver en vrai mais on s'en prémunit quand même ici.

Il y a dans les tests des libellés qui sont à None, ça ne devrait pas arriver en vrai mais on s'en prémunit quand même ici.
@ -235,0 +236,4 @@
def get_carddef_options_by_ids(self, carddef, options_ids):
if carddef.id_template:
cards = carddef.data_class().select([Contains('id_display', options_ids)])
return [(str(x.id_display), x.get_display_label()) for x in cards]
Author
Owner

Si on est sur des id personnalisés, il faut filtrer sur id_display, et retourner id_display comme id.

Ce cas n'était pas à gérer avant, vu que les valeurs d'id étaient tirées de la db. (sauf pour le cas "ItemsField", qui se trouve ainsi corrigé).

Si on est sur des id personnalisés, il faut filtrer sur id_display, et retourner id_display comme id. Ce cas n'était pas à gérer avant, vu que les valeurs d'id étaient tirées de la db. (sauf pour le cas "ItemsField", qui se trouve ainsi corrigé).
lguerin approved these changes 2023-12-04 16:39:15 +01:00
fpeters merged commit c57ec6fca8 into main 2023-12-04 16:50:48 +01:00
fpeters deleted branch wip/84145-card-item-filter-labels 2023-12-04 16:50:48 +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#887
No description provided.