Bug 6584 - Enable searching on Zebra's Bib-level index
Summary: Enable searching on Zebra's Bib-level index
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Searching (show other bugs)
Version: 3.6
Hardware: All All
: PATCH-Sent (DO NOT USE) minor (vote)
Assignee: Marcel de Rooy
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-13 13:16 UTC by Marcel de Rooy
Modified: 2012-10-25 23:09 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Patch (1.50 KB, patch)
2011-07-13 13:19 UTC, Marcel de Rooy
Details | Diff | Splinter Review
6584: Enable searching on Zebra's Bib-level index (1.56 KB, patch)
2011-07-16 22:32 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 Marcel de Rooy 2011-07-13 13:16:30 UTC
Bib-level is already indexed in MARC21 record.abs.
But you cannot search this index, because it is commented in ccl.properties and not listed in getIndexes of Search.pm.
This very simple patch does only do those two things.

Before testing it, search in the OPAC for Bib-level=m. Do you get an error? 
Test this patch now by applying the patch, copy ccl.properties to your zebra config folder (in etc/zebradb folder) and reindex! After that search for Bib-level=m. Probably, you will have many hits. But you should not get an error.
Comment 1 Marcel de Rooy 2011-07-13 13:19:39 UTC Comment hidden (obsolete)
Comment 2 Katrin Fischer 2011-07-16 22:32:45 UTC
Created attachment 4641 [details] [review]
6584: Enable searching on Zebra's Bib-level index

Bib-level is already indexed in MARC21 record.abs.
But you cannot search this index, because it is commented in ccl.properties and
not listed in getIndexes of Search.pm.
This very simple patch does only do those two things.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Comment 3 Paul Poulain 2011-10-06 11:49:25 UTC
very small patch, nothing specific about QA, marking "passed QA"
Comment 4 Chris Cormack 2011-10-07 00:54:18 UTC
Pushed, please test