Attachment 33568 Details for Bug 13254 – Bug 13254 - Delete record don't wait for confirmation

[patch] Bug 13254 - Delete record don't wait for confirmation

Bug-13254---Delete-record-dont-wait-for-confirmati.patch (text/plain), 2.03 KB, created by Rafal Kopaczka on 2014-11-14 13:17:30 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Rafal Kopaczka

Created: 2014-11-14 13:17:30 UTC

Size: 2.03 KB

patch

obsolete

>From 7fdc75246cbbfa5caa15f321cfad3375901a4010 Mon Sep 17 00:00:00 2001
>From: Rafal Kopaczka <rkk0@poczta.onet.pl>
>Date: Fri, 14 Nov 2014 14:06:22 +0100
>Subject: [PATCH] Bug 13254 - Delete record don't wait for confirmation
>
>In some cases (eg. when Staf Client Search is active), when user choose
>Edit->Delete record on record tool bar, browser don't wait for
>confirmation and goes immediately to delete record.
>
>To reproduce:
>1. Search for some biblio records and choose one without items attached.
>2. Note that there, must be "Return to search results" box on left side,
>bug works in that case, when in normal view everything work fine.
>3. Click Edit->Delete record, watch that confirmation box shows, but
>don't wait for OK and runs immediately. If you are fast enough to
>click OK, then you get error as below, because record was deleted
>earlier.
>
>To test:
>1. Apply patch.
>2. Follow reproduce steps.
>3. Check if waits for confirmation in all cases.
>4. Check if deletes record after confirm.
>---
> koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc
>index 3044f16..cf72265 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc
>@@ -199,7 +199,7 @@ CAN_user_serials_create_subscription ) %]
>                 [% IF ( count ) %]
>                     <li class="disabled"><a id="deletebiblio" data-toggle="tooltip" data-placement="left" title="[% count %] item(s) are attached to this record. You must delete all items before deleting this record." href="#">Delete record</a></li>
>                 [% ELSE %]
>-                    <li><a id="deletebiblio" href="/cgi-bin/koha/cataloguing/addbiblio.pl?op=delete&amp;biblionumber=[% biblionumber %]">Delete record</a></li>
>+                    <li><a id="deletebiblio" href="#">Delete record</a></li>
>                 [% END %]
>             [% END %]
> 
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 13254: 33568 | 33581 | 33586