Attachment 166190 Details for Bug 36786 – Bug 36786: (Bug 31791 follow-up) Koha explodes when trying to edit a biblio with invalid biblionumber

[patch] Bug 36786: (Bug 31791 follow-up) Koha explodes when trying to edit a biblio with invalid biblionumber

Bug-36786-Bug-31791-follow-up-Koha-explodes-when-t.patch (text/plain), 1.65 KB, created by Janusz Kaczmarek on 2024-05-06 09:19:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Janusz Kaczmarek

Created: 2024-05-06 09:19:06 UTC

Size: 1.65 KB

patch

obsolete

>From 73eb409a1c7e8eb8542f019a65c1a35547d40f8f Mon Sep 17 00:00:00 2001
>From: Janusz Kaczmarek <januszop@gmail.com>
>Date: Mon, 6 May 2024 09:08:50 +0000
>Subject: [PATCH] Bug 36786: (Bug 31791 follow-up) Koha explodes when trying to
> edit a biblio with invalid biblionumber
>
>After changes made to addbiblio with the bug 31791, when trying to open the
>editor with biblionumber=<invalid_number> (e.g. a deleted biblionumber)
>Koha explodes with a message << Can't call method "can_be_edited" on
>an undefined value at /kohadevbox/koha/cataloguing/addbiblio.pl line 555 >>
>
>Test plan:
>==========
>1. Try to edit a biblio giving as a biblionumber (in URL) a non-existing
>   biblionumber, e.g. in ktd, with standard ktd test data:
>   http://your_ktd:8081/cgi-bin/koha/cataloguing/addbiblio.pl?biblionumber=1234567
>   Koha should explode with the message:
>   Can't call method "can_be_edited" on an undefined value at
>   /kohadevbox/koha/cataloguing/addbiblio.pl line 555
>2. Apply the patch; restart_all.
>3. Repeat p.1.  You should get the 403 error page.
>
>Sponsored-by: Ignatianum University in Cracow
>---
> cataloguing/addbiblio.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/cataloguing/addbiblio.pl b/cataloguing/addbiblio.pl
>index d81215d486..a3cfc55ad5 100755
>--- a/cataloguing/addbiblio.pl
>+++ b/cataloguing/addbiblio.pl
>@@ -552,7 +552,7 @@ if ($biblionumber) {
>         $template->param( bib_doesnt_exist => 1 );
>     }
> 
>-    unless ( $biblio->can_be_edited($logged_in_patron) ) {
>+    unless ( $biblio && $biblio->can_be_edited($logged_in_patron) ) {
>         print $input->redirect("/cgi-bin/koha/errors/403.pl");    # escape early
>         exit;
>     }
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 36786: 166190 | 166233 | 166260 | 166262 | 166285 | 166328