[Koha-bugs] [Bug 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Kyle M Hall changed: What|Removed |Added Resolution|--- |FIXED Status|Pushed to master|RESOLVED CC||k...@bywatersolutions.com -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 --- Comment #11 from Jonathan Druart --- Pushed to master for 21.11, thanks to everybody involved! -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Jonathan Druart changed: What|Removed |Added Version(s)||21.11.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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Katrin Fischer changed: What|Removed |Added Attachment #124242|0 |1 is obsolete|| --- Comment #10 from Katrin Fischer --- Created attachment 126624 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126624=edit Bug 16446: (follow-up) Fix translation and adjust radio buttons Signed-off-by: Katrin Fischer -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Katrin Fischer changed: What|Removed |Added Attachment #124182|0 |1 is obsolete|| --- Comment #9 from Katrin Fischer --- Created attachment 126623 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126623=edit Bug 16446: Add ability to add patrons to list by borrowernumber AddPatronsToList already took a borrowernumber parameter, however, it did not check if those were valid numbers. This patch expands the search to apply to cardnumber or borrowernumbers in the subroutine. Template and script are adjusted to allow choosing borrowernumbers or cardnumbers To test: 1 - Apply patch 2 - Browse to Tools->Patron lists 3 - Create a list, or choose Actions->Add patrons for an existing list 4 - Click 'Add multiple patrons' 5 - Cardnumbers is preselected 6 - Enter a list of cardnumbers, ensure you test: Cardnumber already in list Cardnumber not in list Non-existent cardnumber 7 - Patrons should be added/errors reported correctly 8 - Repeat with borowernumbers Signed-off-by: Andrew Fuerste-Henry Signed-off-by: Katrin Fischer -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Passed QA Patch complexity|--- |Small 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 --- Comment #8 from Nick Clemens --- Created attachment 124242 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124242=edit Bug 16446: (follow-up) Fix translation and adjust radio buttons -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Nick Clemens changed: What|Removed |Added Status|Failed QA |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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 --- Comment #7 from Katrin Fischer --- This is only one of the lines I think are not good. Please make sure cardnumber or borrowernumber remain translatable throughout instead of using the variables to replace the text. Otherwise this looks pretty good to me. QA tools and tests pass. As translation length of strings may vary I think I'd like the text for the radio box after the radio box instead of below. It kind of looks strange to me that there are not center or beginning of the text now. -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #6 from Katrin Fischer --- Nice try, Nick, but you have been caught by the translation police ;) +[% id_column | html FILTER ucfirst %] not found -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Andrew Fuerste-Henry changed: What|Removed |Added Attachment #124177|0 |1 is obsolete|| --- Comment #5 from Andrew Fuerste-Henry --- Created attachment 124182 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124182=edit Bug 16446: Add ability to add patrons to list by borrowernumber AddPatronsToList already took a borrowernumber parameter, however, it did not check if those were valid numbers. This patch expands the search to apply to cardnumber or borrowernumbers in the subroutine. Template and script are adjusted to allow choosing borrowernumbers or cardnumbers To test: 1 - Apply patch 2 - Browse to Tools->Patron lists 3 - Create a list, or choose Actions->Add patrons for an existing list 4 - Click 'Add multiple patrons' 5 - Cardnumbers is preselected 6 - Enter a list of cardnumbers, ensure you test: Cardnumber already in list Cardnumber not in list Non-existent cardnumber 7 - Patrons should be added/errors reported correctly 8 - Repeat with borowernumbers Signed-off-by: Andrew Fuerste-Henry -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Andrew Fuerste-Henry 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Nick Clemens changed: What|Removed |Added CC||n...@bywatersolutions.com Assignee|gmcha...@gmail.com |n...@bywatersolutions.com -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 --- Comment #4 from Nick Clemens --- Created attachment 124177 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124177=edit Bug 16446: Add ability to add patrons to list by borrowernumber AddPatronsToList already took a borrowernumber parameter, however, it did not check if those were valid numbers. This patch expands the search to apply to cardnumber or borrowernumbers in the subroutine. Template and script are adjusted to allow choosing borrowernumbers or cardnumbers To test: 1 - Apply patch 2 - Browse to Tools->Patron lists 3 - Create a list, or choose Actions->Add patrons for an existing list 4 - Click 'Add multiple patrons' 5 - Cardnumbers is preselected 6 - Enter a list of cardnumbers, ensure you test: Cardnumber already in list Cardnumber not in list Non-existent cardnumber 7 - Patrons should be added/errors reported correctly 8 - Repeat with borowernumbers -- 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Nick Clemens 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 16446] Allow librarians to add borrowers to patron lists by borrowernumber
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16446 Katrin Fischer changed: What|Removed |Added Component|Lists |Tools Version|3.22|master Summary|Allow librarians to add |Allow librarians to add |borrowers to lists by |borrowers to patron lists |borrowernumber or |by borrowernumber |cardnumber | --- Comment #3 from Katrin Fischer --- There is an option to enter cardnumbers now, but no option for borrowernumbers. Changing the bug title slightly. -- 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/