From 5ce78e4398ca2cf1daaded36c421861d68f5abde Mon Sep 17 00:00:00 2001 From: Colin Campbell Date: Thu, 25 Oct 2018 10:12:08 +0100 Subject: [PATCH] Bug 21658 Pass fee_ack value to Renew transaction Response to a renew request should mirror that of checkout However fee_ack was not being passed to the renewal transaction so that it was always 'N' not allowing the renewal to proceed even although the fee was paid This bug was not seen with many units as they use checkout requests both for an initial issue and renewal. Code now passes fee_ack as is done for checkout Signed-off-by: Liz Rea Signed-off-by: Liz Rea --- C4/SIP/ILS.pm | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/C4/SIP/ILS.pm b/C4/SIP/ILS.pm index aeaa0d2bec..40c4e92524 100644 --- a/C4/SIP/ILS.pm +++ b/C4/SIP/ILS.pm @@ -442,6 +442,10 @@ sub renew { $trans->screen_msg("Renewals not allowed."); return $trans; } + # pass fee_ack to the transaction + if ($fee_ack) { + $trans->fee_ack($fee_ack); + } # Previously: renewing a title, rather than an item (sort of) # This is gross, but in a real ILS it would be better -- 2.11.0