From ca6bd89b852f4561fdc4ff328d19d4a71d759b7f Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Wed, 13 Jan 2016 16:36:47 +0000
Subject: [PATCH] Bug 15569: Do not displayed "automatic renewal" if the patron
 cannot checkout
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

If the patron cannot checkout (debarred) and on-site checkout is
enabled, the "Automatic renewal" checkbox should not be displayed on the
circulation screen.

Test plan:
Enable OnSiteCheckouts and OnSiteCheckoutsForce
Debar a patron and go on the circulation page.
Without this patch, the automatic renewal checkbox is displayed.
With this patch applied, it's not.

Patch works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com
---
 .../prog/en/modules/circ/circulation.tt       | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
index 2ffbb540b9..cf336a2687 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
@@ -633,14 +633,17 @@ No patron matched <span class="ex">[% message %]</span>
     [% END %]
     <button type="submit" class="btn">Check out</button>
 
-    <div class="date-select">
-        [% IF NEEDSCONFIRMATION %]
-            <input type="checkbox" name="auto_renew" id="auto_renew" value="auto_renew" disabled="disabled" />
-        [% ELSE %]
-            <input type="checkbox" name="auto_renew" id="auto_renew" value="auto_renew" />
-        [% END %]
-        <label for="auto_renew">Automatic renewal</label>
-    </div>
+    [% UNLESS flagged and noissues %]
+        <div class="date-select">
+            [% IF NEEDSCONFIRMATION %]
+                <input type="checkbox" name="auto_renew" id="auto_renew" value="auto_renew" disabled="disabled" />
+            [% ELSE %]
+                <input type="checkbox" name="auto_renew" id="auto_renew" value="auto_renew" />
+            [% END %]
+
+            <label for="auto_renew">Automatic renewal</label>
+        </div>
+    [% END %]
 
     [% UNLESS ( noissues && Koha.Preference('OnSiteCheckoutsForce') ) %]
         [% IF ( SpecifyDueDate ) %]
-- 
2.39.5