From f402b0eccdc44c5511bbf6b7da88b6e143b7163e Mon Sep 17 00:00:00 2001 From: Janusz Kaczmarek 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= (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 a page with the info: "The record you are trying to edit doesn't exist...". Sponsored-by: Ignatianum University in Cracow Signed-off-by: Matt Blenkinsop Signed-off-by: Tomas Cohen Arazi Signed-off-by: Katrin Fischer --- cataloguing/addbiblio.pl | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/cataloguing/addbiblio.pl b/cataloguing/addbiblio.pl index 3d92cdad1f..97ecddd5e5 100755 --- a/cataloguing/addbiblio.pl +++ b/cataloguing/addbiblio.pl @@ -547,15 +547,15 @@ if ($biblionumber) { $biblio = Koha::Biblios->find($biblionumber); - unless ($biblio) { + if ($biblio) { + unless ( $biblio->can_be_edited($logged_in_patron) ) { + print $input->redirect("/cgi-bin/koha/errors/403.pl"); # escape early + exit; + } + } else { $biblionumber = undef; $template->param( bib_doesnt_exist => 1 ); } - - unless ( $biblio->can_be_edited($logged_in_patron) ) { - print $input->redirect("/cgi-bin/koha/errors/403.pl"); # escape early - exit; - } } if ($frameworkcode eq 'FA'){ -- 2.39.5