Attachment 62811 Details for Bug 18450 – Bug 18450 - Implemented check if either AllowRenewLimitOverride or AllowRenewalIfOtherItemsAvaliable were set to allow, so the user is only blocked if both of these sysprefs are set to 'Dont allow'

[patch] Bug 18450 - Implemented check if either AllowRenewLimitOverride or AllowRenewalIfOtherItemsAvaliable were set to allow, so the user is only blocked if both of these sysprefs are set to 'Dont allow'

Bug-18450---Implemented-check-if-either-AllowRenew.patch (text/plain), 3.99 KB, created by Alex Buckley on 2017-04-28 08:56:49 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Buckley

Created: 2017-04-28 08:56:49 UTC

Size: 3.99 KB

patch

obsolete

>From a1172c68ad35247f3b4a0c1393c68c1b0c3a032c Mon Sep 17 00:00:00 2001
>From: Alex Buckley <alexbuckley@catalyst.net.nz>
>Date: Thu, 27 Apr 2017 16:04:54 +0000
>Subject: [PATCH] Bug 18450 - Implemented check if either
> AllowRenewLimitOverride or AllowRenewalIfOtherItemsAvaliable were set to
> allow, so the user is only blocked if both of these sysprefs are set to 'Dont
> allow'
>
>Test plan:
>1. In More->Administration->Global system preferences
>ensure that both the AllowRenewalLimitOverride and
>AllowRenewalIfOtherItemsAvaliable preferences are both set to 'Dont
>allow'.
>2. Place a hold on an item from a patron
>3. Checkout the item out to a different patron
>4. Renew the item and notice the 'Cannot renew: The item is on hold for
>another patron' warning
>5. Click the 'Override and renew' button and notice that you
>can bypass the hold
>6. Apply patch
>7. Renew the item again and notice that the message does not have a
>'Override and renew' button it just has the 'Ignore and continue' button
>8. Go back to More->Administration->Global system preferences and set
>AllowRenewalLimitOverride to 'Allow'
>9. Try renewing the same item again and notice that the 'Override and
>renew' button is displayed
>10. In systems preferences now set the
>AllowRenewalIfOtherItemsAvailable syspref to 'Allow' and
>AllowRenewalLimitOverride to 'Dont allow' and notice the 'Override and
>renew' button is displayed
>11. Now set both these sysprefs to 'Allow' and notice the 'Override and renew' button is displayed
>because you will not be violating these preferences.
>
>Note: Checking if the renewal exceeds the renewal level, if auto renew
>is set up, or if renewal is to soon is checked earlier in this
>conditional
>if statement earlier and so the check in this patch does not need to
>check the renewal level, auto renewal, if renewal is too soon.
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt
>index 73d7d8c..5db95a6 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt
>@@ -113,6 +113,20 @@
>                                         <input type="hidden" name="override_holds" value="1" />
>                                         <button type="submit" class="approve"><i class="fa fa-check"></i> Override and renew</button>
>                                     </form>
>+                                [% ELSIF (!(Koha.Preference('AllowRenewalLimitOverride')) && (Koha.Preference('AllowRenewalIfOtherItemsAvailable'))) %]
>+                                    <form method="post" action="/cgi-bin/koha/circ/renew.pl">
>+                                       <input type="hidden" name="barcode" value="[% item.barcode %]"/>
>+                                       <input type="hidden" name="override_limit" value="1" />
>+                                       <input type="hidden" name="override_holds" value="1" />
>+                                       <button type="submit" class="approve"><i class="fa fa-check"></i> Override and renew</button>
>+                                     </form>
>+                                [% ELSIF ((Koha.Preference('AllowRenewalLimitOverride')) && !(Koha.Preference('AllowRenewalIfOtherItemsAvai    lable'))) %]
>+                                    <form method="post" action="/cgi-bin/koha/circ/renew.pl">
>+                                       <input type="hidden" name="barcode" value="[% item.barcode %]"/>
>+                                       <input type="hidden" name="override_limit" value="1" />
>+                                       <input type="hidden" name="override_holds" value="1" />
>+                                       <button type="submit" class="approve"><i class="fa fa-check"></i> Override and renew</button>
>+                                    </form>
>                                 [% END %]
>                             [% ELSIF error == "patron_restricted" %]
> 
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 18450: 62339 | 62447 | 62811 | 63226 | 63292 | 63293 | 63377 | 63764 | 64402 | 69475 | 69476 | 69489 | 72835 | 72836 | 86587 | 86588 | 86589 | 86590 | 86591 | 87337 | 87338 | 87528 | 87529