[Bug 34592] New: The Patron Search window, given just a sort field value, doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Bug ID: 34592 Summary: The Patron Search window, given just a sort field value, doesn't work Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Label/patron card printing Assignee: cnighswonger@foundations.edu Reporter: michael.hafen@washk12.org QA Contact: testopia@bugs.koha-community.org Depends on: 28726 In the patron card creator, card batch. When doing a patron search in the patron search window, if only a sort field value is given the form gets submitted and the patron search window comes back without the sort fields. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 [Bug 28726] Add sort1 and sort2 to patron card creator patron search -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Michael Hafen <michael.hafen@washk12.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|cnighswonger@foundations.ed |michael.hafen@washk12.org |u | Status|NEW |Needs Signoff --- Comment #1 from Michael Hafen <michael.hafen@washk12.org> --- Created attachment 154703 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154703&action=edit Bug 34592: Patron Search filter fields needs to be at least an empty array -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|The Patron Search window, |The patron search window, |given just a sort field |given just a sort field |value, doesn't work |value, doesn't work -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- The attachment has only a test plan, no patch! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Michael Hafen <michael.hafen@washk12.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #154703|0 |1 is obsolete| | --- Comment #3 from Michael Hafen <michael.hafen@washk12.org> --- Created attachment 154742 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154742&action=edit ug 34592: Patron Search filter fields needs to be at least an empty array (Sorry. I don't know how that happened.) If buildPatronSearchQuery() doesn't return an empty array when it exits early, then the code that adds the sort fields to the filters will fail. Test plan: If there is not all ready a patron with a sort1 or sort2 value from the bSort1 or bSort2 authorized values, add an authorized value if necessary, add a patron if necessary, and set the patrons sort field to the added value. In Tools -> Patron card creator start a new card batch. Click on Add Patron(s) to open the patron search window. pick the value in either the sort1 or sort2 drop down. Submit the page. The page will reload and there won't be results. Apply the patch, close the patron search window, clear the browser cache, reload the "New patron card batch" page, and open the patron search window again. Repeat the search. There will be results. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Michael Hafen <michael.hafen@washk12.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Owen Leonard <oleonard@myacpl.org> 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.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #154742|0 |1 is obsolete| | --- Comment #4 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 154802 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154802&action=edit Bug 34592: Patron Search filter fields needs to be at least an empty array If buildPatronSearchQuery() doesn't return an empty array when it exits early, then the code that adds the sort fields to the filters will fail. Test plan: - If there is not all ready a patron with a sort1 or sort2 value from the bSort1 or bSort2 authorized values, add an authorized value if necessary. - Add a patron if necessary, and set the patrons sort field to the added value. - In Tools -> Patron card creator start a new card batch. - Click on Add Patron(s) to open the patron search window. - Pick the value in either the sort1 or sort2 drop down. - Submit the page. The page will reload and there won't be results. - Apply the patch, close the patron search window, clear the browser cache, reload the "New patron card batch" page, and open the patron search window again. - Repeat the search. There will be results. Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |katrin.fischer@bsz-bw.de |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #154802|0 |1 is obsolete| | --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 154806 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154806&action=edit Bug 34592: Patron Search filter fields needs to be at least an empty array If buildPatronSearchQuery() doesn't return an empty array when it exits early, then the code that adds the sort fields to the filters will fail. Test plan: - If there is not all ready a patron with a sort1 or sort2 value from the bSort1 or bSort2 authorized values, add an authorized value if necessary. - Add a patron if necessary, and set the patrons sort field to the added value. - In Tools -> Patron card creator start a new card batch. - Click on Add Patron(s) to open the patron search window. - Pick the value in either the sort1 or sort2 drop down. - Submit the page. The page will reload and there won't be results. - Apply the patch, close the patron search window, clear the browser cache, reload the "New patron card batch" page, and open the patron search window again. - Repeat the search. There will be results. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |23.11.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 --- Comment #6 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 23.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart+koha@gmail. | |com --- Comment #7 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Any changes to patron search should add selenium tests. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|23.11.00 |23.11.00,23.05.04 released in| | Status|Pushed to master |Pushed to stable CC| |fridolin.somers@biblibre.co | |m --- Comment #8 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 23.05.x for 23.05.04 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |Pushed to oldstable Version(s)|23.11.00,23.05.04 |23.11.00,23.05.04,22.11.10 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 --- Comment #9 from Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> --- Nice work everyone! Pushed to oldstable for 22.11.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 Bug 34592 depends on bug 28726, which changed state. Bug 28726 Summary: Add sort1 and sort2 to patron card creator patron search https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to oldstable |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org