Summary: | Add CNI (Control Number Identifier) search index (MARC21) | ||
---|---|---|---|
Product: | Koha | Reporter: | Martin Renvoize (ashimema) <martin.renvoize> |
Component: | Searching | Assignee: | Martin Renvoize (ashimema) <martin.renvoize> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | kyle, nick, tomascohen |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
21.11.00
|
|
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 21469, 27507, 11175, 26314, 31537, 32416 | ||
Attachments: |
Bug 28830: Add cni index for 003
Bug 28830: Add cni index for 003 Bug 28830: Add cni index for 003 |
Description
Martin Renvoize (ashimema)
2021-08-09 11:02:22 UTC
Created attachment 123616 [details] [review] Bug 28830: Add cni index for 003 This patch adds the cni/Control-number-identifier index to enable searches to use the 003 field. Test plan 1/ Apply patch 2/ Re-index using updated configurations 3/ Confirm cni:number searches yield the expected results 4/ Signoff Split-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Pasi Kallinen <pasi.kallinen@koha-suomi.fi> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> This patch was split out of bug 11175 to allow it to be a dependanchy of other bugs without risking hold ups whilst 11175 makes it way through QA. Signoffs carried across from that bug. Kind of wondering, why it took us THAT long to add this! I checked the LOC documentation, but it appears there is no standard attribute for 003, so using 9014 is fine (https://www.loc.gov/z3950/agency/bib1.html) Martin, could you please add the indexing changes for Elasticsearch in a follow-up? Created attachment 123697 [details] [review] Bug 28830: Add cni index for 003 This patch adds the cni/Control-number-identifier index to enable searches to use the 003 field. Test plan 1/ Apply patch 2/ Re-index using updated configurations 3/ Confirm cni:number searches yield the expected results 4/ Signoff Split-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Pasi Kallinen <pasi.kallinen@koha-suomi.fi> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Good catch, seems I missed the ES bit from the other bug.. added into the patch :) Created attachment 123920 [details] [review] Bug 28830: Add cni index for 003 This patch adds the cni/Control-number-identifier index to enable searches to use the 003 field. Test plan 1/ Apply patch 2/ Re-index using updated configurations 3/ Confirm cni:number searches yield the expected results 4/ Signoff Split-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Pasi Kallinen <pasi.kallinen@koha-suomi.fi> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 21.11, thanks to everybody involved! |