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.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

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 e80c904..30f606c 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
@@ -356,7 +356,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>
@@ -671,22 +671,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"/>
-                    [% 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" />
+            <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" />
+                        [% 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 %]