Attachment 109678 Details for Bug 25969 – Bug 25969: Don't set itemnumber as reserve->itemnumber

[patch] Bug 25969: Don't set itemnumber as reserve->itemnumber

Bug-25969-Dont-set-itemnumber-as-reserve-itemnumbe.patch (text/plain), 2.41 KB, created by Nick Clemens (kidclamp) on 2020-09-04 16:22:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2020-09-04 16:22:05 UTC

Size: 2.41 KB

patch

obsolete

>From 919af6596b7a1271ee00cb3baabe78afc3db9e40 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Fri, 4 Sep 2020 16:16:27 +0000
>Subject: [PATCH] Bug 25969: Don't set itemnumber as reserve->itemnumber
>
>When a hold is checked in at the wrong destination we revert the found status
>For a title level hold this means we return it to a title level hold, with no itemnumber
>
>When we checkin the item we find the hold, but won't set the itemnumber in the hold until it is confirmed
>
>In the script we were setting the itemnumber param, but then setting it again using the reserve itemnumber.
>In all conditions before the itemnumber has been set, we don't need to set it again, especially when we do
>so incorrectly
>
>To test:
>- Log in at Library A
>- Place hold on item at Library A, for pickup at Library A
>- Check item in at Library A
>- Click "Confirm hold"
>- Set library to Library B
>- Check item in again (try with both the box on reutrns.pl and the box in the header)
>- Click "Confirm hold and transfer"
>- get error: Can't call method "biblio" on an undefined value at /kohadevbox/koha/circ/returns.pl line 147
>- apply patch
>- repeat
>- success!
>- confirm trappings of holds works as normally otherwise too
>---
> circ/returns.pl | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
>diff --git a/circ/returns.pl b/circ/returns.pl
>index 4ffea1ed8b..a7556c7b07 100755
>--- a/circ/returns.pl
>+++ b/circ/returns.pl
>@@ -347,6 +347,7 @@ if ($barcode) {
> }
> $template->param( inputloop => \@inputloop );
> 
>+
> my $found    = 0;
> my $waiting  = 0;
> my $reserved = 0;
>@@ -427,11 +428,9 @@ if ( $messages->{'ResFound'}) {
>         if ( $messages->{'transfert'} ) {
>             $template->param(
>                 itemtitle        => $biblio->title,
>-                itemnumber       => $item->itemnumber,
>                 itembiblionumber => $biblio->biblionumber,
>                 iteminfo         => $biblio->author,
>                 diffbranch       => 1,
>-                itemnumber       => $itemnumber,
>             );
>         }
>     }
>@@ -458,7 +457,6 @@ if ( $messages->{'ResFound'}) {
>         patron         => $patron,
>         barcode        => $barcode,
>         destbranch     => $reserve->{'branchcode'},
>-        itemnumber     => $reserve->{'itemnumber'},
>         reservenotes   => $reserve->{'reservenotes'},
>         reserve_id     => $reserve->{reserve_id},
>         bormessagepref => $holdmsgpreferences->{'transports'},
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 25969: 106758 | 106771 | 107697 | 109678 | 109755 | 109756 | 110006 | 110007 | 111328 | 111344