From: Owen Leonard <[email protected]>

Correction in opac holds template. Corrections made are logically
consistent, but I couldn't reproduce the combination of factors
to trigger the "disabled" attribute on the branch selector. In
that respect this hasn't been tested 100%.

Signed-off-by: Katrin Fischer <[email protected]>
---
 .../opac-tmpl/prog/en/modules/opac-reserve.tt      |   33 +++++++++++++------
 1 files changed, 22 insertions(+), 11 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt 
b/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt
index 4d81a17..543b469 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt
@@ -423,17 +423,28 @@
                         [% IF ( bibitemloo.holdable ) %]
                            [% IF ( choose_branch ) %]
                                           <td>
-                                              <select name="branch" 
id="branch_[% bibitemloo.biblionumber %]"
-                                                [% UNLESS ( 
bibitemloo.holdable ) %]disabled="disabled"[% END %] >
-                                                [% FOREACH branchChoicesLoo IN 
bibitemloo.branchChoicesLoop %]
-                                                  [% IF ( 
branchChoicesLoo.selected ) %]
-                                                    <option value="[% 
branchChoicesLoo.value %]" selected="selected">[% branchChoicesLoo.branchname 
%]</option>
-                                                  [% ELSE %]
-                                                    <option value="[% 
branchChoicesLoo.value %]">[% branchChoicesLoo.branchname %]</option>
-                                                  [% END %]
-                                                [% END %]
-                                              </select>
-                                           </td>
+                         [% UNLESS ( bibitemloo.holdable ) %]
+                            <select name="branch" id="branch_[% 
bibitemloo.biblionumber %]" disabled="disabled">
+                              [% FOREACH branchChoicesLoo IN 
bibitemloo.branchChoicesLoop %]
+                                [% IF ( branchChoicesLoo.selected ) %]
+                                  <option value="[% branchChoicesLoo.value %]" 
selected="selected">[% branchChoicesLoo.branchname %]</option>
+                                [% ELSE %]
+                                  <option value="[% branchChoicesLoo.value 
%]">[% branchChoicesLoo.branchname %]</option>
+                                [% END %]
+                              [% END %]
+                          </select>
+                          [% ELSE %]
+                            <select name="branch" id="branch_[% 
bibitemloo.biblionumber %]">
+                              [% FOREACH branchChoicesLoo IN 
bibitemloo.branchChoicesLoop %]
+                                [% IF ( branchChoicesLoo.selected ) %]
+                                  <option value="[% branchChoicesLoo.value %]" 
selected="selected">[% branchChoicesLoo.branchname %]</option>
+                                [% ELSE %]
+                                  <option value="[% branchChoicesLoo.value 
%]">[% branchChoicesLoo.branchname %]</option>
+                                [% END %]
+                              [% END %]
+                            </select>
+                          [% END %]
+                       </td>
                            [% END %]
                        [% END %]
                     [% END %]
-- 
1.6.3.3


_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-patches
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to