on branch_transfer_limits.pl, we see the wording "Check the boxes for the libraries you accept to checkin items from." However, what we're actually doing here is checking boxes for the libraries to which you are allowed to transfer items. To verify that this is the case: 1- Set UseBranchTransferLimits set to Enforce using itemtype 2- Confirm circ rules allow holds 3- Confirm hold policy by item type allows holds 4- Confirm all branches are valid pickup locations in branch setup 5- In Branch Transfer Limits, set dropdown to Branch A, select itemtype Book, disable all transfers so Branch A does not accept books from any other branch 6- Find a patron from Branch A 7- Find a book from Branch B 8- On the intranet, successfully place hold on item for patron for pickup at branch A 9- Cancel hold 10- Log into opac as patron 11- Successfully place hold on item for pickup at Branch A 12- Cancel hold 13- In Branch Transfer Limits, set dropdown to Branch A, select itemtype Book, ENABLE all transfers so Branch A DOES accept books from any other branch 14- In Branch Transfer Limits, set dropdown to Branch B, select itemtype Book, disable all transfers so Branch B does not accept books from any other branch 15- On the intranet, try to place a hold on item for patron, confirm Branch B is your only pickup option 16- On the OPAC, log in as patron, try to place a hold on item, confirm Branch B is your only pickup option
Created attachment 99715 [details] [review] BUG 24747: Corrects on-screen instructions in branch transfer limits To test: 1- go to cgi-bin/koha/admin/branch_transfer_limits.pl 2- see wording "Check the boxes for the libraries you accept to checkin items from" 3- apply patch 4- see wording "Check the boxes for the libraries you allow your items to be transferred to" To verify new wording accuractely describes behavior: 1- Set UseBranchTransferLimits set to Enforce using itemtype 2- Confirm circ rules allow holds 3- Confirm hold policy by item type allows holds 4- Confirm all branches are valid pickup locations in branch setup 5- In Branch Transfer Limits, set dropdown to Branch A, select itemtype Book, disable all transfers so Branch A does not accept books from any other branch 6- Find a patron from Branch A 7- Find a book from Branch B 8- On the intranet, successfully place hold on item for patron for pickup at branch A 9- Cancel hold 10- Log into opac as patron 11- Successfully place hold on item for pickup at Branch A 12- Cancel hold 13- In Branch Transfer Limits, set dropdown to Branch A, select itemtype Book, ENABLE all transfers so Branch A DOES accept books from any other branch 14- In Branch Transfer Limits, set dropdown to Branch B, select itemtype Book, disable all transfers so Branch B does not accept books from any other branch 15- On the intranet, try to place a hold on item for patron, confirm Branch B is your only pickup option 16- On the OPAC, log in as patron, try to place a hold on item, confirm Branch B is your only pickup option
Created attachment 99716 [details] [review] BUG 24747: Corrects on-screen instructions in branch transfer limits To test: 1- go to cgi-bin/koha/admin/branch_transfer_limits.pl 2- see wording "Check the boxes for the libraries you accept to checkin items from" 3- apply patch 4- see wording "Check the boxes for the libraries you allow your items to be transferred to" To verify new wording accuractely describes behavior: 1- Set UseBranchTransferLimits set to Enforce using itemtype 2- Confirm circ rules allow holds 3- Confirm hold policy by item type allows holds 4- Confirm all branches are valid pickup locations in branch setup 5- In Branch Transfer Limits, set dropdown to Branch A, select itemtype Book, disable all transfers so Branch A does not accept books from any other branch 6- Find a patron from Branch A 7- Find a book from Branch B 8- On the intranet, successfully place hold on item for patron for pickup at branch A 9- Cancel hold 10- Log into opac as patron 11- Successfully place hold on item for pickup at Branch A 12- Cancel hold 13- In Branch Transfer Limits, set dropdown to Branch A, select itemtype Book, ENABLE all transfers so Branch A DOES accept books from any other branch 14- In Branch Transfer Limits, set dropdown to Branch B, select itemtype Book, disable all transfers so Branch B does not accept books from any other branch 15- On the intranet, try to place a hold on item for patron, confirm Branch B is your only pickup option 16- On the OPAC, log in as patron, try to place a hold on item, confirm Branch B is your only pickup option Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com>
Clarifying and simplifying the part that confirms new wording is accurate: 1- Set UseBranchTransferLimits set to Enforce using itemtype 2- Confirm all branches are valid pickup locations in branch setup 3- In Branch Transfer Limits, set dropdown to Branch A, select itemtype Book, disable all transfers so Branch A does send books to any other branch 4- Find a patron from Branch B 5- Find a book from Branch A 6- On the intranet, confirm that Branch A is not available as a pickup location for your patron and item 7- Log into opac as patron 8- Confirm Branch A is not available as a pickup location for your item
Created attachment 99808 [details] [review] BUG 24747: Corrects on-screen instructions in branch transfer limits To test: 1- go to cgi-bin/koha/admin/branch_transfer_limits.pl 2- see wording "Check the boxes for the libraries you accept to checkin items from" 3- apply patch 4- see wording "Check the boxes for the libraries you allow your items to be transferred to" To verify new wording accuractely describes behavior: 1- Set UseBranchTransferLimits set to Enforce using itemtype 2- Confirm circ rules allow holds 3- Confirm hold policy by item type allows holds 4- Confirm all branches are valid pickup locations in branch setup 5- In Branch Transfer Limits, set dropdown to Branch A, select itemtype Book, disable all transfers so Branch A does not accept books from any other branch 6- Find a patron from Branch A 7- Find a book from Branch B 8- On the intranet, successfully place hold on item for patron for pickup at branch A 9- Cancel hold 10- Log into opac as patron 11- Successfully place hold on item for pickup at Branch A 12- Cancel hold 13- In Branch Transfer Limits, set dropdown to Branch A, select itemtype Book, ENABLE all transfers so Branch A DOES accept books from any other branch 14- In Branch Transfer Limits, set dropdown to Branch B, select itemtype Book, disable all transfers so Branch B does not accept books from any other branch 15- On the intranet, try to place a hold on item for patron, confirm Branch B is your only pickup option 16- On the OPAC, log in as patron, try to place a hold on item, confirm Branch B is your only pickup option Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
This looks to be the first bug you've authored on Andrew, great to see, welcome aboard! :)
Nice work everyone! Pushed to master for 20.05
Thanks, Martin!
backported to 19.11.x branch for 19.11.05
backported to 19.05.x for 19.05.10
Backported to 18.11.x for 18.11.16