From: Jan Büren <jan@kivitendo-premium.de>
Date: Sun, 23 Jul 2017 19:20:41 +0000 (+0200)
Subject: DB/BankTransaction: Kosmetik Fall SEPA-Check
X-Git-Tag: release-3.5.4~975
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=69bfa40a001bfd0ca75d60babdb63f9199d4e45c;p=kivitendo-erp.git

DB/BankTransaction: Kosmetik Fall SEPA-Check
---

diff --git a/SL/DB/BankTransaction.pm b/SL/DB/BankTransaction.pm
index 5ad598326..26c38e51b 100644
--- a/SL/DB/BankTransaction.pm
+++ b/SL/DB/BankTransaction.pm
@@ -211,25 +211,24 @@ sub get_agreement_with_invoice {
 
 #  # if there is exactly one non-executed sepa_export_item for the invoice
   if ( my $seis = $invoice->find_sepa_export_items({ executed => 0 }) ) {
-    if ( scalar @$seis == 1 ) {
+    if (scalar @$seis == 1) {
       my $sei = $seis->[0];
 
       # test for amount and id matching only, sepa transfer date and bank
       # transaction date needn't match
       my $arap = $invoice->is_sales ? 'ar' : 'ap';
-      if (    abs($self->amount) == ($sei->amount)
-          && $invoice->id        == $sei->arap_id
-         ) {
+
+      if (abs($self->amount) == ($sei->amount) && $invoice->id == $sei->arap_id) {
         $agreement += $points{sepa_export_item};
-          $rule_matches .= 'sepa_export_item(' . $points{'sepa_export_item'} . ') ';
-      };
+        $rule_matches .= 'sepa_export_item(' . $points{'sepa_export_item'} . ') ';
+      }
     } else {
       # zero or more than one sepa_export_item, do nothing for this invoice
       # zero: do nothing, no sepa_export_item exists, no match
       # more than one: does this ever apply? Currently you can't create sepa
       # exports for invoices that already have a non-executed sepa_export
-    };
-  };
+    }
+  }
 
   return ($agreement,$rule_matches);
 };