Attachment 167711 Details for Bug 37080 – Bug 37080: Cannot delete MARC bibliographic frameworks or authority types

[patch] Bug 37080: Cannot delete MARC bibliographic frameworks or authority types

Bug-37080-Cannot-delete-MARC-bibliographic-framewo.patch (text/plain), 2.58 KB, created by Martin Renvoize (ashimema) on 2024-06-14 10:53:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2024-06-14 10:53:43 UTC

Size: 2.58 KB

patch

obsolete

>From 03d2b5e3a1b42d1d08a0834879c391c404f43d2c Mon Sep 17 00:00:00 2001
>From: Phil Ringnalda <phil@chetcolibrary.org>
>Date: Wed, 12 Jun 2024 19:25:14 -0700
>Subject: [PATCH] Bug 37080: Cannot delete MARC bibliographic frameworks or
> authority types
>
>The scripts for managing both bib and authority frameworks are now expecting
>an op 'cud-delete_confirm' but delete_confirm isn't cud, it's just a GET of a
>page asking if you are sure. The scripts just need to expect the op they are
>actually getting, delete_confirm.
>
>Test plan:
> 1. Administration - MARC bibliographic framework - New framework
> 2. Give the framework any code and description and Submit
> 3. Actions menu for your new framework - Delete
> 4. Note the message says 'Confirm deletion of framework '' ()?" rather than
>    having the code and description included.
> 5. Click Yes, note that deleting failed
> 6. Administration - Authority types - New authority type
> 7. Give the authority type any type and description and Submit
> 8. Actions menu for your authority type - Delete
> 9. Note the message again says '' () rather than 'desc' (FOO)
>10. Click yes, note that deleting failed
>11. Finally! Apply patch, reset_all
>12. Repeat steps 1-10, this time noting that you get 'desc' (FOO) rather than
>    '' (), and that deleting works.
>
>Signed-off-by: David Nind <david@davidnind.com>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> admin/authtypes.pl        | 2 +-
> admin/biblio_framework.pl | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/admin/authtypes.pl b/admin/authtypes.pl
>index 5445c5b7f0e..13f573b300a 100755
>--- a/admin/authtypes.pl
>+++ b/admin/authtypes.pl
>@@ -82,7 +82,7 @@ if ( $op eq 'add_form' ) {
>     }
>     $op = 'list';
> 
>-} elsif ( $op eq 'cud-delete_confirm' ) {
>+} elsif ( $op eq 'delete_confirm' ) {
>     my $authority_type = Koha::Authority::Types->find($authtypecode);
>     my $authorities_using_it = Koha::Authorities->search( { authtypecode => $authtypecode } )->count;
>     $template->param(
>diff --git a/admin/biblio_framework.pl b/admin/biblio_framework.pl
>index 40f5ac83dc0..a6f0e100508 100755
>--- a/admin/biblio_framework.pl
>+++ b/admin/biblio_framework.pl
>@@ -81,7 +81,7 @@ if ( $op eq 'add_form' ) {
>     $cache->clear_from_cache("MarcSubfieldStructure-$frameworkcode");
>     $cache->clear_from_cache("MarcCodedFields-$frameworkcode");
>     $op = 'list';
>-} elsif ( $op eq 'cud-delete_confirm' ) {
>+} elsif ( $op eq 'delete_confirm' ) {
>     my $framework = Koha::BiblioFrameworks->find($frameworkcode);
>     my $count = Koha::Biblios->search( { frameworkcode => $frameworkcode, } )->count;
> 
>-- 
>2.45.2

Actions: View | Diff | Splinter Review

Attachments on bug 37080: 167680 | 167682 | 167711