Attachment 138877 Details for Bug 22321 – Bug 22321: Fix store() crash when modifying borrowernumber of a new ILL request

[patch] Bug 22321: Fix store() crash when modifying borrowernumber of a new ILL request

Bug-22321-Fix-store-crash-when-modifying-borrowern.patch (text/plain), 1.40 KB, created by Katrin Fischer on 2022-08-09 06:16:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2022-08-09 06:16:56 UTC

Size: 1.40 KB

patch

obsolete

>From 0e9bbcb9131bafcaf1605a9b843271cfb1903077 Mon Sep 17 00:00:00 2001
>From: Lari Taskula <lari.taskula@hypernova.fi>
>Date: Tue, 22 Feb 2022 14:13:58 +0000
>Subject: [PATCH] Bug 22321: Fix store() crash when modifying borrowernumber of
> a new ILL request
>
>To test:
>1. prove t/db_dependent/Illrequests.t
>
>Signed-off-by: Magnus Enger <magnus@libriotech.no>
>Tried to change the borrowernumber connected to an ILL request
>and checked that the associated hold was updated to the new
>patron as well.
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> Koha/Illrequest.pm | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/Koha/Illrequest.pm b/Koha/Illrequest.pm
>index ae82921a05..0805111538 100644
>--- a/Koha/Illrequest.pm
>+++ b/Koha/Illrequest.pm
>@@ -1637,7 +1637,7 @@ sub store {
>     my %updated_columns = $self->_result->get_dirty_columns;
> 
>     my @holds;
>-    if( defined $updated_columns{'borrowernumber'} and
>+    if( $self->in_storage and defined $updated_columns{'borrowernumber'} and
>         Koha::Patrons->find( $updated_columns{'borrowernumber'} ) )
>     {
>         # borrowernumber has changed
>@@ -1645,7 +1645,7 @@ sub store {
>         @holds = Koha::Holds->search( {
>             borrowernumber => $old_illreq->borrowernumber,
>             biblionumber   => $self->biblio_id,
>-        } )->as_list;
>+        } )->as_list if $old_illreq;
>     }
> 
>     my $ret = $self->SUPER::store;
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 22321: 130544 | 130545 | 130546 | 131012 | 131013 | 131055 | 131056 | 131057 | 131058 | 131059 | 138873 | 138874 | 138875 | 138876 | 138877 | 138878 | 141342 | 141343 | 141344 | 141345 | 141346 | 141347