From 973251c2a471e5c4b2901b21f187c124f7529509 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 9 Feb 2023 14:56:01 -0500 Subject: [PATCH] Bug 32934: Correctly honor the no block due date for SIP checkout messages --- C4/Circulation.pm | 8 ++++---- C4/SIP/ILS/Transaction/Checkout.pm | 6 ++++-- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index 109809d4b99..28f91ad88f0 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -4164,7 +4164,7 @@ sub ProcessOfflineOperation { if ( $operation->{action} eq 'return' ) { $report = ProcessOfflineReturn( $operation ); } elsif ( $operation->{action} eq 'issue' ) { - $report = ProcessOfflineIssue( $operation ); + ( $report ) = ProcessOfflineIssue( $operation ); } elsif ( $operation->{action} eq 'payment' ) { $report = ProcessOfflinePayment( $operation ); } @@ -4224,13 +4224,13 @@ sub ProcessOfflineIssue { } AddIssue( $patron, - $operation->{'barcode'}, - undef, + $operation->{barcode}, + $operation->{due_date}, undef, $operation->{timestamp}, undef, ); - return "Success."; + return ( "Success.", $checkout ); } else { return "Borrower not found."; } diff --git a/C4/SIP/ILS/Transaction/Checkout.pm b/C4/SIP/ILS/Transaction/Checkout.pm index c13e5682607..fefb22d12bd 100644 --- a/C4/SIP/ILS/Transaction/Checkout.pm +++ b/C4/SIP/ILS/Transaction/Checkout.pm @@ -148,11 +148,13 @@ sub do_checkout { if ( $no_block_due_date ) { $overridden_duedate = $no_block_due_date; - ProcessOfflineIssue({ + my ( $msg, $checkout ) = ProcessOfflineIssue({ cardnumber => $patron->cardnumber, barcode => $barcode, - timestamp => $no_block_due_date, + due_date => $no_block_due_date, + timestamp => dt_from_string, }); + $self->{due} = $self->duedatefromissue($checkout, $itemnumber); } else { # can issue my $recall_id; -- 2.30.2