Attachment 29481 Details for Bug 12517 – [PASSED QA] Bug 12517 - Download and send list popups broken in bootstrap OPAC

[patch] [PASSED QA] Bug 12517 - Download and send list popups broken in bootstrap OPAC

PASSED-QA-Bug-12517---Download-and-send-list-popup.patch (text/plain), 3.04 KB, created by Kyle M Hall (khall) on 2014-07-03 14:21:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2014-07-03 14:21:16 UTC

Size: 3.04 KB

patch

obsolete

>From 790e4101c6faf6dc18cf1f6127e854e9a0e28667 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 2 Jul 2014 09:10:33 -0400
>Subject: [PATCH] [PASSED QA] Bug 12517 - Download and send list popups broken in bootstrap OPAC
>
>A JavaScript error prevents the download list and send list links in the
>bootstrap OPAC from functioning correctly. A variable declaration was
>removed from basket.js during the creation of the theme but
>opac-shelves.tt was not changed accordingly. This patch corrects it.
>
>To test, view an existing shelf in the bootstrap OPAC and test the
>"download list" and "send list" links. They should each trigger popups
>without any errors.
>
>Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>Popups pops again, no koha-qa errors
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> .../opac-tmpl/bootstrap/en/modules/opac-shelves.tt |    4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt
>index 87223bb..a6d9040 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt
>@@ -156,10 +156,10 @@
>                             <div id="toolbar" class="toolbar clearfix">
>                                 <div class="list-actions">
>                                     <a class="newshelf" href="/cgi-bin/koha/opac-shelves.pl?shelves=1">New list</a> <span class="sep">|</span>
>-                                    <a href="/cgi-bin/koha/opac-downloadshelf.pl?shelfid=[% shelfnumber %]" class="download" onclick="open(CGIBIN+'opac-downloadshelf.pl?shelfid=[% shelfnumber %]','win_form','scrollbars=no,resizable=no,height=300,width=450,top=50,left=100'); return false;">Download list</a>
>+                                    <a href="/cgi-bin/koha/opac-downloadshelf.pl?shelfid=[% shelfnumber %]" class="download" onclick="open('/cgi-bin/koha/opac-downloadshelf.pl?shelfid=[% shelfnumber %]','win_form','scrollbars=no,resizable=no,height=300,width=450,top=50,left=100'); return false;">Download list</a>
> 
>                                     [% IF Koha.Preference( 'opacuserlogin' ) == 1 %]
>-                                        <span class="sendlist"><a href="/cgi-bin/koha/opac-sendshelf.pl?shelfid=[% shelfnumber %]" class="send" onclick="open(CGIBIN+'opac-sendshelf.pl?shelfid=[% shelfnumber %]','win_form','scrollbars=no,resizable=no,height=300,width=450,top=50,left=100'); return false; ">Send list</a></span>
>+                                        <span class="sendlist"><a href="/cgi-bin/koha/opac-sendshelf.pl?shelfid=[% shelfnumber %]" class="send" onclick="open('/cgi-bin/koha/opac-sendshelf.pl?shelfid=[% shelfnumber %]','win_form','scrollbars=no,resizable=no,height=300,width=450,top=50,left=100'); return false; ">Send list</a></span>
>                                     [% END %]
> 
>                                     <a class="print-small" href="/cgi-bin/koha/opac-shelves.pl" onclick="print(); return false;">Print list</a>
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 12517: 29437 | 29443 | 29481