Bug 7639: tiny QA follow-up
authorPaul Poulain <paul.poulain@biblibre.com>
Thu, 21 Mar 2013 10:24:38 +0000 (11:24 +0100)
committerGalen Charlton <gmc@esilibrary.com>
Tue, 17 Sep 2013 15:21:23 +0000 (15:21 +0000)
no need to create a variable just to call to send it as parameter the line after

Signed off by: Alex Hatley <alexh@cctexas.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
C4/Circulation.pm

index 16575a4..b775b4f 100644 (file)
@@ -3310,9 +3310,7 @@ sub LostItem{
         my $borrower = C4::Members::GetMemberDetails( $borrowernumber );
 
         if (C4::Context->preference('WhenLostForgiveFine')){
-            my $exemptfine=1;
-            my $dropbox=0;
-            my $fix = _FixOverduesOnReturn($borrowernumber, $itemnumber, $exemptfine, $dropbox);
+            my $fix = _FixOverduesOnReturn($borrowernumber, $itemnumber, 1, 0); # 1, 0 = exemptfine, no-dropbox
             defined($fix) or warn "_FixOverduesOnReturn($borrowernumber, $itemnumber...) failed!";  # zero is OK, check defined
         }
         if (C4::Context->preference('WhenLostChargeReplacementFee')){