Summary: | Typo in circulation_batch_checkouts.tt | ||
---|---|---|---|
Product: | Koha | Reporter: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Component: | Circulation | Assignee: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | fridolin.somers, gmcharlt, jesse, kyle.m.hall, martin.renvoize, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 22083: Typo in circulation_batch_checkouts.tt
Bug 22083: Typo in circulation_batch_checkouts.tt Bug 22083: Typo in circulation_batch_checkouts.tt |
Description
Caroline Cyr La Rose
2019-01-08 14:24:08 UTC
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 |