[Bug 22760] New: cataloging search yields no results if queryparser is on
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Bug ID: 22760 Summary: cataloging search yields no results if queryparser is on Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Searching Assignee: koha-bugs@lists.koha-community.org Reporter: mikael.olangcay-brisebois@inLibro.com QA Contact: testopia@bugs.koha-community.org Target Milestone: --- If QueryParser is on and defaultsortorder is set to relevance,dsc, when you do a cataloging search, first page yields results but other pages don't return any results half the time. It also seems to break the sort by popularity, but that is not adressed by this patch. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Mikaël Olangcay Brisebois <mikael.olangcay-brisebois@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 --- Comment #1 from Mikaël Olangcay Brisebois <mikael.olangcay-brisebois@inLibro.com> --- Created attachment 88536 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=88536&action=edit Bug 22760: Correct parseQuery regex to match full words when QueryParser is on Test plan: 1. Set system preferences QueryParser to Try and set defaultsortorder/defaultsortfield to relevance,dsc. 2. Search the catalog with a keyword that will return more than 1 page of results. 3. Alternate between the page results a few times. 1/2 times, the page will return no results. 4. Apply patch. 5. Do steps 2-3 and verify that it works. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Mikaël Olangcay Brisebois <mikael.olangcay-brisebois@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mikael.olangcay-brisebois@i | |nLibro.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Mikaël Olangcay Brisebois <mikael.olangcay-brisebois@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |mikael.olangcay-brisebois@i |ity.org |nLibro.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Mikaël Olangcay Brisebois <mikael.olangcay-brisebois@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|cataloging search yields no |catalogue search yields no |results if queryparser is |results if queryparser is |on |on -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Margaret <margaret@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |margaret@bywatersolutions.c | |om -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 --- Comment #2 from Margaret <margaret@bywatersolutions.com> --- Hello Mikaël, Can you give me a little more context for this testing step: "Alternate between the page results a few times. 1/2 times, the page will return no results." Does this mean paginating through the results? Or clicking into results details, then back to results, then in to details and back again? Thanks, Margaret -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #88536|0 |1 is obsolete| | --- Comment #3 from Liz Rea <wizzyrea@gmail.com> --- Created attachment 89968 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=89968&action=edit Bug 22760: Correct parseQuery regex to match full words when QueryParser is on Signed-off-by: Liz Rea <wizzyrea@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Bin Wen <bin.wen@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #89968|0 |1 is obsolete| | --- Comment #4 from Bin Wen <bin.wen@inlibro.com> --- Created attachment 90082 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90082&action=edit Bug 22760: Correct parseQuery regex to match full words when QueryParser is on Signed-off-by: Liz Rea <wizzyrea@gmail.com> Signed-off-by: Bin Wen <bin.wen@inlibro.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Bin Wen <bin.wen@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bin.wen@inlibro.com --- Comment #5 from Bin Wen <bin.wen@inlibro.com> --- Test plan: 1. Set system preferences QueryParser to Try and set defaultsortorder/defaultsortfield to relevance,dsc. 2. Search the catalog with a keyword that will return more than 1 page of results. 3. Go to page 2, page 1,page 2, page 1,Alternate between the page results a few times, the page will return no results. 4. Apply patch. 5. Do steps 2-3 and verify that it works. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de Status|Signed Off |Failed QA --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I am sorry, but I can't replicate the problem with the test plan provided. There are also some other smaller issues: - Please include a description of the fix and the test plan in your commit message. - Can you please provide a regression test to this change in Search.pm? It's quite a sensitive area. ... and also verify the problem still appears and make sure the test plan is correct. I set QueryParser to Try and searched for "de" in the sample database. I then switched back between page 1 and 2 using the navigation on top of the result list with the links labelled the same. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |wizzyrea@gmail.com --- Comment #7 from Liz Rea <wizzyrea@gmail.com> --- I was able to replicate this, as I've seen this error before. When I tested, I clicked through the numbered pages and each time with the original settings, unpatched, one of the pages would end up with "no results found." The behaviour did not appear with the patch installed. You're right about the unit test though. :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22760 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |RESOLVED CC| |dcook@prosentient.com.au Resolution|--- |INVALID --- Comment #8 from David Cook <dcook@prosentient.com.au> --- QueryParser doesn't exist anymore so this should be invalid -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org