Attachment 166606 Details for Bug 36832 – Bug 36832: (bug 36791 follow-up) Allow authid=0

[patch] Bug 36832: (bug 36791 follow-up) Allow authid=0

Bug-36832-bug-36791-follow-up-Allow-authid0.patch (text/plain), 1.45 KB, created by Janusz Kaczmarek on 2024-05-10 19:01:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Janusz Kaczmarek

Created: 2024-05-10 19:01:44 UTC

Size: 1.45 KB

patch

obsolete

>From b27f8f60f1d1e5a4bfd94a2997a0e9cbf54daea5 Mon Sep 17 00:00:00 2001
>From: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
>Date: Fri, 10 May 2024 14:01:59 -0400
>Subject: [PATCH] Bug 36832: (bug 36791 follow-up) Allow authid=0
>
>To test:
>1. Go to the Authorities module and do a Z39.50 search that will return
>   results (e.g. a general subject heading)
>2. Import an authority record from the results
>--> Confirm that the record is imported into the editor
>3. Authorities > New, replace it via Z39.50, confirm it's imported
>4. Authorities > New, fill the required fields, confirm it saves
>5. Edit an existing authority, replace it via Z39.50, confirm it saves
>6. Edit an existing authority, replace authid=nn in the URL with a
>   number that doesn't exist like 1000000, confirm you get a 404 page
>
>Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
>Signed-off-by: Janusz Kaczmarek <januszop@gmail.com>
>---
> authorities/authorities.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/authorities/authorities.pl b/authorities/authorities.pl
>index b2fad5d2a8..3b8a235636 100755
>--- a/authorities/authorities.pl
>+++ b/authorities/authorities.pl
>@@ -553,7 +553,7 @@ if ( $op eq 'cud-change-framework' ) {
> 
> my $dbh = C4::Context->dbh;
> my $authobj = Koha::Authorities->find($authid);
>-if ( defined $authid && !$authobj ) {
>+if ( $authid && !$authobj ) {
>     print $input->redirect("/cgi-bin/koha/errors/404.pl");    # escape early
>     exit;
> }
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 36832: 166592 | 166604 | 166605 | 166606 | 166613