Bug 24380: (QA follow-up) Simplify condition

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

diff --git a/C4/Circulation.pm b/C4/Circulation.pm
index 527b878..faaa269 100644 (file)
--- a/C4/Circulation.pm
+++ b/C4/Circulation.pm
@@ -1973,11 +1973,14 @@ sub AddReturn {
                 MarkIssueReturned( $borrowernumber, $item->itemnumber, $return_date, $patron->privacy );
             };
             unless ( $@ ) {
-                my $calc_fine = C4::Context->preference('CalculateFinesOnReturn');
-                $calc_fine ||= $return_date_specified && C4::Context->preference('CalculateFinesOnBackdate');
-                $calc_fine &&= !$item->itemlost;
-
-                if ( $calc_fine ) {
+                if (
+                    (
+                        C4::Context->preference('CalculateFinesOnReturn')
+                        || ( $return_date_specified && C4::Context->preference('CalculateFinesOnBackdate') )
+                    )
+                    && !$item->itemlost
+                  )
+                {
                     _CalculateAndUpdateFine( { issue => $issue, item => $item->unblessed, borrower => $patron_unblessed, return_date => $return_date } );
                 }
             } else {