Attachment 50586 Details for Bug 16281 – Bug 16281 [Follow-up] Remove the use of "onclick" from Reports module

[patch] Bug 16281 [Follow-up] Remove the use of "onclick" from Reports module

Bug-16281-Follow-up-Remove-the-use-of-onclick-from.patch (text/plain), 1.61 KB, created by Owen Leonard on 2016-04-22 13:34:11 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2016-04-22 13:34:11 UTC

Size: 1.61 KB

patch

obsolete

>From c5a354ebb6a0d2081b01d2e92c00dd715a9191e8 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 22 Apr 2016 09:31:10 -0400
>Subject: [PATCH] Bug 16281 [Follow-up] Remove the use of "onclick" from
> Reports module
>Content-Type: text/plain; charset="utf-8"
>
>This minor follow-up adds a dummy href attribute to the "Return to the
>previous page" link so that the link behaves like a link. A
>"preventDefault()" is added to the goback() function to keep the "#"
>link from triggering a page jump.
>
>To test, apply the patch, create a report with an SQL error and run
>it. Confirm that the 'Return to previous page' link works as expected.
>---
> .../intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt    | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
>index 0ff2e8b..37f05ab 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
>@@ -68,7 +68,8 @@ function load_group_subgroups () {
> 
> $(document).ready(function(){
> 
>-$(".goback").on("click",function(){
>+$(".goback").on("click",function(e){
>+    e.preventDefault();
>     window.history.back();
> });
> 
>@@ -1017,7 +1018,7 @@ Sub report:<select name="subreport">
>     [% ELSE %]
>     [% END %]
>     <div id="onerror_actions">
>-        <a class="button goback">Return to previous page</a>
>+        <a href="#" class="button goback">Return to previous page</a>
>     </div>
> [% END %]
> </div>
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 16281: 50303 | 50548 | 50585 | 50586 | 50589 | 50603 | 50604