Attachment 128278 Details for Bug 9565 – Bug 9565: Deleting a biblio should alert/fail if there are existent subscriptions

[patch] Bug 9565: Deleting a biblio should alert/fail if there are existent subscriptions

Bug-9565-Deleting-a-biblio-should-alertfail-if-the.patch (text/plain), 2.34 KB, created by Jonathan Druart on 2021-12-06 13:45:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2021-12-06 13:45:34 UTC

Size: 2.34 KB

patch

obsolete

>From 713eba48044f2b745b06c5a18ddfd968d59adcf4 Mon Sep 17 00:00:00 2001
>From: Timothy Alexis Vass <timothy_alexis.vass@ub.lu.se>
>Date: Fri, 13 Nov 2020 09:47:32 +0000
>Subject: [PATCH] Bug 9565: Deleting a biblio should alert/fail if there are
> existent subscriptions
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Currently Koha will delete a biblio record if there are existent serial subscriptions.
>This patch will prevent deleting the biblio until attached subscriptions are removed.
>
>To test:
>1) Add a subscription to a biblio, or check a biblio which has one attached already.
>2) Remove all items if there are any.
>3) Apply patch.
>4) Confirm that the 'Delete record' item is disabled in the list.
>5) Sign off.
>
>Sponsored by: Lunds Universitetsbibliotek
>
>Signed-off-by: SÃ©verine QUEUNE <severine.queune@bulac.fr>
>
>Signed-off-by: Frank Hansen <frank.hansen@ub.lu.se>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc | 2 ++
> 1 file changed, 2 insertions(+)
>
>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 e194411af5d..1f352ab3ae2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc
>@@ -72,6 +72,8 @@
>             [% IF CAN_user_editcatalogue_edit_catalogue or ( frameworkcode == 'FA' and CAN_user_editcatalogue_fast_cataloging ) %]
>                 [% IF ( count ) %]
>                     <li class="disabled"><a id="deletebiblio" data-toggle="tooltip" data-placement="left" title="[% count | html %] item(s) are attached to this record. You must delete all items before deleting this record." href="#">Delete record</a></li>
>+                [% ELSIF ( biblio.subscriptions.count ) %]
>+                    <li class="disabled"><a id="deletebiblio" data-toggle="tooltip" data-placement="left" title="[% biblio.subscriptions.count | html %] subscription(s) are attached to this record. You must delete all subscriptions before deleting this record." href="#">Delete record</a></li>
>                 [% ELSE %]
>                     <li><a id="deletebiblio" data-order-manage="[% CAN_user_acquisition_order_manage | html %]" href="#">Delete record</a></li>
>                 [% END %]
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 9565: 113589 | 113591 | 126821 | 126905 | 128051 | 128160 | 128163 | 128164 | 128165 | 128278 | 128279 | 128280 | 128281 | 128764 | 128832