From c13e245409062deece6624779bbd8e934cb5ff11 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Wed, 1 Oct 2025 16:34:38 +0200 Subject: [PATCH] Bug 40929: Add $hold test in opac-modrequest MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Test plan: Login at OPAC. Pass an unexisting reserve_id in a opac-modrequest.pl URL. Verify that you get a 403 redirect. Signed-off-by: Marcel de Rooy Signed-off-by: Tomás Cohen Arazi --- opac/opac-modrequest.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/opac/opac-modrequest.pl b/opac/opac-modrequest.pl index de61d61a12..2cf6712685 100755 --- a/opac/opac-modrequest.pl +++ b/opac/opac-modrequest.pl @@ -46,7 +46,7 @@ if ( $reserve_id && $borrowernumber ) { my $hold = Koha::Holds->find($reserve_id); - unless ( $hold->borrowernumber == $borrowernumber ) { + unless ( $hold && $hold->borrowernumber == $borrowernumber ) { # whatcha tryin to do? print $query->redirect('/cgi-bin/koha/errors/403.pl'); -- 2.51.0