[Koha-bugs] [Bug 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Lucas Gass changed: What|Removed |Added Resolution|--- |FIXED Status|Pushed to oldstable |RESOLVED -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Blocks||34592 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34592 [Bug 34592] The Patron Search window, given just a sort field value, doesn't work -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m Version(s)|23.11.00,22.11.08 |23.11.00,23.05.02,22.11.08 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #49 from Matt Blenkinsop --- Nice work everyone! Pushed to oldstable for 22.11.x -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Matt Blenkinsop changed: What|Removed |Added Version(s)|23.11.00|23.11.00,22.11.08 released in|| Status|Pushed to stable|Pushed to oldstable -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Martin Renvoize changed: What|Removed |Added Status|Needs documenting |Pushed to stable --- Comment #48 from Martin Renvoize --- Needed for simple backport of 34094 and the tree above it. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Martin Renvoize changed: What|Removed |Added Blocks||34094 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34094 [Bug 34094] Apply DefaultPatronSearchMethod to all patron search forms -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Martin Renvoize changed: What|Removed |Added Status|Pushed to master|Needs documenting -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com --- Comment #47 from Martin Renvoize --- This has wider reaching impact than the title suggests.. and I'm not sure we want to force exposure of these fields for all cases.. as such I'm opting not to backport this one right now (though I am backporting most of the other patron search improvements as they go a long way towards fixing the bugs in patron search people have been highlighting when switching to 22.11.x) -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Jonathan Druart changed: What|Removed |Added Keywords|additional_work_needed | -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Lucas Gass changed: What|Removed |Added Blocks||34233 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34233 [Bug 34233] Pickup location pulldowns in request.pl are blank -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Lucas Gass changed: What|Removed |Added Blocks|34233 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34233 [Bug 34233] Pickup location pulldowns in request.pl are blank -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Lucas Gass changed: What|Removed |Added Blocks||34233 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34233 [Bug 34233] Pickup location pulldowns in request.pl are blank -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Blocks||34209 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34209 [Bug 34209] Follow up on Bug 28726 - move whole search header div into checkbox column condition -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #46 from Michael Hafen --- I'm guessing you are seeing that on the patron search popup from a page other then the patron card creator. I'll open another bug with a follow up patch to move the whole div into the checkbox column condition. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Jonathan Druart changed: What|Removed |Added Keywords||additional_work_needed --- Comment #45 from Jonathan Druart --- This empty div is visible when I search for patrons in the popup: https://snipboard.io/2zBDoA.jpg -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #44 from Jonathan Druart --- (In reply to Jonathan Druart from comment #43) > This patch is actually adding Sort1 and Sort2 to all patrons searches. This is not coming from this 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #43 from Jonathan Druart --- This patch is actually adding Sort1 and Sort2 to all patrons searches. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #42 from Tomás Cohen Arazi --- Pushed to master for 23.11. 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Tomás Cohen Arazi changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||23.11.00 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Marcel de Rooy changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.org | -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Marcel de Rooy changed: What|Removed |Added Attachment #148489|0 |1 is obsolete|| --- Comment #41 from Marcel de Rooy --- Created attachment 150650 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150650=edit Bug 28726: Add sort1 and sort2 to patron card creator - patron search. Also adds a select all / clear all and checkboxes. Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. Observe that Category and Library are the only options. 3. Close search for patron window. 4. Apply patch. 5. click the Add patron(s) button. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search and observe the 'Select all | Clear all | Add selected patrons' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card batch's Add by borrowernumber(s) text input field. 9. close the search for patron window. Signed-off-by: Solene Ngamga Signed-off-by: Marcel de Rooy -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Marcel de Rooy changed: What|Removed |Added Patch complexity|Medium patch|Small patch 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #40 from Marcel de Rooy --- (In reply to Michael Hafen from comment #39) Thanks. > 8. use the Add selected patrons button to add the selected patrons to the > card batch's Add by borrowernumber(s) text input field in the main window. This is the confusing part. Since I clicked on that button before selecting the patrons, it is quite strange that you should click it again to kind of confirm the action you did. It is outside the scope of this report, but the operation is not intuitive. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #39 from Michael Hafen --- Let me try to amend the test plan with more detail. I was just able to apply this patch to a current master and get through the test plan successfully, but there are details missing from the test plan that could cause confusion if one isn't familiar with the module. Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. This will open the 'Patron search' window. Observe that Category and Library are the only options. 3. Close 'Patron search' window. 4. Apply patch. Restart Plack, etc. 5. click the Add patron(s) button. This will reopen the 'Patron search' window. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search using a sort1 or sort2 value that will have multiple results. Observe the 'Select all | Clear all | Add selected patrons' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card batch's Add by borrowernumber(s) text input field in the main window. 9. close the 'Patron search' window. Click the Add patron(s) button again to add the patrons to the batch. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #38 from Marcel de Rooy --- If I am adding borrowers to the patron card batch, I am still seeing the box "There are no patrons in this batch" and I have no possibility to go further? Could be that I am missing config data for this module still, but this form should be more helpful. Probably out of scope, but it limits testing of this change. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #37 from Marcel de Rooy --- Looking 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/
[Koha-bugs] [Bug 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Marcel de Rooy changed: What|Removed |Added Attachment #147477|0 |1 is obsolete|| -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Marcel de Rooy changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #36 from solene.nga...@inlibro.com --- Created attachment 148489 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148489=edit Bug 28726: Add sort1 and sort2 to patron card creator - patron search. Also adds a select all / clear all and checkboxes. Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. Observe that Category and Library are the only options. 3. Close search for patron window. 4. Apply patch. 5. click the Add patron(s) button. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search and observe the 'Select all | Clear all | Add selected patrons' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card batch's Add by borrowernumber(s) text input field. 9. close the search for patron window. Signed-off-by: Solene Ngamga -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 solene.nga...@inlibro.com changed: What|Removed |Added Status|Needs Signoff |Signed Off -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #35 from Michael Hafen --- I didn't even realize that the sql search would be case-insensitive. Now that you mention it, of course it is. Anyway, here's a patch that just pulls from AV Bsort1 and Bsort2, using av-build-dropbox.inc, so it's either a select or an input. I can do otherwise on my local fork if it upsets my librarians (which I expect it will). -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Attachment #143646|0 |1 is obsolete|| --- Comment #34 from Michael Hafen --- Created attachment 147477 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=147477=edit Bug 28726: Add sort1 and sort2 to patron card creator - patron search. [rebased 2023-02-27] [changed to pass QA - use av-build-dropbox.inc] Also adds a select all / clear all and checkboxes. Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. Observe that Category and Library are the only options. 3. Close search for patron window. 4. Apply patch. 5. click the Add patron(s) button. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search and observe the 'Select all | Clear all | Add selected patrons' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card batch's Add by borrowernumber(s) text input field. 9. close the search for patron window. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #33 from Katrin Fischer --- We'll need something that works universally. The auto-complete seemed like a good idea, but I'd like Kyle and others to comment on performance concerns. Koha's tables are case insensitive in the database - Koha doesn't care about it in searches that are SQL based. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #32 from Michael Hafen --- The patron entry form and the batch patron mod form are both for setting the value, which is a different circumstance from using a value as a filter in a patron search. For those entry forms having either the AV drop down or a free text input makes sense. For a patron search it's easier to use a drop down, where the librarian probably want's an exact match to a value used in the database. In the case of my libraries most of our patron information is controlled by LDAP, and the librarians aren't the ones entering the information in the LDAP Directory. So they have an idea what to look for, but stuff like capitalization, or whether first name or last name comes first, isn't known. Or remembering 40 to 90 values is a burden I would spare them from. Does that explain my use case? -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #31 from Katrin Fischer --- (In reply to Michael Hafen from comment #30) > Making it a free text field doesn't work for me. In my opinion having a > free text input doesn't work well for a filter field on a search page; > especially in my case where the librarians don't control that field, it's > controlled by LDAP. > > Katrin mentioned the Patron Statistics Wizard. Looking at that, what's done > for the sort2 column is to pull distinct values limited to 200. That would > work for me, that should be well under what my librarians would encounter in > that field. > > Would doing that instead be acceptable? It won't work for any library that stored an identifier or unique value in there that has more than 200 patrons. They would see an incomplete pull down. Also: a 200 entry pull down is not really nice to use? Actually I was not aware the wizard did that :( we should fix it as well. I'll claim the patron form then as the example and the batch patron modification templates (sorry). Why will a free text not work? If they have no idea what they are looking for, will also not be very helpful? I think it's a specific use case we are dealing with here that I'd like to understand better. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #30 from Michael Hafen --- Making it a free text field doesn't work for me. In my opinion having a free text input doesn't work well for a filter field on a search page; especially in my case where the librarians don't control that field, it's controlled by LDAP. Katrin mentioned the Patron Statistics Wizard. Looking at that, what's done for the sort2 column is to pull distinct values limited to 200. That would work for me, that should be well under what my librarians would encounter in that field. Would doing that instead be acceptable? -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #29 from Katrin Fischer --- I think maybe it will be easier to keep it simple and have a text field or a pull down from AV depending on config first. As Kyle said sort fields can have unique values, so you will have an endless pull down with one entry per patron, this is not useful. Also: pull down and free text will match how this field is used in all other places in Koha (= existing pattern). Further improvements could then be made on a follow-up bug. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #28 from Michael Hafen --- The patch right now is trying the Authorized Values first, and if those are empty, then it hits the database. Kyle, are you suggesting an auto-complete style text input? I could do that instead, though I'd have to figure out how. That would still mean a lot of database queries though. Maybe a (not unique) index on each of those two columns? (assuming we don't already have one.) I don't think we can't avoid querying the database at this point. As long as sort1 and sort2 are free entry fields on the member entry page, we can't avoid querying the database to collect those values. And we need those values in order to use them as a filter when searching members. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #27 from Katrin Fischer --- (In reply to Kyle M Hall from comment #26) > I don't think we should be pre-populating a select pulldown from sort1/2. > These fields could contain unique information, and a library could have tens > or hundreds of thousands of patrons. In that case bad things (tm) would > happen. I think it would be best of those fields were free text fields. Just to clarify, if AV values exist, would you be ok with populating from the config (authorised values) instead of the data? If the category has no entries, we do freetext, otherwise a pull down. That matches the behviour in other spots, like the patron edit/display forms and the statistical wizard for patrons. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Kyle M Hall changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #26 from Kyle M Hall --- I don't think we should be pre-populating a select pulldown from sort1/2. These fields could contain unique information, and a library could have tens or hundreds of thousands of patrons. In that case bad things (tm) would happen. I think it would be best of those fields were free text fields. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Kyle M Hall changed: What|Removed |Added CC||k...@bywatersolutions.com Attachment #141405|0 |1 is obsolete|| -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Laura changed: What|Removed |Added CC||Laura.escamilla@bywatersolu ||tions.com Status|Needs Signoff |Signed Off -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #25 from ByWater Sandboxes --- Created attachment 143646 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143646=edit Bug 28726: Add sort1 and sort2 to patron card creator - patron search. Also adds a select all / clear all and checkboxes. Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. Observe that Category and Library are the only options. 3. Close search for patron window. 4. Apply patch. 5. click the Add patron(s) button. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search and observe the 'Select all | Clear all | Add selected patrons' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card batch's Add by borrowernumber(s) text input field. 9. close the search for patron window. Signed-off-by: Laura Escamilla -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Status|In Discussion |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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Attachment #138653|0 |1 is obsolete|| --- Comment #24 from Michael Hafen --- Created attachment 141405 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141405=edit Bug 28726: Add sort1 and sort2 to patron card creator - patron search. [ 2022-10-05 updated for recommendations: AV and select2 ] Bug 28726: Add sort1 and sort2 to patron card creator - patron search. Also adds a select all / clear all and checkboxes. Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. Observe that Category and Library are the on ly options. 3. Close search for patron window. 4. Apply patch. 5. click the Add patron(s) button. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search and observe the 'Select all | Clear all | Add selected patro ns' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card b atch's Add by borrowernumber(s) text input field. 9. close the search for patron window. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #23 from Katrin Fischer --- I cannot tell about performance, but UX wise, maybe switching to a Select2 style pull down could be a compromise (like the ones used in the edit item form). -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Status|Failed QA |In Discussion --- Comment #22 from Michael Hafen --- I'll take a look, but it will mean a big change to how I'm handling patrons, as my use case is a little different. I'm in a school district, and all our patron info comes in from LDAP. We aren't using the Authorized Values categories for the sort fields. I wasn't even planning on trying to use them since our servers handle building those drop down pretty well as is. We also have Independant Branches on, so the list gets filtered by branchcode, so I'd also have to see if the Authorized Values categories can do that too. We have 72079 patrons with 2975 unique sort2 values, but each library has no more than 200 values in the sort2, and it's usually closer to or less than 100 values. Even as the SuperLibrarian, when I load the patron card create patron search window it doesn't take more than a second on my servers, so I've been fine with it that way. I can see that this isn't optimal, but the alternative is creating a cron script to scrape LDAP to pull all the possible values for those two attributes to sync the Authorized Values categories. Anyway, I'll have to look at the Authorized Values stuff to see if it can meet our use case. And I don't mind changing the patch to use that instead, and then changing it again in my local fork to scrape the database instead. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #21 from Katrin Fischer --- To explain a little more: even with just the sample data the pull down is a little hard to handle. If we get over 100 entries there, it will be much harder. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #20 from Katrin Fischer --- Hi Michaeal, could you take a look at changing the form as suggested in comment#19? The patron edit form could be a good starting place for this. I realize there is another place in Koha at least where it's done like you do here - the statistics wizard for patrons - but I believe it would be worth changing and changing it there too. I can easily see a library using the field for individual notes. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Katrin Fischer changed: What|Removed |Added Attachment #138552|0 |1 is obsolete|| -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #19 from Jonathan Druart --- Thanks Katrin, that's exactly what I wanted to say but some parts got lost when I rephrased the comment apparently. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #18 from Katrin Fischer --- (In reply to Jonathan Druart from comment #17) > +@sort1 = sort {$a cmp $b} uniq( > Koha::Patrons->search($sort_filter)->get_column('sort1') ); > > This is terrible if you have lot of patrons. You should at least do the > unique (distinct) at the DBMS level. > > What about the following suggestion: you could display sort1 and sort2 > authorised values in the dropdown list, or simply display a search input. > You could also add it to the table's headers (filter at the top of the > column). sort1 can be free text or an AV. The AV category always exists, but maybe we could check for existing values. If values exist: show a drop down with the values. If there are no entries for the AV, use a free text input field. That would match the behavior on the patron edit form. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Jonathan Druart changed: What|Removed |Added CC||jonathan.druart+koha@gmail. ||com --- Comment #17 from Jonathan Druart --- +@sort1 = sort {$a cmp $b} uniq( Koha::Patrons->search($sort_filter)->get_column('sort1') ); This is terrible if you have lot of patrons. You should at least do the unique (distinct) at the DBMS level. What about the following suggestion: you could display sort1 and sort2 authorised values in the dropdown list, or simply display a search input. You could also add it to the table's headers (filter at the top of the column). -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Victor Grousset/tuxayo changed: What|Removed |Added Patch complexity|--- |Medium 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Emmanuel Bétemps changed: What|Removed |Added Status|Needs Signoff |Signed Off -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #16 from sandbo...@biblibre.com --- Created attachment 138653 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138653=edit Bug 28726: Add sort1 and sort2 to patron card creator - patron search. Also adds a select all / clear all and checkboxes. Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. Observe that Category and Library are the only options. 3. Close search for patron window. 4. Apply patch. 5. click the Add patron(s) button. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search and observe the 'Select all | Clear all | Add selected patrons' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card batch's Add by borrowernumber(s) text input field. 9. close the search for patron window. Signed-off-by: Emmanuel Bétemps -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Status|Failed QA |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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Attachment #137854|0 |1 is obsolete|| --- Comment #15 from Michael Hafen --- Created attachment 138552 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138552=edit Bug 28726: Add sort1 and sort2 to patron card creator - patron search. Re-upload (2022-08-03). Don't know how the previous one was uploaded as just the commit message. Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. Observe that Category and Library are the only options. 3. Close search for patron window. 4. Apply patch. 5. click the Add patron(s) button. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search and observe the 'Select all | Clear all | Add selected patrons' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card batch's Add by borrowernumber(s) text input field. 9. close the search for patron window. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #12 from Katrin Fischer --- Ok, I have the same error. Chris said it could be related to git-bz not working with ipv6. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Emmanuel Bétemps changed: What|Removed |Added CC||e.bete...@gmail.com --- Comment #9 from Emmanuel Bétemps --- I got this message after applying the patch (which is not displayed in the KohaGitLog) (I test using a sandbox) --- Bug 28726 - Add sort1 and sort2 to patron card creator patron search 137854 - Bug 28726: Add sort1 and sort2 to patron card creator - patron search. Apply? [(y)es, (n)o, (i)nteractive] Traceback (most recent call last): File "/usr/bin/git-bz", line 2716, in applied = do_apply(bug_ref) File "/usr/bin/git-bz", line 1827, in do_apply users = bug.server.get_xmlrpc_proxy().User.get({ 'names': [patch.attacher] })['users'] File "/usr/lib/python2.7/xmlrpclib.py", line 1243, in __call__ return self.__send(self.__name, args) File "/usr/lib/python2.7/xmlrpclib.py", line 1602, in __request verbose=self.__verbose File "/usr/lib/python2.7/xmlrpclib.py", line 1283, in request return self.single_request(host, handler, request_body, verbose) File "/usr/lib/python2.7/xmlrpclib.py", line 1331, in single_request response.msg, xmlrpclib.ProtocolError: -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #13 from Katrin Fischer --- (In reply to Katrin Fischer from comment #12) > Ok, I have the same error. Chris said it could be related to git-bz not > working with ipv6. Ok, we did some more testing and git-bz works on other bugs. I am not sure what's causing the problem here, but it appears to be bug specific and so probably not network related. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #11 from Katrin Fischer --- Hi Emmanuel, what kind of environment are you using for testing? -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #10 from Michael Hafen --- (In reply to Emmanuel Bétemps from comment #9) > I got this message after applying the patch (which is not displayed in the > KohaGitLog) > (I test using a sandbox) > > --- > Bug 28726 - Add sort1 and sort2 to patron card creator patron search > > 137854 - Bug 28726: Add sort1 and sort2 to patron card creator - patron > search. > > Apply? [(y)es, (n)o, (i)nteractive] Traceback (most recent call last): > File "/usr/bin/git-bz", line 2716, in > applied = do_apply(bug_ref) > File "/usr/bin/git-bz", line 1827, in do_apply > users = bug.server.get_xmlrpc_proxy().User.get({ 'names': > [patch.attacher] })['users'] > File "/usr/lib/python2.7/xmlrpclib.py", line 1243, in __call__ > return self.__send(self.__name, args) > File "/usr/lib/python2.7/xmlrpclib.py", line 1602, in __request > verbose=self.__verbose > File "/usr/lib/python2.7/xmlrpclib.py", line 1283, in request > return self.single_request(host, handler, request_body, verbose) > File "/usr/lib/python2.7/xmlrpclib.py", line 1331, in single_request > response.msg, > xmlrpclib.ProtocolError: bugs.koha-community.org/xmlrpc.cgi: 404 Not Found> To be that looks like a problem with the koha-community.org git server. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Katrin Fischer changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #14 from Katrin Fischer --- Michael, please check the attache file, this doesn't appear to be a correct patch file, but only the commit message. -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Katrin Fischer changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff --- Comment #8 from Katrin Fischer --- Please don't forget to switch back the status when you rebase :) -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Attachment #134003|0 |1 is obsolete|| --- Comment #7 from Michael Hafen --- Created attachment 137854 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=137854=edit Bug 28726: Add sort1 and sort2 to patron card creator - patron search. [ Rebased 2022-07-18 ] -- 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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Katrin Fischer changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #6 from Katrin Fischer --- Hi Michaeal, could you please rebase against the master branch? Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 28726: Add sort1 and sort2 to patron card creator - patron search. error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc). error: could not build fake ancestor Patch failed at 0001 Bug 28726: Add sort1 and sort2 to patron card creator - patron search. 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-28726---Add-sort1-and-sort2-to-patron-card-cre-61BF84.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 28726] Add sort1 and sort2 to patron card creator patron search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Katrin Fischer changed: What|Removed |Added Summary|Add sort1 and sort2 to |Add sort1 and sort2 to |Patron Card Creator - |patron card creator patron |Patron Search |search -- 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 28726] Add sort1 and sort2 to Patron Card Creator - Patron Search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Attachment #122991|0 |1 is obsolete|| Attachment #129203|0 |1 is obsolete|| --- Comment #5 from Michael Hafen --- Created attachment 134003 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=134003=edit Bug 28726 - Add sort1 and sort2 to patron card creator - patron search. Add sort1 and sort2 filters, checkboxes and select/clear all links, and a add selected button to the patron search (for the patron card creator). Rebase after change to how the patron search works. Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. Observe that Category and Library are the only options. 3. Close search for patron window. 4. Apply patch. 5. click the Add patron(s) button. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search and observe the 'Select all | Clear all | Add selected patrons' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card batch's Add by borrowernumber(s) text input field. 9. close the search for patron window. -- 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 28726] Add sort1 and sort2 to Patron Card Creator - Patron Search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #4 from The Minh Luong --- Created attachment 129203 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129203=edit Bug 28726 - Add sort1 and sort2 to patron card creator - patron search. Also adds a select all / clear all and checkboxes. Signed-off-by: The Minh Luong -- 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 28726] Add sort1 and sort2 to Patron Card Creator - Patron Search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #3 from Michael Hafen --- Test plan: 1. start a new batch in the patron card creator (Tools -> Patron card creator -> New -> card batch) 2. click the Add patron(s) button. Observe that Category and Library are the only options. 3. Close search for patron window. 4. Apply patch. 5. click the Add patron(s) button. Observe that you can now search for patrons by their sort1 and sort2 values. 6. perform a search and observe the 'Select all | Clear all | Add selected patrons' links and button. 7. use the Select all link to select all the patrons found by the search. 8. use the Add selected patrons button to add the selected patrons to the card batch's Add by borrowernumber(s) text input field. 9. close the search for patron window. -- 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 28726] Add sort1 and sort2 to Patron Card Creator - Patron Search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Owen Leonard changed: What|Removed |Added Version|unspecified |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 28726] Add sort1 and sort2 to Patron Card Creator - Patron Search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Owen Leonard changed: What|Removed |Added Assignee|cnighswon...@foundations.ed |michael.ha...@washk12.org |u | -- 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 28726] Add sort1 and sort2 to Patron Card Creator - Patron Search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com --- Comment #2 from Lucas Gass --- Hey Micheal, Can you provide a test plan for testing this 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 28726] Add sort1 and sort2 to Patron Card Creator - Patron Search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 Michael Hafen changed: What|Removed |Added Status|NEW |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 28726] Add sort1 and sort2 to Patron Card Creator - Patron Search
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28726 --- Comment #1 from Michael Hafen --- Created attachment 122991 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122991=edit Proposed 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/