Attachment 22418 Details for Bug 11133 – Bug 11133 - Markup errors in OPAC place hold form HTML

[patch] Bug 11133 - Markup errors in OPAC place hold form HTML

Bug-11133---Markup-errors-in-OPAC-place-hold-form-.patch (text/plain), 1.84 KB, created by Kyle M Hall (khall) on 2013-10-25 16:09:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2013-10-25 16:09:44 UTC

Size: 1.84 KB

patch

obsolete

>From 310370d1f4ebb888bcc0c1c56cc5db4bd6ca8600 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 25 Oct 2013 10:39:03 -0400
>Subject: [PATCH] Bug 11133 - Markup errors in OPAC place hold form HTML
>
>There are markup errors in the OPAC place hold form which prevent it
>from validating successfully. These errors don't seem to cause any
>visual problem but can potentially cause problems when debugging
>template logic.
>
>To test, apply the patch and validate the page using your preferred
>validator (since it's a page which requires login you'll have to upload
>or paste the source). The page should validate successfully. Test when
>placing holds on:
>
>- a single item
>- multiple items
>- items which you cannot place on hold
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> .../opac-tmpl/prog/en/modules/opac-reserve.tt      |    7 ++++---
> 1 files changed, 4 insertions(+), 3 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 bf334fa..671e1f9 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt
>+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt
>@@ -460,8 +460,8 @@
>                                 </li>
> 
>                             [% END %]
>-                        [% END %]
>                 </ul>
>+                        [% END %]
>                   [% IF ( OPACItemHolds ) %]
>                   [% IF ( bibitemloo.holdable ) %]
> 
>@@ -540,11 +540,12 @@
>                         </table>
>                     </div>
> 
>+                </fieldset>
>+                  [% END %]
>+
>                   [% END %]
> 
>-                  [% END %]
>             </div>
>-                </fieldset>
>                 [% END %]
> 
>               [% END %] 
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 11133: 22411 | 22418 | 22454