Attachment 107625 Details for Bug 26078 – Bug 26078: Prevent generating a new transfer from return policy when resolving a transfer

[patch] Bug 26078: Prevent generating a new transfer from return policy when resolving a transfer

Bug-26078-Prevent-generating-a-new-transfer-from-r.patch (text/plain), 2.84 KB, created by ByWater Sandboxes on 2020-07-31 09:26:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: ByWater Sandboxes

Created: 2020-07-31 09:26:59 UTC

Size: 2.84 KB

patch

obsolete

>From bff62f85ab258761d431dd23147a1cea81665f48 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Thu, 30 Jul 2020 16:59:14 +0000
>Subject: [PATCH] Bug 26078: Prevent generating a new transfer from return
> policy when resolving a transfer
>
>To test:
>- Have 2+ branches
>- Set your default return policy to Item returns to issuing library
>- Find an item with holding branch of Branch A
>- Set your library to Branch B
>- Check item in
>- Koha generates a transfer to Branch A. In doing so, it sets the item holding branch to Branch B
>- Set your library to Branch A
>- Check item in
>- Koha generate a transfer to Branch B. In doing so, it sets the item holding branch to Branch A
>- Set you library to Branch A
>- Check the item in
> . . . Repeat indefinitely.
> Apply patch
> Repeat
> No more loop
>
>Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
>---
> C4/Circulation.pm                   | 3 +++
> t/db_dependent/Circulation/Branch.t | 8 +++++++-
> 2 files changed, 10 insertions(+), 1 deletion(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 4b468f80f8..6a4f7999ca 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -2049,6 +2049,9 @@ sub AddReturn {
>                 "UPDATE branchtransfers SET datearrived = now() WHERE itemnumber= ? AND datearrived IS NULL"
>             );
>             $sth->execute( $item->itemnumber );
>+
>+            # If we are completing a transfer we should not generate a new transfer from return policy
>+            $returnbranch = $branch;
>         } else {
>             $messages->{'WrongTransfer'}     = $tobranch;
>             $messages->{'WrongTransferItem'} = $item->itemnumber;
>diff --git a/t/db_dependent/Circulation/Branch.t b/t/db_dependent/Circulation/Branch.t
>index 16e590fcc3..c8fe89dc97 100644
>--- a/t/db_dependent/Circulation/Branch.t
>+++ b/t/db_dependent/Circulation/Branch.t
>@@ -25,7 +25,7 @@ use Koha::CirculationRules;
> 
> use Koha::Patrons;
> 
>-use Test::More tests => 14;
>+use Test::More tests => 15;
> use t::lib::Mocks;
> use t::lib::TestBuilder;
> 
>@@ -303,6 +303,12 @@ $dbh->do( $query, {}, $borrower_id1, $item_id2, $samplebranch2->{branchcode} );
>     $samplebranch2->{branchcode});
> is( $messages->{NeedsTransfer}, $samplebranch1->{branchcode}, "AddReturn respects branch return policy - item2->homebranch policy = 'holdingbranch'" );
> 
>+# Generate the transfer from above
>+ModItemTransfer($item_id2, $samplebranch2->{branchcode}, $samplebranch1->{branchcode}, "ReturnToHolding");
>+# Fulfill it
>+($doreturn, $messages, $iteminformation, $borrower) = AddReturn('barcode_2',$samplebranch1->{branchcode});
>+is( $messages->{NeedsTransfer}, undef, "AddReturn does not generate a new transfer for return policy when resolving an existing transfer" );
>+
> # item3 should not trigger transfer - floating collection
> $query =
> "INSERT INTO issues (borrowernumber,itemnumber,branchcode) VALUES( ?,?,? )";
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 26078: 107597 | 107625 | 108188 | 108259 | 108278 | 108496