use SL::DB::MetaSetup::BankTransaction;
use SL::DB::Manager::BankTransaction;
use SL::DB::Helper::LinkedRecords;
+use Carp;
require SL::DB::Invoice;
require SL::DB::PurchaseInvoice;
sub get_agreement_with_invoice {
my ($self, $invoice) = @_;
- die "first argument is not an invoice object"
+ carp "get_agreement_with_invoice needs an invoice object as its first argument"
unless ref($invoice) eq 'SL::DB::Invoice' or ref($invoice) eq 'SL::DB::PurchaseInvoice';
my %points = (
exact_amount => 4,
exact_open_amount => 4,
invnumber_in_purpose => 2,
+ own_invnumber_in_purpose => 5,
# overpayment => -1, # either other invoice is more likely, or several invoices paid at once
payment_before_invoice => -2,
payment_within_30_days => 1,
remote_account_number => 3,
skonto_exact_amount => 5,
wrong_sign => -1,
+ sepa_export_item => 5,
);
my ($agreement,$rule_matches);
if ( $invoice->skonto_date && abs(abs($invoice->amount_less_skonto) - abs($self->amount)) < 0.01) {
$agreement += $points{skonto_exact_amount};
$rule_matches .= 'skonto_exact_amount(' . $points{'skonto_exact_amount'} . ') ';
+ $invoice->{skonto_type} = 'with_skonto_pt';
};
#search invoice number in purpose
my $invnumber = $invoice->invnumber;
- # invnumbernhas to have at least 3 characters
- if ( length($invnumber) > 2 && $self->purpose =~ /\b$invnumber\b/i ) {
+ # invnumber has to have at least 3 characters
+ my $squashed_purpose = $self->purpose;
+ $squashed_purpose =~ s/ //g;
+ if (length($invnumber) > 4 && $squashed_purpose =~ /$invnumber/ && $invoice->is_sales){
+ $agreement += $points{own_invnumber_in_purpose};
+ $rule_matches .= 'own_invnumber_in_purpose(' . $points{'own_invnumber_in_purpose'} . ') ';
+ } elsif (length($invnumber) > 3 && $squashed_purpose =~ /$invnumber/ ) {
$agreement += $points{invnumber_in_purpose};
$rule_matches .= 'invnumber_in_purpose(' . $points{'invnumber_in_purpose'} . ') ';
- };
+ }
#check sign
if ( $invoice->is_sales && $self->amount < 0 ) {
};
};
+# # 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];
+#
+# # 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
+# ) {
+# $agreement += $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);
};