From e84686ec0c8b5aa6108eb699b5dc2751743318b2 Mon Sep 17 00:00:00 2001 From: Katrin Fischer Date: Sun, 14 Dec 2014 22:31:04 +0100 Subject: [PATCH] [PASSED QA] 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 --- C4/Circulation.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index f24f2f7..5f7e866 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; } -- 1.7.10.4