There is a typo in circulation_batch_checkouts.tt that makes Koha skip a warning if the user cannot borrow due to circulation rules. To reproduce: 1. In Administration > Global system preferences, 'Allow' BatchCheckouts and enter patron categories in BatchCheckoutsValidCategories 2. Go to Administration > Circulation and fines rules 3. Enter a rule where a certain category of patrons (that is in BatchCheckoutsValidCategories) cannot take out a particular item type 4. Search the catalog for an item of that item type 5. Copy the barcode 6. Go into patrons and find a patron of that category 7. Click on the Batch checkout tab 8. Paste the barcode in the box and click Check out 9. It should say 'Too many checked out' It should also say 'This patron can't check out this item per library circulation policy'.
Created attachment 83715 [details] [review] Bug 22083: Typo in circulation_batch_checkouts.tt There is a typo in circulation_batch_checkouts.tt that makes Koha skip a warning if the user cannot borrow due to circulation rules. This patch corrects this typo. To test: 1. In Administration > Global system preferences, 'Allow' BatchCheckouts and enter patron categories in BatchCheckoutsValidCategories 2. Go to Administration > Circulation and fines rules 3. Enter a rule where a certain category of patrons (that is in BatchCheckoutsValidCategories) cannot take out a particular item type 4. Search the catalog for an item of that item type 5. Copy the barcode 6. Go into patrons and find a patron of that category 7. Click on the Batch checkout tab 8. Paste the barcode in the box and click Check out 9. It should say 'Too many checked out' 10. Apply the patch 11. Try the check out again, it should show both warnings ('Too many checked out' and 'This patron can't check out this item per library circulation policy')
I also think that the 'too many checked out' warning should only appear if the circulation rules allow 1 or more. If the circulation rules allow 0, only 'This patron can't check out this item per library circulation rules' should show... It is, however, outside of the scope of this bug and outside my capabilities... Just throwing it out there if someone wants to take it over :)
Created attachment 83763 [details] [review] Bug 22083: Typo in circulation_batch_checkouts.tt There is a typo in circulation_batch_checkouts.tt that makes Koha skip a warning if the user cannot borrow due to circulation rules. This patch corrects this typo. To test: 1. In Administration > Global system preferences, 'Allow' BatchCheckouts and enter patron categories in BatchCheckoutsValidCategories 2. Go to Administration > Circulation and fines rules 3. Enter a rule where a certain category of patrons (that is in BatchCheckoutsValidCategories) cannot take out a particular item type 4. Search the catalog for an item of that item type 5. Copy the barcode 6. Go into patrons and find a patron of that category 7. Click on the Batch checkout tab 8. Paste the barcode in the box and click Check out 9. It should say 'Too many checked out' 10. Apply the patch 11. Try the check out again, it should show both warnings ('Too many checked out' and 'This patron can't check out this item per library circulation policy') Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 83842 [details] [review] Bug 22083: Typo in circulation_batch_checkouts.tt There is a typo in circulation_batch_checkouts.tt that makes Koha skip a warning if the user cannot borrow due to circulation rules. This patch corrects this typo. To test: 1. In Administration > Global system preferences, 'Allow' BatchCheckouts and enter patron categories in BatchCheckoutsValidCategories 2. Go to Administration > Circulation and fines rules 3. Enter a rule where a certain category of patrons (that is in BatchCheckoutsValidCategories) cannot take out a particular item type 4. Search the catalog for an item of that item type 5. Copy the barcode 6. Go into patrons and find a patron of that category 7. Click on the Batch checkout tab 8. Paste the barcode in the box and click Check out 9. It should say 'Too many checked out' 10. Apply the patch 11. Try the check out again, it should show both warnings ('Too many checked out' and 'This patron can't check out this item per library circulation policy') Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Thx, Caroline!
Awesome work all! Pushed to master for 19.05
Pushed to 18.11.x for 18.11.03
Pushed to 18.05.x for 18.05.09
Pushed to 17.11.x for 17.11.15