Summary: | arp - Accelerated reader point searches fail due to conflicting attribute | ||
---|---|---|---|
Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
Component: | Searching - Zebra | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | normal | ||
Priority: | P5 - low | CC: | arthur.suzuki, david, lucas, m.de.rooy, tomascohen, wainuiwitikapark |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: |
This fixes
|
Version(s) released in: |
23.05.00,22.11.03,22.05.10,21.11.16
|
Circulation function: | |||
Bug Depends on: | 20078, 28830 | ||
Bug Blocks: | 32741 | ||
Attachments: |
Bug 32416: Update attribute number for arp search to 9015
Bug 32416: Add a warning to the file Bug 32416: Update attribute number for arp search to 9015 Bug 32416: Add a warning to the file Bug 32416: Update attribute number for arp search to 9015 Bug 32416: Add a warning to the file |
Description
Nick Clemens (kidclamp)
2022-12-07 12:54:55 UTC
Created attachment 144465 [details] [review] Bug 32416: Update attribute number for arp search to 9015 Bug 20078 updated the attribute for arp to 2014 to avoid conflict with 9013 not-on-loan-count Bug 28830 then added Control-number-identifier as 2014, breaking arp again This patch updates the number to 9015 To test: 1 - Apply first patch 2 - Attempt searching by arp, no results (add a unique 526$d to a record to ease searching) 3 - Apply this patch 4 - Copy bib1.att and ccl.properties to the correct locations cp etc/zebradb/biblios/etc/bib1.att /etc/koha/zebradb/biblios/etc/bib1.att cp etc/zebradb/ccl.properties /etc/koha/zebradb/ccl.properties 5 - Restart zebra 6 - Rebuild indexes 7 - Search again, success! Created attachment 144466 [details] [review] Bug 32416: Add a warning to the file Created attachment 144519 [details] [review] Bug 32416: Update attribute number for arp search to 9015 Bug 20078 updated the attribute for arp to 2014 to avoid conflict with 9013 not-on-loan-count Bug 28830 then added Control-number-identifier as 2014, breaking arp again This patch updates the number to 9015 To test: 1 - Apply first patch 2 - Attempt searching by arp, no results (add a unique 526$d to a record to ease searching) 3 - Apply this patch 4 - Copy bib1.att and ccl.properties to the correct locations cp etc/zebradb/biblios/etc/bib1.att /etc/koha/zebradb/biblios/etc/bib1.att cp etc/zebradb/ccl.properties /etc/koha/zebradb/ccl.properties 5 - Restart zebra 6 - Rebuild indexes 7 - Search again, success! Signed-off-by: David Nind <david@davidnind.com> Created attachment 144520 [details] [review] Bug 32416: Add a warning to the file Signed-off-by: David Nind <david@davidnind.com> Testing notes (using KTD): 1. Used notes from bug 20078 2. Added to a record: . 526$c (arl) - 5.5 . 526$d (arp) - 14 3. Used arp:14 in search (both staff interface and OPAC) Created attachment 145713 [details] [review] Bug 32416: Update attribute number for arp search to 9015 Bug 20078 updated the attribute for arp to 2014 to avoid conflict with 9013 not-on-loan-count Bug 28830 then added Control-number-identifier as 2014, breaking arp again This patch updates the number to 9015 To test: 1 - Apply first patch 2 - Attempt searching by arp, no results (add a unique 526$d to a record to ease searching) 3 - Apply this patch 4 - Copy bib1.att and ccl.properties to the correct locations cp etc/zebradb/biblios/etc/bib1.att /etc/koha/zebradb/biblios/etc/bib1.att cp etc/zebradb/ccl.properties /etc/koha/zebradb/ccl.properties 5 - Restart zebra 6 - Rebuild indexes 7 - Search again, success! Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 145714 [details] [review] Bug 32416: Add a warning to the file Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Hi y'all. I'm pushing this, but this clearly deserves a regression test IMHO. I have actually filed bug 32741 in which I provide such regression tests, but would love your help fixing the other cases we didn't catch here :-D Pushed to master for 23.05. Nice work everyone, thanks! Nice work everyone! Pushed to stable for 22.11.x Backported to 22.05.x for upcoming 22.05.10 applied to 21.11.x for 21.11.16 Not backported to 21.05.x |