From 2f876afdd368bf6a261ebe3627ff8ae1c0ac40a0 Mon Sep 17 00:00:00 2001
From: Jesse Weaver <jweaver@bywatersolutions.com>
Date: Wed, 30 Sep 2015 11:48:14 -0400
Subject: [PATCH] Bug 11565: (QA followup) add class to allow feature to be
 easily hidden

This wraps both checkboxes (in the confirmation dialog and checkout
interface) in a class of "circ-override-high-holds" so they can be
easily hidden.
---
 .../prog/en/modules/circ/circulation.tt            | 30 ++++++++++++----------
 1 file changed, 16 insertions(+), 14 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 3935829..c2d1008 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
@@ -349,7 +349,7 @@ $(document).ready(function() {
 [% IF (forceallow) %]<input type="hidden" name="forceallow" value="1">[% END %]
 
 [% IF HIGHHOLDS %]
-    <p>
+    <p class="circ-override-high-holds">
     <input type="checkbox" name="override_high_holds_tmp" id="override_high_holds_tmp" value="1" />
     <label for="override_high_holds_tmp">Don't decrease loan length based on holds</label>
     </p>
@@ -664,22 +664,24 @@ No patron matched <span class="ex">[% message %]</span>
 
             <label for="auto_renew">Automatic renewal</label>
 
-            [% IF Koha.Preference('decreaseLoanHighHolds') %]
-                [% IF NEEDSCONFIRMATION %]
-                    [% IF override_high_holds %]
-                        <input type="checkbox" name="override_high_holds" id="override_high_holds" value="1" disabled="disabled" checked="checked"/>
+            <span class="circ-override-high-holds">
+                [% IF Koha.Preference('decreaseLoanHighHolds') %]
+                    [% IF NEEDSCONFIRMATION %]
+                        [% IF override_high_holds %]
+                            <input type="checkbox" name="override_high_holds" id="override_high_holds" value="1" disabled="disabled" checked="checked"/>
+                        [% ELSE %]
+                            <input type="checkbox" name="override_high_holds" id="override_high_holds" value="1" disabled="disabled"/>
+                        [% END %]
                     [% ELSE %]
-                        <input type="checkbox" name="override_high_holds" id="override_high_holds" value="1" disabled="disabled"/>
-                    [% END %]
-                [% ELSE %]
-                    [% IF override_high_holds %]
-                        <input type="checkbox" name="override_high_holds" id="override_high_holds" value="1" checked="checked" />
-                    [% ELSE %]
-                        <input type="checkbox" name="override_high_holds" id="override_high_holds" value="1" />
+                        [% IF override_high_holds %]
+                            <input type="checkbox" name="override_high_holds" id="override_high_holds" value="1" checked="checked" />
+                        [% ELSE %]
+                            <input type="checkbox" name="override_high_holds" id="override_high_holds" value="1" />
+                        [% END %]
                     [% END %]
+                    <label for="override_high_holds">Don't decrease loan length based on holds</label>
                 [% END %]
-                <label for="override_high_holds">Don't decrease loan length based on holds</label>
-            [% END %]
+            </span>
         </div>
     [% END %]
 
-- 
2.8.0.rc3