In the footers of the tables of the search fields and mappings, the options selected are not consistent. Additionally they are not reset correctly when a new line is added.
Created attachment 163678 [details] [review] Bug 36394: Fix footers behaviour on the ES mapping page In the footers of the tables of the search fields and mappings, the options selected are not consistent. Additionally they are not reset correctly when a new line is added. Test plan: 0. Do not apply to recreate the problem 1. Go to the ES mapping page 2. Click tab "Bibliographic records" 3. At the bottom, notice the Yes/No dropdown lists => They are inconsistent, sometimes 'Yes', sometimes 'No' 4. Click Add => Values are kept 5. Apply this patch 6. Repeat 1, 2, 3 => No (first option) is always selected 7. Click Add after you modified some values => "No" is selected Sponsored-by: The Research University in the Helmholtz Association (KIT)
Created attachment 163717 [details] [review] Bug 36394: Fix footers behaviour on the ES mapping page In the footers of the tables of the search fields and mappings, the options selected are not consistent. Additionally they are not reset correctly when a new line is added. Test plan: 0. Do not apply to recreate the problem 1. Go to the ES mapping page 2. Click tab "Bibliographic records" 3. At the bottom, notice the Yes/No dropdown lists => They are inconsistent, sometimes 'Yes', sometimes 'No' 4. Click Add => Values are kept 5. Apply this patch 6. Repeat 1, 2, 3 => No (first option) is always selected 7. Click Add after you modified some values => "No" is selected Sponsored-by: The Research University in the Helmholtz Association (KIT) Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>
Works as described.
Created attachment 164202 [details] [review] Bug 36394: Fix footers behaviour on the ES mapping page In the footers of the tables of the search fields and mappings, the options selected are not consistent. Additionally they are not reset correctly when a new line is added. Test plan: 0. Do not apply to recreate the problem 1. Go to the ES mapping page 2. Click tab "Bibliographic records" 3. At the bottom, notice the Yes/No dropdown lists => They are inconsistent, sometimes 'Yes', sometimes 'No' 4. Click Add => Values are kept 5. Apply this patch 6. Repeat 1, 2, 3 => No (first option) is always selected 7. Click Add after you modified some values => "No" is selected Sponsored-by: The Research University in the Helmholtz Association (KIT) Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Created attachment 164203 [details] [review] Bug 36394: Fix 'Filter' selection in new line All other fields in the new line use data-id instead of name, this fixes an inconsistency To test: 1 - Before this patch select some values, set filter to punctuation - hit 'Add' 2 - Punctuation vlaue is cleared 3 - After patch add a new line and confirm 'Filter' value is saved Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Pushed for 24.05! Well done everyone, thank you!
Pushed to 23.11.x for 23.11.06
Doesn't apply cleanly to 23.05.x, no backport.