Bug 22083: Typo in circulation_batch_checkouts.tt
authorCaroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Tue, 8 Jan 2019 14:28:35 +0000 (09:28 -0500)
committerFridolin Somers <fridolin.somers@biblibre.com>
Tue, 12 Feb 2019 15:24:16 +0000 (16:24 +0100)
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>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 3ee95bd0213943b625ea5a35eb04e45835cf6f65)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 4c928041f1ee5ec4c21aab24dfd7f69399ae330f)

Signed-off-by: Jesse Maseto <jesse@bywatersolutions.com>
(cherry picked from commit c641fb1b0840b30f189a16693ffe4306e009aaff)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation_batch_checkouts.tt

index 2416f36..da34d13 100644 (file)
@@ -163,7 +163,7 @@ $(document).ready(function() {
                   [% IF checkout_info.BORRNOTSAMEBRANCH %]
                       <li><i class="fa fa-li fa-warning"></i>This patron is from a different library ([% Branches.GetName( checkout_info.BORRNOTSAMEBRANCH ) %]).</li>
                   [% END %]
-                  [% IF checkout_ino.PATRON_CANT %]
+                  [% IF checkout_info.PATRON_CANT %]
                       <li><i class="fa fa-li fa-warning"></i>This patron can't check out this item per library circulation policy.</li>
                   [% END %]
                   [% IF checkout_info.NOT_FOR_LOAN_FORCING %]