https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33159 --- Comment #11 from Frank Hansen <frank.hansen@ub.lu.se> --- (In reply to Nick Clemens from comment #5)
Created attachment 147970 [details] [review] Bug 33159: Simplify ES handling and fix zebra handling
Before this patch we used two indexes for the thesaurus values, we can simply index both needed fields into a single index and just form the search correctly.
This patch also ensures we pass the 'thesaurus' vlaue for the heading directly to the query builder - for zebra it goes through, and for ES we convert it to the expected code.
This patch also moves the necessary mappings out of the user definable mappings and hardcodes them. There is precedent for this with 'match-heading', it ensures matching works as expected
To test: 1 - Follow previous test plan in Zebra and ES
It seems to work with Zebra now. But for ES, after resetting the mappings and re-index (recreating) the indexes, linking doesn't work anymore. I notices that both Subject-heading-thesaurus and Subject-heading-thesaurus-conventions index was removed from mappings.yaml. Was that a mistake? I thought you want want a single index for both 008/_11 and 040f like Janusz did? I added these lines to the mappings file: @@ -805,6 +805,20 @@ authorities: sort: ~ suggestible: '' type: '' + Subject-heading-thesaurus: + label: Subject-heading-thesaurus + mappings: + - facet: '' + marc_field: 008_/11 + marc_type: marc21 + sort: ~ + suggestible: '' + - facet: '' + marc_field: 040f + marc_type: marc21 + sort: ~ + suggestible: '' + type: thesaurus authtype: label: authtype mappings: After resetting the mappings and re-index, linking works again in ES. Regards, Frank -- You are receiving this mail because: You are watching all bug changes.