Bug 28378 - Elasticsearch - Add 264c to default copydate mappings (MARC21)
Summary: Elasticsearch - Add 264c to default copydate mappings (MARC21)
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Searching - Elasticsearch (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Caroline Cyr La Rose
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-05-18 20:02 UTC by Caroline Cyr La Rose
Modified: 2022-12-12 21:23 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
This enhancement adds 264$c to the default mapping for the copydate index when using Elasticsearch.
Version(s) released in:
21.11.00


Attachments
Bug 28378: Elasticsearch - Add 264c to default copydate mappings (MARC21) (907 bytes, patch)
2021-05-18 20:07 UTC, Caroline Cyr La Rose
Details | Diff | Splinter Review
Bug 28378: Elasticsearch - Add 264c to default copydate mappings (MARC21) (955 bytes, patch)
2021-08-06 11:14 UTC, David Nind
Details | Diff | Splinter Review
Bug 28378: Elasticsearch - Add 264c to default copydate mappings (MARC21) (1003 bytes, patch)
2021-08-10 11:17 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Caroline Cyr La Rose 2021-05-18 20:02:29 UTC
In the current default index mappings, copydate only includes 260$c. I think it should also include 264$c which is used as much as, if not more than, 260$c.
Comment 1 Caroline Cyr La Rose 2021-05-18 20:07:59 UTC
Created attachment 121126 [details] [review]
Bug 28378: Elasticsearch - Add 264c to default copydate mappings (MARC21)

This patch adds 264$c to the default mapping for the copydate index.

To test:
1) Apply patch
2) Reindex using rebuildelasticsearch.pl -r

If you don't have access to a terminal (in a sandbox for example)
2a) Go to Administration > Search engine configuration, click "Reset mappings" 
and confirm
2b) Then reindex

3) Search for a copyright date found in 264$c
Comment 2 David Nind 2021-08-06 11:14:24 UTC
Created attachment 123567 [details] [review]
Bug 28378: Elasticsearch - Add 264c to default copydate mappings (MARC21)

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2021-08-06 11:15:38 UTC
Not sure what I did again when signing off...patch description is not the same as in Bugzilla.
Comment 4 Katrin Fischer 2021-08-10 11:17:43 UTC
Created attachment 123673 [details] [review]
Bug 28378: Elasticsearch - Add 264c to default copydate mappings (MARC21)

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Jonathan Druart 2021-08-16 10:04:44 UTC
Pushed to master for 21.11, thanks to everybody involved!