Bug 22037: (QA follow-up) Correct misleading comment
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 13 Sep 2019 07:36:25 +0000 (07:36 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Tue, 17 Sep 2019 11:39:04 +0000 (12:39 +0100)
Comment needs to be corrected, and now applies to the whole code
paragraph following it.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
C4/SIP/ILS/Patron.pm

index 96ec74a..497ec98 100644 (file)
@@ -66,7 +66,8 @@ sub new {
     my $dexpiry     = $kp->{dateexpiry};
     $dexpiry and $dexpiry =~ s/-//g;    # YYYYMMDD
 
-    my $fines_amount = $flags->{CHARGES}->{amount}; # This "amount" is the negative balance or the one of the guarantees
+    # Get fines and add fines for guarantees (depends on preference NoIssuesChargeGuarantees)
+    my $fines_amount = $flags->{CHARGES}->{amount};
     $fines_amount = ($fines_amount and $fines_amount > 0) ? $fines_amount : 0;
     my $guarantees_fines_amount = $flags->{CHARGES_GUARANTEES} ? $flags->{CHARGES_GUARANTEES}->{amount} : 0;
     $fines_amount += $guarantees_fines_amount;