[Koha-bugs] [Bug 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 wainuiwitikap...@catalyst.net.nz changed: What|Removed |Added CC||wainuiwitikapark@catalyst.n ||et.nz --- Comment #17 from wainuiwitikap...@catalyst.net.nz --- Not backported to 21.05.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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Arthur Suzuki changed: What|Removed |Added Status|Pushed to oldstable |Pushed to oldoldstable CC||arthur.suz...@biblibre.com Version(s)|23.05.00,22.11.03,22.05.10 |23.05.00,22.11.03,22.05.10, released in||21.11.16 --- Comment #16 from Arthur Suzuki --- applied to 21.11.x for 21.11.16 -- 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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com Version(s)|23.05.00,22.11.03 |23.05.00,22.11.03,22.05.10 released in|| Status|Pushed to stable|Pushed to oldstable --- Comment #15 from Lucas Gass --- Backported to 22.05.x for upcoming 22.05.10 -- 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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 --- Comment #14 from Matt Blenkinsop --- Nice work everyone! Pushed to stable 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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Matt Blenkinsop changed: What|Removed |Added Version(s)|23.05.00|23.05.00,22.11.03 released in|| Status|Pushed to master|Pushed to stable -- 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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 --- Comment #13 from Tomás Cohen Arazi --- Pushed to master for 23.05. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Tomás Cohen Arazi changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||23.05.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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com Severity|enhancement |trivial -- 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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Kyle M Hall changed: What|Removed |Added Attachment #138946|0 |1 is obsolete|| --- Comment #12 from Kyle M Hall --- Created attachment 144691 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=144691=edit Bug 18247: Remove SQL queries from branch_transfer_limit.pl 1. Visit Administration > Patrons and circulation > Limit transfer limits 2. Observe collection codes are displayed as tabs on the Limit transfer limits page when the BranchTransferLimitsType syspref = 'Collection code' 3. Switch the BranchTransferLimitsType syspref = 'item type'. Refresh the Limit transfer limits page and observe the tabs have changed to item type codes 4. Apply patchset and restart services 5. Reload the Limit transfer limits page and confirm it displays correctly with the BranchTransferLimitsType = 'collection code' and 'item type' 6. Confirm you can successfully save transfer policies Sponsored-by: Catalyst IT Signed-off-by: Jacob Omara Signed-off-by: Kyle M Hall -- 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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Kyle M Hall changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Kyle M Hall changed: What|Removed |Added CC||k...@bywatersolutions.com QA Contact|testo...@bugs.koha-communit |k...@bywatersolutions.com |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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 jacob.om...@ptfs-europe.com changed: What|Removed |Added CC||jacob.om...@ptfs-europe.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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Alex Buckley changed: What|Removed |Added Keywords||Academy -- 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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Alex Buckley changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #11 from Alex Buckley --- I've replaced the Get* methods in my original patch with Koha::YourObjects->find, or ->search and you suggested Jonathan in comment #9 Ready for testing! Many thanks, Alex -- 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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 --- Comment #10 from Alex Buckley --- Created attachment 138946 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138946=edit Bug 18247: Remove SQL queries from branch_transfer_limit.pl 1. Visit Administration > Patrons and circulation > Limit transfer limits 2. Observe collection codes are displayed as tabs on the Limit transfer limits page when the BranchTransferLimitsType syspref = 'Collection code' 3. Switch the BranchTransferLimitsType syspref = 'item type'. Refresh the Limit transfer limits page and observe the tabs have changed to item type codes 4. Apply patchset and restart services 5. Reload the Limit transfer limits page and confirm it displays correctly with the BranchTransferLimitsType = 'collection code' and 'item type' 6. Confirm you can successfully save transfer policies Sponsored-by: Catalyst 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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Alex Buckley changed: What|Removed |Added Attachment #62586|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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 --- Comment #9 from Jonathan Druart--- The Get* method you added with this patch can be replaced with Koha::YourObjects->find, or ->search No need to add new methods. For instance GetBranchCodes can be replaced with my @branchcodes = Koha::Libraries->search->get_column('branchcode'); -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Jonathan Druartchanged: What|Removed |Added Status|Signed Off |Failed QA --- Comment #8 from Jonathan Druart --- Alex, same as bug 18291 comment 6: you should not need to add new method. Ping me on IRC if you need more details. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Jonathan Druartchanged: What|Removed |Added Version|17.05 |unspecified CC||jonathan.dru...@bugs.koha-c ||ommunity.org -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Marc Véronchanged: 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 http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Marc Véronchanged: What|Removed |Added Attachment #62583|0 |1 is obsolete|| --- Comment #7 from Marc Véron --- Created attachment 62586 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62586=edit Bug 18247 - Moved the SQL queries out of the branch_transfer_limits.pl script into the Koha::AuthorisedValue, Koha::Libraries, Koha::ItemType perl modules. Also added 3 successfully working unit tests for the new subroutines Followed test plan from comment #3, works as expected Signed-off-by: Marc Véron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Alex Buckleychanged: What|Removed |Added Status|Patch doesn't apply |Needs Signoff --- Comment #6 from Alex Buckley --- (In reply to Marc Veron in comment 4) Hi Marc I have updated my patches, rebased them into a single patch and have just attached that to the bug report. So should now apply successfully. Please continue to use the test plan in comment 3 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 --- Comment #5 from Alex Buckley--- Created attachment 62583 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62583=edit Bug 18247 - Moved the SQL queries out of the branch_transfer_limits.pl script into the Koha::AuthorisedValue, Koha::Libraries, Koha::ItemType perl modules. Also added 3 successfully working unit tests for the new subroutines -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Alex Buckleychanged: What|Removed |Added Attachment #61014|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 http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Alex Buckleychanged: What|Removed |Added Attachment #61015|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 http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Marc Véronchanged: What|Removed |Added Status|Needs Signoff |Patch doesn't apply CC||ve...@veron.ch --- Comment #4 from Marc Véron --- Sorry, I get a merge conflict in t/db_dependent/Koha/ItemTypes.t -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 Alex Buckleychanged: What|Removed |Added Status|ASSIGNED|Needs Signoff --- Comment #3 from Alex Buckley --- Test plan: 1. Restart memcached: sudo service memcached restart 2. Drop and recreate the Koha instance database 3. Go through the web installer, selecting all data to be installed in step 3 of the web installer 4. After the web installer is finished log in with the Koha database administrator credentials 5. Create yourself a patron account 6. Set the patron account to have superlibrarian privileges 7. Log out and back in as your newly created patron 8. Navigate to the Administrator interface and type in the url: cgi-bin/koha/admin/branch_transfer_limits.pl 9. Observe the list of default installed libraries displayed 10. Change the url to cgi-bin/koha/admin/branch_transfer_limits.pl?limitType=ccode 11. Observe the same list of libraries is displayed 12. Apply all the patches attached to this bug report 13. Repeat steps 1 through to 11 observing that all the libraries in the list are still visible with the patches applied when repeating steps 9 and 11 14. In your terminal navigate from your Koha instance root directory to t/db_dependent 15. Enter koha shell: sudo koha-shell 16. Run AuthorisedValues unit tests: prove -v AuthorisedValues.t 17. The tests should run successful 18. Exit the koha shell 19. cd into the 'Koha' directory from where you are 20. Enter the koha shell and run the ItemTypes.t and Libraries.t unit tests by writing in: prove -v filename.t (both of which should be successful) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 --- Comment #2 from Alex Buckley--- Created attachment 61015 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61015=edit Bug 18247 - Added 3 successfully working unit tests for each new subroutine -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18247] Remove SQL queries from branch_transfer_limit.pl administrative script
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18247 --- Comment #1 from Alex Buckley--- Created attachment 61014 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61014=edit Bug 18247 - Moved the SQL queries out of the branch_transfer_limits.pl script into the Koha::AuthorisedValue, Koha::Libraries, Koha::ItemType perl modules All the queries run successfully in the perl modules. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/