backoffice: update data-multi-values attribute on criteria change (#84748) #933

Merged
fpeters merged 1 commits from wip/84748-js-criteria-multi-remove into main 2024-01-12 14:43:00 +01:00
Owner
No description provided.
fpeters added 1 commit 2023-12-14 16:13:03 +01:00
fpeters reviewed 2023-12-14 16:14:17 +01:00
@ -526,3 +526,3 @@
$copy.find('select').val('').trigger('change');
values = values.filter(item => item !== value)
if($value_select.length) {
$value_select.attr('data-multi-values', values.join('|'));
Author
Owner

Cette partie-ci correspond au cas présentement évoqué et ça a l'air d'aller.

Cette partie-ci correspond au cas présentement évoqué et ça a l'air d'aller.
@ -531,3 +531,4 @@
.attr('value', values.join('|'));
}
else {
$copy.find('select').val('').trigger('change');
Author
Owner

ici c'est pour garder le même code que précédemment dans la branche qui n'est pas testée, parce que peut-être que là il servait.

ici c'est pour garder le même code que précédemment dans la branche qui n'est pas testée, parce que peut-être que là il servait.
pmarillonnet approved these changes 2024-01-02 16:21:00 +01:00
fpeters merged commit 59753af856 into main 2024-01-12 14:43:00 +01:00
fpeters deleted branch wip/84748-js-criteria-multi-remove 2024-01-12 14:43:00 +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#933
No description provided.