t/helper/shipped_qty.t: items Sortierung stabilisiert

[mfinanz.git] / SL / DB / BankTransaction.pm

diff --git a/SL/DB/BankTransaction.pm b/SL/DB/BankTransaction.pm
index 26c38e51b04ca2d4af60c9a730400412ac67eee6..6ad0348c7ffe46bf38a6612616dd8f49a37fb02f 100644 (file)
--- a/SL/DB/BankTransaction.pm
+++ b/SL/DB/BankTransaction.pm
@@ -209,7 +209,7 @@ sub get_agreement_with_invoice {
     };
   };
 
     };
   };
 

-#  # if there is exactly one non-executed sepa_export_item for the 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) {
       my $sei = $seis->[0];
   if ( my $seis = $invoice->find_sepa_export_items({ executed => 0 }) ) {
     if (scalar @$seis == 1) {
       my $sei = $seis->[0];


@@ -219,7 +219,7 @@ sub get_agreement_with_invoice {
       my $arap = $invoice->is_sales ? 'ar' : 'ap';
 
       if (abs($self->amount) == ($sei->amount) && $invoice->id == $sei->arap_id) {
       my $arap = $invoice->is_sales ? 'ar' : 'ap';
 
       if (abs($self->amount) == ($sei->amount) && $invoice->id == $sei->arap_id) {

-        $agreement += $points{sepa_export_item};
+        $agreement    += $points{sepa_export_item};

         $rule_matches .= 'sepa_export_item(' . $points{'sepa_export_item'} . ') ';
       }
     } else {
         $rule_matches .= 'sepa_export_item(' . $points{'sepa_export_item'} . ') ';
       }
     } else {


@@ -227,6 +227,8 @@ sub get_agreement_with_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
       # 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

+      # TODO: Catch the more than one case. User is allowed to split
+      # payments for one invoice item in one sepa export.

     }
   }
 
     }
   }