From 4225278c87e9ff1eabf7fd76fbf7a3da4bf8b532 Mon Sep 17 00:00:00 2001 From: Katrin Fischer Date: Sun, 14 Dec 2014 22:31:04 +0100 Subject: [PATCH] Bug 13461 - Circ always asking for confirmation if RentalFeesCheckoutConfirmation is used To test: - Check RentalFeesCheckoutConfirmation is activated - Try to check out an item without rental fine - Verify confirmation message without explanation is shown - Apply patch - Verify confirmation message is no longer shown - Configure itemtype to have rental fee - Veirfy now the confirmation message appears as it should Signed-off-by: Paola Rossi Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 22b8d559f9c8f5733d051313c4d6266570d5be6f) Signed-off-by: Chris Cormack --- C4/Circulation.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index f24f2f749d..5f7e866b33 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -928,7 +928,7 @@ sub CanBookBeIssued { if ( $rentalConfirmation ){ my ($rentalCharge) = GetIssuingCharges( $item->{'itemnumber'}, $borrower->{'borrowernumber'} ); - if ( $rentalCharge ){ + if ( $rentalCharge > 0 ){ $rentalCharge = sprintf("%.02f", $rentalCharge); $needsconfirmation{RENTALCHARGE} = $rentalCharge; } -- 2.39.5