From ecf3aa475232b4b7cb13362e3551fe1047c52b47 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Joonas=20Kylm=C3=A4l=C3=A4?= Date: Thu, 29 Jul 2021 19:35:47 +0300 Subject: [PATCH] Bug 28779: Skip processing of non-existent biblios Just checking with a regex that whether an input looks like a biblionumber is not enough, we need to also verify there is a biblio really existing in the database and skip processing of hold request for non-existent biblionumbers. To test: 1) Go to page /cgi-bin/koha/reserve/request.pl?biblionumbers=XXXXX where XXXXX is non-existent biblionumber, notice internal server error 2) Apply patch 3) Repeat step 1 and notice we cannot place a hold Signed-off-by: Owen Leonard Signed-off-by: Nick Clemens Signed-off-by: Petro Vashchuk Signed-off-by: Katrin Fischer Signed-off-by: Jonathan Druart Signed-off-by: Kyle M Hall (cherry picked from commit 9f1a5069ed836f99988cabbeb052ff245e96fb52) Signed-off-by: Fridolin Somers --- reserve/request.pl | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/reserve/request.pl b/reserve/request.pl index 8bf82f284e..2f57381924 100755 --- a/reserve/request.pl +++ b/reserve/request.pl @@ -299,6 +299,11 @@ foreach my $biblionumber (@biblionumbers) { my %biblioloopiter = (); my $biblio = Koha::Biblios->find( $biblionumber ); + unless ($biblio) { + $biblioloopiter{noitems} = 1; + $template->param('noitems' => 1); + next; + } my $force_hold_level; if ( $patron ) { -- 2.39.5