Attachment 169161 Details for Bug 37385 – Bug 37385: Fix Transfer not triggering automatically when cancelling a hold by checking in an item

[patch] Bug 37385: Fix Transfer not triggering automatically when cancelling a hold by checking in an item

Bug-37385-Fix-Transfer-not-triggering-automaticall.patch (text/plain), 1.38 KB, created by Kyle M Hall (khall) on 2024-07-18 17:55:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2024-07-18 17:55:03 UTC

Size: 1.38 KB

patch

obsolete

>From a508a99a78242a711bf9f63ae9da77d3baa13efe Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Thu, 18 Jul 2024 12:54:30 +0000
>Subject: [PATCH] Bug 37385: Fix Transfer not triggering automatically when
> cancelling a hold by checking in an item
>
>Upadates for CSRF are inadvertently stopping AddReturn from being trigger
>when a hold is canceled. This is necessary to generate the transfer
>back to the originating library.
>
>Test Plan:
>1) Find item with a waiting hold, and check it in
>2) In the popup notifying me of the waiting hold, select 'Cancel hold'
>3) Note no second popup appears notifying you of the need to transfer the item to its home library
>4) Apply this patch
>5) Restart all the things!
>6) Repeat steps 1-2
>7) Second popup appears!
>
>Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
>---
> circ/returns.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/circ/returns.pl b/circ/returns.pl
>index 019d8e36c84..f3182464df3 100755
>--- a/circ/returns.pl
>+++ b/circ/returns.pl
>@@ -293,7 +293,7 @@ if ($transit && $op eq 'cud-transfer') {
> 
> # actually return book and prepare item table.....
> my $returnbranch;
>-if ($barcode && $op eq 'cud-checkin') {
>+if ($barcode && ( $op eq 'cud-checkin' || $op eq 'cud-affect_reserve' ) ) {
>     $barcode = barcodedecode($barcode) if $barcode;
>     my $item = Koha::Items->find({ barcode => $barcode });
> 
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 37385: 169132 | 169134 | 169161 | 169166