[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #21 from Fridolin Somers --- (In reply to David Cook from comment #3) > In Master: > 1) Turn off QueryAutoTruncate > 2) Go to > http://localhost:8081/cgi-bin/koha/catalogue/search.pl?q=GEN=location > 3) Note "No results found" > > The problem is that QueryWeightFields adds a "phr" qualifier which is a > problem since "location" doesn't have a phrase index defined...: > > (rk=( location,ext,r1=" GEN " or location,phr,r3=" GEN " or > location,wrdl,r6=" GEN " or location,wrdl,fuzzy,r8=" GEN " or > location,wrdl,rt,r9=""gen"? ")) We have seen other problems like that. We should add phrase search to all search fields to fix once and for all. But for this cas, it disables autotruncation of location code, so its a good fix. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 Lucas Gass changed: What|Removed |Added Resolution|--- |FIXED Status|Pushed to stable|RESOLVED CC||lu...@bywatersolutions.com --- Comment #20 from Lucas Gass --- Doesn't apply cleanly to 22.05.x, no backport. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #19 from Jacob O'Mara --- Nice work, thanks everyone! Pushed to 22.11.x for the next release. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 Jacob O'Mara changed: What|Removed |Added Status|Pushed to master|Pushed to stable Version(s)|23.05.00|23.05.00,22.11.05 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #18 from Tomás Cohen Arazi --- Pushed to master for 23.05. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 Tomás Cohen Arazi changed: What|Removed |Added Version(s)||23.05.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 Nick Clemens changed: What|Removed |Added Attachment #148856|0 |1 is obsolete|| --- Comment #17 from Nick Clemens --- Created attachment 148916 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148916=edit Bug 31841: Use location,wrdl when searching Shelving location The "location" index only has a word index defined, so we need to specify the wrdl qualifier when searching it via the staff interface dropdown menu Test plan: 1) Change QueryAutoTruncate to "only if * is added" 2) Change IntranetCatalogSearchPulldown to "Show" 3) Search for "Shelving location" of "GEN" in catalogue search bar 4) Note no results returned 5) Apply patch 6) koha-plack --restart kohadev 7) Repeat Step #3 8) Note that results appear for 'location,wrdl: GEN' Signed-off-by: Magnus Enger Works as advertised. Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 Nick Clemens changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 Magnus Enger changed: What|Removed |Added Attachment #148848|0 |1 is obsolete|| --- Comment #16 from Magnus Enger --- Created attachment 148856 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148856=edit Bug 31841: Use location,wrdl when searching Shelving location The "location" index only has a word index defined, so we need to specify the wrdl qualifier when searching it via the staff interface dropdown menu Test plan: 1) Change QueryAutoTruncate to "only if * is added" 2) Change IntranetCatalogSearchPulldown to "Show" 3) Search for "Shelving location" of "GEN" in catalogue search bar 4) Note no results returned 5) Apply patch 6) koha-plack --restart kohadev 7) Repeat Step #3 8) Note that results appear for 'location,wrdl: GEN' Signed-off-by: Magnus Enger Works as advertised. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 Magnus Enger changed: What|Removed |Added Status|Needs Signoff |Signed Off Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #15 from Fridolin Somers --- (In reply to David Cook from comment #13) > (In reply to Fridolin Somers from comment #9) > > > since "location" doesn't have a phrase index defined... > > Why not add it to Zebra MARC21 configuration ? > > > > Even if unlikely : > > Imagine two locations "foo bar" and "bar foo", words list type search "wrdl" > > will match both. > > > > This indexing can be useful for other cases. > > It might be worth adding that the status quo right now is doing a "wrdl" > search for "location", so I'm just trying to make the status quo explicit > and functional in all cases. OKi no problem. A fix that works is a good fix ;) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 David Cook changed: What|Removed |Added Attachment #141950|0 |1 is obsolete|| --- Comment #14 from David Cook --- Created attachment 148848 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148848=edit Bug 31841: Use location,wrdl when searching Shelving location The "location" index only has a word index defined, so we need to specify the wrdl qualifier when searching it via the staff interface dropdown menu Test plan: 1) Change QueryAutoTruncate to "only if * is added" 2) Change IntranetCatalogSearchPulldown to "Show" 3) Search for "Shelving location" of "GEN" in catalogue search bar 4) Note no results returned 5) Apply patch 6) koha-plack --restart kohadev 7) Repeat Step #3 8) Note that results appear for 'location,wrdl: GEN' -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 David Cook changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #13 from David Cook --- (In reply to Fridolin Somers from comment #9) > > since "location" doesn't have a phrase index defined... > Why not add it to Zebra MARC21 configuration ? > > Even if unlikely : > Imagine two locations "foo bar" and "bar foo", words list type search "wrdl" > will match both. > > This indexing can be useful for other cases. It might be worth adding that the status quo right now is doing a "wrdl" search for "location", so I'm just trying to make the status quo explicit and functional in all cases. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #12 from David Cook --- Thanks for looking Magnus. It's a simple fix. I'll look at doing that now actually... -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 Magnus Enger changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply CC||mag...@libriotech.no --- Comment #11 from Magnus Enger --- Sorry, patch does not apply and I don't see how to resolve the conflict: $ git bz apply 31841 Bug 31841 - Shelving location search in staff interface sometimes creates invalid Zebra query 141950 - Bug 31841: Use location,wrdl when searching Shelving location Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 31841: Use location,wrdl when searching Shelving location Using index info to reconstruct a base tree... M koha-tmpl/intranet-tmpl/prog/en/includes/search_indexes.inc Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/search_indexes.inc CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/search_indexes.inc error: Failed to merge in the changes. Patch failed at 0001 Bug 31841: Use location,wrdl when searching Shelving location hint: Use 'git am --show-current-patch=diff' to see the failed patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/Bug-31841-Use-locationwrdl-when-searching-Shelving-qsJQXm.patch <<< HEAD [% IF ms_location %] Shelving location (code) === [% IF ms_locationcommawrdl %] Shelving location (code) >>> Bug 31841: Use location,wrdl when searching Shelving location -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #9 from Fridolin Somers --- > since "location" doesn't have a phrase index defined... Why not add it to Zebra MARC21 configuration ? Even if unlikely : Imagine two locations "foo bar" and "bar foo", words list type search "wrdl" will match both. This indexing can be useful for other cases. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #8 from David Cook --- (In reply to Katrin Fischer from comment #7) > Sorry, I got confused :) No worries! I'd love to someone to sign it off though heh. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #7 from Katrin Fischer --- (In reply to David Cook from comment #6) > (In reply to Katrin Fischer from comment #5) > > Hi David, did you mean to sign-off here? > > I'm not sure I understand the question? Sorry, I got confused :) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #6 from David Cook --- (In reply to Katrin Fischer from comment #5) > Hi David, did you mean to sign-off here? I'm not sure I understand the question? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 31841] Shelving location search in staff interface sometimes creates invalid Zebra query
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31841 --- Comment #5 from Katrin Fischer --- Hi David, did you mean to sign-off here? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/