Attachment 168106 Details for Bug 37152 – Bug 37152: Deleting patron categories doesn't warn about categories in use

[patch] Bug 37152: Deleting patron categories doesn't warn about categories in use

Bug-37152-Deleting-patron-categories-doesnt-warn-a.patch (text/plain), 1.59 KB, created by Nick Clemens (kidclamp) on 2024-06-25 22:26:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2024-06-25 22:26:44 UTC

Size: 1.59 KB

patch

obsolete

>From 8a339a90db83f3fd338ae7863ee7add39780daca Mon Sep 17 00:00:00 2001
>From: Phil Ringnalda <phil@chetcolibrary.org>
>Date: Fri, 21 Jun 2024 12:05:11 -0700
>Subject: [PATCH] Bug 37152: Deleting patron categories doesn't warn about
> categories in use
>
>You can't delete a patron category when it is in use, but because the
>script which handles delete confirmation is expecting the wrong op,
>cud-delete_confirm rather than delete_confirm, you don't get any warning,
>just a failure to delete error message.
>
>Test plan:
> 1. Without the patch, Administration - Patron categories
> 2. On the row for Staff, click Delete, Delete this category
> 3. Note an error message telling you to check the logs for details
> 4. Apply patch, restart_all
> 5. With the patch, repeat steps 1-2, note that instead of a button for
>    Delete this category you get a warning that it is in use
> 6. In the list of categories, row for Board, click Delete, Delete this category
> 7. Note that the (unused) Board category was deleted
>
>Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> admin/categories.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/admin/categories.pl b/admin/categories.pl
>index 5495210b6db..c89fce0fbaf 100755
>--- a/admin/categories.pl
>+++ b/admin/categories.pl
>@@ -168,7 +168,7 @@ elsif ( $op eq 'cud-add_validate' ) {
>     $searchfield = q||;
>     $op = 'list';
> }
>-elsif ( $op eq 'cud-delete_confirm' ) {
>+elsif ( $op eq 'delete_confirm' ) {
> 
>     my $count = Koha::Patrons->search({
>         categorycode => $categorycode
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 37152: 167987 | 167988 | 167989 | 167990 | 167993 | 167994 | 167995 | 167996 | 168104 | 168105 | 168106 | 168107