Bug 22951: Markup error in OPAC holds template
authorOwen Leonard <oleonard@myacpl.org>
Wed, 26 Jun 2019 12:43:36 +0000 (12:43 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 15 Jul 2019 10:27:57 +0000 (11:27 +0100)
This patch makes minor markup changes to the OPAC holds template in
order to fix validation errors.

To test, apply the patch and go through the process of placing a
hold in the OPAC. Test various aspects of the process, including cases
where the hold is blocked and an error message is displayed (for
instance, when there are no items to be placed on hold).

The page should work correctly and validating the source should
return no relevant errors.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Frederik Chenier <frederik.chenier@inlibro.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt

index b26fbaf..30bbd9b 100644 (file)
                                 None of the libraries are available for pickup location.
                             </li>
                         [% END %]
-                    [% ELSE %]
+                    </ul>
+                </div> <!-- /.message -->
+            [% ELSE # /IF message %]
 
                         [% IF ( none_available && multi_hold ) %]
                             <div id="none_available" class="alert">
                                 Sorry, none of these titles can be placed on hold.
                             </div>
                         [% END %]
-                </div>
+
             [% END # / IF message %]
 
                     [% UNLESS ( message ) %]