1 package SL::DB::Helper::Payment;
 
   5 use parent qw(Exporter);
 
   6 our @EXPORT = qw(pay_invoice);
 
   7 our @EXPORT_OK = qw(skonto_date skonto_charts amount_less_skonto within_skonto_period percent_skonto reference_account reference_amount open_amount open_percent remaining_skonto_days skonto_amount check_skonto_configuration valid_skonto_amount get_payment_suggestions validate_payment_type open_sepa_transfer_amount get_payment_select_options_for_bank_transaction create_bank_transaction exchangerate forex);
 
   9   "ALL" => [@EXPORT, @EXPORT_OK],
 
  12 require SL::DB::Chart;
 
  15 use SL::DATEV qw(:CONSTANTS);
 
  16 use SL::Locale::String qw(t8);
 
  17 use List::Util qw(sum);
 
  18 use SL::DB::Exchangerate;
 
  23 # Public functions not exported by default
 
  27   my ($self, %params) = @_;
 
  31   my $is_sales = ref($self) eq 'SL::DB::Invoice';
 
  32   my $mult = $is_sales ? 1 : -1;  # multiplier for getting the right sign depending on ar/ap
 
  34   my $paid_amount = 0; # the amount that will be later added to $self->paid, should be in default currency
 
  36   # default values if not set
 
  37   $params{payment_type} = 'without_skonto' unless $params{payment_type};
 
  38   validate_payment_type($params{payment_type});
 
  40   # check for required parameters
 
  41   Common::check_params(\%params, qw(chart_id transdate));
 
  44   if (ref($params{transdate} eq 'DateTime')) {
 
  45     $transdate_obj = $params{transdate};
 
  47    $transdate_obj = $::locale->parse_date_to_object($params{transdate});
 
  49   croak t8('Illegal date') unless ref $transdate_obj;
 
  51   # check for closed period
 
  52   my $closedto = $::locale->parse_date_to_object($::instance_conf->get_closedto);
 
  53   if ( ref $closedto && $transdate_obj < $closedto ) {
 
  54     croak t8('Cannot post payment for a closed period!');
 
  57   # check for maximum number of future days
 
  58   if ( $::instance_conf->get_max_future_booking_interval > 0 ) {
 
  59     croak t8('Cannot post transaction above the maximum future booking date!') if $transdate_obj > DateTime->now->add( days => $::instance_conf->get_max_future_booking_interval );
 
  62   # currency is either passed or use the invoice currency if it differs from the default currency
 
  63   my ($exchangerate,$currency);
 
  64   if ($params{currency} || $params{currency_id} || $self->currency_id != $::instance_conf->get_currency_id) {
 
  65     if ($params{currency} || $params{currency_id} ) { # currency was specified
 
  66       $currency = SL::DB::Manager::Currency->find_by(name => $params{currency}) || SL::DB::Manager::Currency->find_by(id => $params{currency_id});
 
  67     } else { # use invoice currency
 
  68       $currency = SL::DB::Manager::Currency->find_by(id => $self->currency_id);
 
  70     die "no currency" unless $currency;
 
  71     if ($currency->id == $::instance_conf->get_currency_id) {
 
  74       my $rate = SL::DB::Manager::Exchangerate->find_by(currency_id => $currency->id,
 
  75                                                         transdate   => $transdate_obj,
 
  78         $exchangerate = $is_sales ? $rate->buy : $rate->sell;
 
  80         die "No exchange rate for " . $transdate_obj->to_kivitendo;
 
  83   } else { # no currency param given or currency is the same as default_currency
 
  88   if ( $params{'payment_type'} eq 'without_skonto' ) {
 
  89     croak "invalid amount for payment_type 'without_skonto': $params{'amount'}\n" unless abs($params{'amount'}) > 0;
 
  92   # options with_skonto_pt and difference_as_skonto don't require the parameter
 
  93   # amount, but if amount is passed, make sure it matches the expected value
 
  94   if ( $params{'payment_type'} eq 'difference_as_skonto' ) {
 
  95     croak "amount $params{amount} doesn't match open amount " . $self->open_amount . ", diff = " . ($params{amount}-$self->open_amount) if $params{amount} && abs($self->open_amount - $params{amount} ) > 0.0000001;
 
  96   } elsif ( $params{'payment_type'} eq 'with_skonto_pt' ) {
 
  97     croak "amount $params{amount} doesn't match amount less skonto: " . $self->amount_less_skonto . "\n" if $params{amount} && abs($self->amount_less_skonto - $params{amount} ) > 0.0000001;
 
  98     croak "payment type with_skonto_pt can't be used if payments have already been made" if $self->paid != 0;
 
 101   # absolute skonto amount for invoice, use as reference sum to see if the
 
 102   # calculated skontos add up
 
 103   # only needed for payment_term "with_skonto_pt"
 
 105   my $skonto_amount_check = $self->skonto_amount; # variable should be zero after calculating all skonto
 
 106   my $total_open_amount   = $self->open_amount;
 
 108   # account where money is paid to/from: bank account or cash
 
 109   my $account_bank = SL::DB::Manager::Chart->find_by(id => $params{chart_id});
 
 110   croak "can't find bank account" unless ref $account_bank;
 
 112   my $reference_account = $self->reference_account;
 
 113   croak "can't find reference account (link = AR/AP) for invoice" unless ref $reference_account;
 
 115   my $memo   = $params{'memo'}   || '';
 
 116   my $source = $params{'source'} || '';
 
 118   my $rounded_params_amount = _round( $params{amount} ); # / $exchangerate);
 
 119   my $fx_gain_loss_amount = 0; # for fx_gain and fx_loss
 
 122   $db->with_transaction(sub {
 
 125     # all three payment type create 1 AR/AP booking (the paid part)
 
 126     # difference_as_skonto creates n skonto bookings (1 for each tax type)
 
 127     # with_skonto_pt creates 1 bank booking and n skonto bookings (1 for each tax type)
 
 128     # without_skonto creates 1 bank booking
 
 130     # as long as there is no automatic tax, payments are always booked with
 
 133     unless ( $params{payment_type} eq 'difference_as_skonto' ) {
 
 134       # cases with_skonto_pt and without_skonto
 
 136       # for case with_skonto_pt we need to know the corrected amount at this
 
 137       # stage if we are going to use $params{amount}
 
 139       my $pay_amount = $rounded_params_amount;
 
 140       $pay_amount = $self->amount_less_skonto if $params{payment_type} eq 'with_skonto_pt';
 
 142       # bank account and AR/AP
 
 143       $paid_amount += $pay_amount * $exchangerate;
 
 145       my $amount = (-1 * $pay_amount) * $mult;
 
 148       # total amount against bank, do we already know this by now?
 
 149       $new_acc_trans = SL::DB::AccTransaction->new(trans_id   => $self->id,
 
 150                                                    chart_id   => $account_bank->id,
 
 151                                                    chart_link => $account_bank->link,
 
 153                                                    transdate  => $transdate_obj,
 
 156                                                    project_id => $params{project_id} ? $params{project_id} : undef,
 
 158                                                    tax_id     => SL::DB::Manager::Tax->find_by(taxkey => 0)->id);
 
 159       $new_acc_trans->save;
 
 161       # deal with fxtransaction
 
 162       if ( $self->currency_id != $::instance_conf->get_currency_id ) {
 
 163         my $fxamount = _round($amount - ($amount * $exchangerate));
 
 164         $new_acc_trans = SL::DB::AccTransaction->new(trans_id       => $self->id,
 
 165                                                      chart_id       => $account_bank->id,
 
 166                                                      chart_link     => $account_bank->link,
 
 167                                                      amount         => $fxamount * -1,
 
 168                                                      transdate      => $transdate_obj,
 
 173                                                      tax_id         => SL::DB::Manager::Tax->find_by(taxkey => 0)->id);
 
 174         $new_acc_trans->save;
 
 175         # if invoice exchangerate differs from exchangerate of payment
 
 176         # deal with fxloss and fxamount
 
 177         if ($self->exchangerate and $self->exchangerate != 1 and $self->exchangerate != $exchangerate) {
 
 178           my $fxgain_chart = SL::DB::Manager::Chart->find_by(id => $::instance_conf->get_fxgain_accno_id) || die "Can't determine fxgain chart";
 
 179           my $fxloss_chart = SL::DB::Manager::Chart->find_by(id => $::instance_conf->get_fxloss_accno_id) || die "Can't determine fxloss chart";
 
 180           my $gain_loss_amount = _round($amount * ($exchangerate - $self->exchangerate ) * -1,2);
 
 181           my $gain_loss_chart = $gain_loss_amount > 0 ? $fxgain_chart : $fxloss_chart;
 
 182           $fx_gain_loss_amount = $gain_loss_amount;
 
 184           $new_acc_trans = SL::DB::AccTransaction->new(trans_id       => $self->id,
 
 185                                                        chart_id       => $gain_loss_chart->id,
 
 186                                                        chart_link     => $gain_loss_chart->link,
 
 187                                                        amount         => $gain_loss_amount,
 
 188                                                        transdate      => $transdate_obj,
 
 193                                                        tax_id         => SL::DB::Manager::Tax->find_by(taxkey => 0)->id);
 
 194           $new_acc_trans->save;
 
 200     if ( $params{payment_type} eq 'difference_as_skonto' or $params{payment_type} eq 'with_skonto_pt' ) {
 
 202       my $total_skonto_amount;
 
 203       if ( $params{payment_type} eq 'with_skonto_pt' ) {
 
 204         $total_skonto_amount = $self->skonto_amount;
 
 205       } elsif ( $params{payment_type} eq 'difference_as_skonto' ) {
 
 206         $total_skonto_amount = $self->open_amount;
 
 209       my @skonto_bookings = $self->skonto_charts($total_skonto_amount);
 
 212       if ( $params{payment_type} eq 'difference_as_skonto' ) {
 
 213         my $calculated_skonto_sum  = sum map { $_->{skonto_amount} } @skonto_bookings;
 
 214         croak "calculated skonto for difference_as_skonto = $calculated_skonto_sum doesn't add up open amount: " . $self->open_amount unless _round($calculated_skonto_sum) == _round($self->open_amount);
 
 217       my $reference_amount = $total_skonto_amount;
 
 219       # create an acc_trans entry for each result of $self->skonto_charts
 
 220       foreach my $skonto_booking ( @skonto_bookings ) {
 
 221         next unless $skonto_booking->{'chart_id'};
 
 222         next unless $skonto_booking->{'skonto_amount'} != 0;
 
 223         my $amount = -1 * $skonto_booking->{skonto_amount};
 
 224         $new_acc_trans = SL::DB::AccTransaction->new(trans_id   => $self->id,
 
 225                                                      chart_id   => $skonto_booking->{'chart_id'},
 
 226                                                      chart_link => SL::DB::Manager::Chart->find_by(id => $skonto_booking->{'chart_id'})->link,
 
 227                                                      amount     => $amount * $mult,
 
 228                                                      transdate  => $transdate_obj,
 
 229                                                      source     => $params{source},
 
 231                                                      tax_id     => SL::DB::Manager::Tax->find_by(taxkey => 0)->id);
 
 233         # the acc_trans entries are saved individually, not added to $self and then saved all at once
 
 234         $new_acc_trans->save;
 
 236         $reference_amount -= abs($amount);
 
 237         $paid_amount      += -1 * $amount * $exchangerate;
 
 238         $skonto_amount_check -= $skonto_booking->{'skonto_amount'};
 
 240       if ( $params{payment_type} eq 'difference_as_skonto' ) {
 
 241           die "difference_as_skonto calculated incorrectly, sum of calculated payments doesn't add up to open amount $total_open_amount, reference_amount = $reference_amount\n" unless _round($reference_amount) == 0;
 
 248     if ( $params{payment_type} eq 'difference_as_skonto' ) {
 
 249       $arap_amount = $total_open_amount;
 
 250     } elsif ( $params{payment_type} eq 'without_skonto' ) {
 
 251       $arap_amount = $rounded_params_amount;
 
 252     } elsif ( $params{payment_type} eq 'with_skonto_pt' ) {
 
 253       # this should be amount + sum(amount+skonto), but while we only allow
 
 254       # with_skonto_pt for completely unpaid invoices we just use the value
 
 256       $arap_amount = $total_open_amount;
 
 259     # regardless of payment_type there is always only exactly one arap booking
 
 260     # TODO: compare $arap_amount to running total
 
 261     my $arap_booking= SL::DB::AccTransaction->new(trans_id   => $self->id,
 
 262                                                   chart_id   => $reference_account->id,
 
 263                                                   chart_link => $reference_account->link,
 
 264                                                   amount     => _round($arap_amount * $mult * $exchangerate - $fx_gain_loss_amount),
 
 265                                                   transdate  => $transdate_obj,
 
 266                                                   source     => '', #$params{source},
 
 268                                                   tax_id     => SL::DB::Manager::Tax->find_by(taxkey => 0)->id);
 
 271     $fx_gain_loss_amount *= -1 if $self->is_sales;
 
 272     $self->paid($self->paid + _round($paid_amount) + $fx_gain_loss_amount) if $paid_amount;
 
 273     $self->datepaid($transdate_obj);
 
 276     # make sure transactions will be reloaded the next time $self->transactions
 
 277     # is called, as pay_invoice saves the acc_trans objects individually rather
 
 278     # than adding them to the transaction relation array.
 
 279     $self->forget_related('transactions');
 
 283       if ( (  $self->invoice && $::instance_conf->get_datev_check_on_sales_invoice  ) ||
 
 284            ( !$self->invoice && $::instance_conf->get_datev_check_on_ar_transaction )) {
 
 288       if ( (  $self->invoice && $::instance_conf->get_datev_check_on_purchase_invoice ) ||
 
 289            ( !$self->invoice && $::instance_conf->get_datev_check_on_ap_transaction   )) {
 
 294     if ( $datev_check ) {
 
 296       my $datev = SL::DATEV->new(
 
 297         exporttype => DATEV_ET_BUCHUNGEN,
 
 298         format     => DATEV_FORMAT_KNE,
 
 300         trans_id   => $self->{id},
 
 303       $datev->clean_temporary_directories;
 
 306       if ($datev->errors) {
 
 307         # this exception should be caught by with_transaction, which handles the rollback
 
 308         die join "\n", $::locale->text('DATEV check returned errors:'), $datev->errors;
 
 314   }) || die t8('error while paying invoice #1 : ', $self->invnumber) . $db->error . "\n";
 
 323   my $is_sales = ref($self) eq 'SL::DB::Invoice';
 
 328     return undef unless ref $self->payment_terms;
 
 329     return undef unless $self->payment_terms->terms_skonto > 0;
 
 330     $skonto_date = DateTime->from_object(object => $self->transdate)->add(days => $self->payment_terms->terms_skonto);
 
 332     return undef unless ref $self->vendor->payment_terms;
 
 333     return undef unless $self->vendor->payment_terms->terms_skonto > 0;
 
 334     $skonto_date = DateTime->from_object(object => $self->transdate)->add(days => $self->vendor->payment_terms->terms_skonto);
 
 340 sub reference_account {
 
 343   my $is_sales = ref($self) eq 'SL::DB::Invoice';
 
 345   require SL::DB::Manager::AccTransaction;
 
 347   my $link_filter = $is_sales ? 'AR' : 'AP';
 
 349   my $acc_trans = SL::DB::Manager::AccTransaction->find_by(
 
 350      trans_id   => $self->id,
 
 351      SL::DB::Manager::AccTransaction->chart_link_filter("$link_filter")
 
 354   return undef unless ref $acc_trans;
 
 356   my $reference_account = SL::DB::Manager::Chart->find_by(id => $acc_trans->chart_id);
 
 358   return $reference_account;
 
 361 sub reference_amount {
 
 364   my $is_sales = ref($self) eq 'SL::DB::Invoice';
 
 366   require SL::DB::Manager::AccTransaction;
 
 368   my $link_filter = $is_sales ? 'AR' : 'AP';
 
 370   my $acc_trans = SL::DB::Manager::AccTransaction->find_by(
 
 371      trans_id   => $self->id,
 
 372      SL::DB::Manager::AccTransaction->chart_link_filter("$link_filter")
 
 375   return undef unless ref $acc_trans;
 
 377   # this should be the same as $self->amount
 
 378   return $acc_trans->amount;
 
 385   # in the future maybe calculate this from acc_trans
 
 387   # if the difference is 0.01 Cent this may end up as 0.009999999999998
 
 388   # numerically, so round this value when checking for cent threshold >= 0.01
 
 390   return ($self->amount // 0) - ($self->paid // 0);
 
 396   return 0 if $self->amount == 0;
 
 398   if ( $self->open_amount < 0 ) {
 
 399     # overpaid, currently treated identically
 
 400     $open_percent = $self->open_amount * 100 / $self->amount;
 
 402     $open_percent = $self->open_amount * 100 / $self->amount;
 
 405   return _round($open_percent) || 0;
 
 411   return $self->amount - $self->amount_less_skonto;
 
 414 sub remaining_skonto_days {
 
 417   return undef unless ref $self->skonto_date;
 
 419   my $dur = DateTime::Duration->new($self->skonto_date - DateTime->today);
 
 420   return $dur->delta_days();
 
 427   my $is_sales = ref($self) eq 'SL::DB::Invoice';
 
 429   my $percent_skonto = 0;
 
 432     return undef unless ref $self->payment_terms;
 
 433     return undef unless $self->payment_terms->percent_skonto > 0;
 
 434     $percent_skonto = $self->payment_terms->percent_skonto;
 
 436     return undef unless ref $self->vendor->payment_terms;
 
 437     return undef unless $self->vendor->payment_terms->terms_skonto > 0;
 
 438     $percent_skonto = $self->vendor->payment_terms->percent_skonto;
 
 441   return $percent_skonto;
 
 444 sub amount_less_skonto {
 
 445   # amount that has to be paid if skonto applies, always return positive rounded values
 
 446   # the result is rounded so we can directly compare it with the user input
 
 449   my $is_sales = ref($self) eq 'SL::DB::Invoice';
 
 451   my $percent_skonto = $self->percent_skonto || 0;
 
 453   return _round($self->amount - ( $self->amount * $percent_skonto) );
 
 457 sub check_skonto_configuration {
 
 460   my $is_sales = ref($self) eq 'SL::DB::Invoice';
 
 462   my $skonto_configured = 1; # default is assume skonto works
 
 464   # my $transactions = $self->transactions;
 
 465   foreach my $transaction (@{ $self->transactions }) {
 
 466     # find all transactions with an AR_amount or AP_amount link
 
 467     my $tax = SL::DB::Manager::Tax->get_first( where => [taxkey => $transaction->taxkey]);
 
 468     croak "no tax for taxkey " . $transaction->{taxkey} unless ref $tax;
 
 470     $transaction->{chartlinks} = { map { $_ => 1 } split(m/:/, $transaction->chart_link) };
 
 471     if ( $is_sales && $transaction->{chartlinks}->{AR_amount} ) {
 
 472       $skonto_configured = 0 unless $tax->skonto_sales_chart_id;
 
 473     } elsif ( !$is_sales && $transaction->{chartlinks}->{AP_amount}) {
 
 474       $skonto_configured = 0 unless $tax->skonto_purchase_chart_id;
 
 478   return $skonto_configured;
 
 481 sub open_sepa_transfer_amount {
 
 484   my ($vc, $key, $type);
 
 485   if ( ref($self) eq 'SL::DB::Invoice' ) {
 
 495   my $sql = qq|SELECT SUM(sei.amount) AS amount FROM sepa_export_items sei | .
 
 496             qq| LEFT JOIN sepa_export se ON (sei.sepa_export_id = se.id)   | .
 
 497             qq| WHERE $key = ? AND NOT se.closed AND (se.vc = '$vc')       |;
 
 499   my ($open_sepa_amount) = $self->db->dbh->selectrow_array($sql, undef, $self->id);
 
 501   return $open_sepa_amount || 0;
 
 509   # TODO: use param for amount, may also want to calculate skonto_amounts by
 
 510   # passing percentage in the future
 
 512   my $amount = shift || $self->skonto_amount;
 
 514   croak "no amount passed to skonto_charts" unless abs(_round($amount)) >= 0.01;
 
 516   # TODO: check whether there are negative values in invoice / acc_trans ... credited items
 
 518   # don't check whether skonto applies, because user may want to override this
 
 519   # return undef unless $self->percent_skonto;  # for is_sales
 
 520   # return undef unless $self->vendor->payment_terms->percent_skonto;  # for purchase
 
 522   my $is_sales = ref($self) eq 'SL::DB::Invoice';
 
 524   my $mult = $is_sales ? 1 : -1;  # multiplier for getting the right sign
 
 526   my @skonto_charts;  # resulting array with all income/expense accounts that have to be corrected
 
 528   # calculate effective skonto (percentage) in difference_as_skonto mode
 
 529   # only works if there are no negative acc_trans values
 
 530   my $effective_skonto_rate = $amount ? $amount / $self->amount : 0;
 
 533   my $total_skonto_amount  = 0;
 
 534   my $total_rounding_error = 0;
 
 536   my $reference_ARAP_amount = 0;
 
 538   # my $transactions = $self->transactions;
 
 539   foreach my $transaction (@{ $self->transactions }) {
 
 540     # find all transactions with an AR_amount or AP_amount link
 
 541     $transaction->{chartlinks} = { map { $_ => 1 } split(m/:/, $transaction->{chart_link}) };
 
 542     # second condition is that we can determine an automatic Skonto account for each AR_amount entry
 
 544     if ( ( $is_sales && $transaction->{chartlinks}->{AR_amount} ) or ( !$is_sales && $transaction->{chartlinks}->{AP_amount}) ) {
 
 545         # $reference_ARAP_amount += $transaction->{amount} * $mult;
 
 547         # quick hack that works around problem of non-unique tax keys in SKR04
 
 548         my $tax = SL::DB::Manager::Tax->get_first( where => [taxkey => $transaction->{taxkey}]);
 
 549         croak "no tax for taxkey " . $transaction->{taxkey} unless ref $tax;
 
 552           die t8('no skonto_chart configured for taxkey #1 : #2 : #3', $transaction->{taxkey} , $tax->taxdescription , $tax->rate*100) unless ref $tax->skonto_sales_chart;
 
 554           die t8('no skonto_chart configured for taxkey #1 : #2 : #3', $transaction->{taxkey} , $tax->taxdescription , $tax->rate*100) unless ref $tax->skonto_purchase_chart;
 
 557         my $skonto_amount_unrounded;
 
 559         my $skonto_percent_abs = $self->amount ? abs($transaction->amount * (1 + $tax->rate) * 100 / $self->amount) : 0;
 
 561         my $transaction_amount = abs($transaction->{amount} * (1 + $tax->rate));
 
 562         my $transaction_skonto_percent = abs($transaction_amount/$self->amount); # abs($transaction->{amount} * (1 + $tax->rate));
 
 565         $skonto_amount_unrounded   = abs($amount * $transaction_skonto_percent);
 
 566         my $skonto_amount_rounded  = _round($skonto_amount_unrounded);
 
 567         my $rounding_error         = $skonto_amount_unrounded - $skonto_amount_rounded;
 
 568         my $rounded_rounding_error = _round($rounding_error);
 
 570         $total_rounding_error += $rounding_error;
 
 571         $total_skonto_amount  += $skonto_amount_rounded;
 
 574           # skonto_percent_abs: relative part of amount + tax to the total invoice amount
 
 575           'skonto_percent_abs'     => $skonto_percent_abs,
 
 576           'chart_id'               => $is_sales ? $tax->skonto_sales_chart->id : $tax->skonto_purchase_chart->id,
 
 577           'skonto_amount'          => $skonto_amount_rounded,
 
 578           # 'rounding_error'         => $rounding_error,
 
 579           # 'rounded_rounding_error' => $rounded_rounding_error,
 
 582         push @skonto_charts, $rec;
 
 586   # if the rounded sum of all rounding_errors reaches 0.01 this sum is
 
 587   # subtracted from the largest skonto_amount
 
 588   my $rounded_total_rounding_error = abs(_round($total_rounding_error));
 
 590   if ( $rounded_total_rounding_error > 0 ) {
 
 591     my $highest_amount_pos = 0;
 
 592     my $highest_amount = 0;
 
 594     foreach my $ref ( @skonto_charts ) {
 
 596       if ( $ref->{skonto_amount} > $highest_amount ) {
 
 597         $highest_amount     = $ref->{skonto_amount};
 
 598         $highest_amount_pos = $i;
 
 601     $skonto_charts[$i]->{skonto_amount} -= $rounded_total_rounding_error;
 
 604   return @skonto_charts;
 
 608 sub within_skonto_period {
 
 610   my $dateref = shift || DateTime->now->truncate( to => 'day' );
 
 612   return undef unless ref $dateref eq 'DateTime';
 
 613   return 0 unless $self->skonto_date;
 
 615   # return 1 if requested date (or today) is inside skonto period
 
 616   # this will also return 1 if date is before the invoice date
 
 617   return $dateref <= $self->skonto_date;
 
 620 sub valid_skonto_amount {
 
 622   my $amount = shift || 0;
 
 623   my $max_skonto_percent = 0.10;
 
 625   return 0 unless $amount > 0;
 
 627   # does this work for other currencies?
 
 628   return ($self->amount*$max_skonto_percent) > $amount;
 
 631 sub get_payment_select_options_for_bank_transaction {
 
 632   my ($self, $bt_id, %params) = @_;
 
 634   my $bt = SL::DB::Manager::BankTransaction->find_by( id => $bt_id );
 
 637   my $open_amount = $self->open_amount;
 
 638   #$main::lxdebug->message(LXDebug->DEBUG2(),"skonto_date=".$self->skonto_date." open amount=".$open_amount);
 
 640   if ( $open_amount &&                   # invoice amount not 0
 
 641        $self->skonto_date &&             # check whether skonto applies
 
 642        ( abs(abs($self->amount_less_skonto) - abs($bt->amount)) < 0.01 ||
 
 643         ( $bt->transaction_code eq "191" && abs($self->amount_less_skonto) < abs($bt->amount) )) &&
 
 644        $self->check_skonto_configuration) {
 
 645          if ( $self->within_skonto_period($bt->transdate) ) {
 
 646            push(@options, { payment_type => 'without_skonto', display => t8('without skonto') });
 
 647            push(@options, { payment_type => 'with_skonto_pt', display => t8('with skonto acc. to pt'), selected => 1 });
 
 649            push(@options, { payment_type => 'without_skonto', display => t8('without skonto') , selected => 1 });
 
 650            push(@options, { payment_type => 'with_skonto_pt', display => t8('with skonto acc. to pt')});
 
 661   return 1 if $self->currency_id == $::instance_conf->get_currency_id;
 
 663   die "transdate isn't a DateTime object:" . ref($self->transdate) unless ref($self->transdate) eq 'DateTime';
 
 664   my $rate = SL::DB::Manager::Exchangerate->find_by(currency_id => $self->currency_id,
 
 665                                                     transdate   => $self->transdate,
 
 667   return undef unless $rate;
 
 669   return $self->is_sales ? $rate->buy : $rate->sell; # also undef if not defined
 
 672 sub get_payment_suggestions {
 
 674   my ($self, %params) = @_;
 
 676   my $open_amount = $self->open_amount;
 
 677   $open_amount   -= $self->open_sepa_transfer_amount if $params{sepa};
 
 679   $self->{invoice_amount_suggestion} = $open_amount;
 
 680   undef $self->{payment_select_options};
 
 681   push(@{$self->{payment_select_options}} , { payment_type => 'without_skonto',  display => t8('without skonto') });
 
 682   if ( $self->within_skonto_period ) {
 
 683     # If there have been no payments yet suggest amount_less_skonto, otherwise the open amount
 
 684     if ( $open_amount &&                   # invoice amount not 0
 
 685          $open_amount == $self->amount &&  # no payments yet, or sum of payments and sepa export amounts is zero
 
 686          $self->check_skonto_configuration) {
 
 687       $self->{invoice_amount_suggestion} = $self->amount_less_skonto;
 
 688       push(@{$self->{payment_select_options}} , { payment_type => 'with_skonto_pt',  display => t8('with skonto acc. to pt') , selected => 1 });
 
 690       if ( ( $self->valid_skonto_amount($self->open_amount) || $self->valid_skonto_amount($open_amount) ) and not $params{sepa} ) {
 
 691         $self->{invoice_amount_suggestion} = $open_amount;
 
 692         # only suggest difference_as_skonto if open_amount exactly matches skonto_amount
 
 693         # AND we aren't in SEPA mode
 
 695         $selected = 1 if _round($open_amount) == _round($self->skonto_amount);
 
 696         push(@{$self->{payment_select_options}} , { payment_type => 'difference_as_skonto',  display => t8('difference as skonto') , selected => $selected });
 
 700     # invoice was configured with skonto, but skonto date has passed, or no skonto available
 
 701     $self->{invoice_amount_suggestion} = $open_amount;
 
 702     # difference_as_skonto doesn't make any sense for SEPA transfer, as this doesn't cause any actual payment
 
 703     if ( $self->valid_skonto_amount($self->open_amount) && not $params{sepa} ) {
 
 704       push(@{$self->{payment_select_options}} , { payment_type => 'difference_as_skonto',  display => t8('difference as skonto') , selected => 0 });
 
 710 # locales for payment type
 
 712 # $main::locale->text('without_skonto')
 
 713 # $main::locale->text('with_skonto_pt')
 
 714 # $main::locale->text('difference_as_skonto')
 
 717 sub validate_payment_type {
 
 718   my $payment_type = shift;
 
 720   my %allowed_payment_types = map { $_ => 1 } qw(without_skonto with_skonto_pt difference_as_skonto);
 
 721   croak "illegal payment type: $payment_type, must be one of: " . join(' ', keys %allowed_payment_types) unless $allowed_payment_types{ $payment_type };
 
 726 sub create_bank_transaction {
 
 727   my ($self, %params) = @_;
 
 729   require SL::DB::Chart;
 
 730   require SL::DB::BankAccount;
 
 733   if ( $params{chart_id} ) {
 
 734     $bank_chart = SL::DB::Manager::Chart->find_by(chart_id => $params{chart_id}) or die "Can't find bank chart";
 
 735   } elsif ( $::instance_conf->get_ar_paid_accno_id ) {
 
 736     $bank_chart   = SL::DB::Manager::Chart->find_by(id => $::instance_conf->get_ar_paid_accno_id);
 
 738     $bank_chart = SL::DB::Manager::Chart->find_by(description => 'Bank') or die "Can't find bank chart";
 
 740   my $bank_account = SL::DB::Manager::BankAccount->find_by(chart_id => $bank_chart->id) or die "Can't find bank account for chart";
 
 742   my $multiplier = $self->is_sales ? 1 : -1;
 
 743   my $amount = ($params{amount} || $self->amount) * $multiplier;
 
 745   my $transdate = $params{transdate} || DateTime->today;
 
 747   my $bt = SL::DB::BankTransaction->new(
 
 748     local_bank_account_id => $bank_account->id,
 
 749     remote_bank_code      => $self->customervendor->bank_code,
 
 750     remote_account_number => $self->customervendor->account_number,
 
 751     transdate             => $transdate,
 
 752     valutadate            => $transdate,
 
 753     amount                => $::form->round_amount($amount, 2),
 
 754     currency              => $self->currency->id,
 
 755     remote_name           => $self->customervendor->depositor,
 
 756     purpose               => $params{purpose} || $self->invnumber
 
 763   $self->currency_id == $::instance_conf->get_currency_id ? return 0 : return 1;
 
 769   return $::form->round_amount($value, 2);
 
 780 SL::DB::Helper::Payment  Mixin providing helper methods for paying C<Invoice>
 
 781                          and C<PurchaseInvoice> objects and using skonto
 
 785 In addition to actually causing a payment via pay_invoice this helper contains
 
 786 many methods that help in determining information about the status of the
 
 787 invoice, such as the remaining open amount, whether skonto applies, until which
 
 788 date skonto applies, the skonto amount and relative percentages, what to do
 
 791 To prevent duplicate code this was all added in this mixin rather than directly
 
 792 in SL::DB::Invoice and SL::DB::PurchaseInvoice.
 
 796 =item C<pay_invoice %params>
 
 798 Create a payment booking for an existing invoice object (type ar/ap/is/ir) via
 
 799 a configured bank account.
 
 801 This function deals with all the acc_trans entries and also updates paid and datepaid.
 
 805   my $ap   = SL::DB::Manager::PurchaseInvoice->find_by( invnumber => '1');
 
 806   my $bank = SL::DB::Manager::BankAccount->find_by( name => 'Bank');
 
 807   $ap->pay_invoice(chart_id      => $bank->chart_id,
 
 808                    amount        => $ap->open_amount,
 
 809                    transdate     => DateTime->now->to_kivitendo,
 
 812                    payment_type  => 'without_skonto',  # default if not specified
 
 817   $ap->pay_invoice(chart_id      => $bank->chart_id,
 
 818                    amount        => $ap->amount,       # doesn't need to be specified
 
 819                    transdate     => DateTime->now->to_kivitendo,
 
 822                    payment_type  => 'with_skonto',
 
 825 or in a certain currency:
 
 826   $ap->pay_invoice(chart_id      => $bank->chart_id,
 
 829                    transdate     => DateTime->now->to_kivitendo,
 
 832                    payment_type  => 'with_skonto',
 
 835 Allowed payment types are:
 
 836   without_skonto with_skonto_pt difference_as_skonto
 
 838 The option C<payment_type> allows for a basic skonto mechanism.
 
 840 C<without_skonto> is the default mode, "amount" is paid to the account in
 
 841 chart_id. This can also be used for partial payments and corrections via
 
 844 C<with_skonto_pt> can't be used for partial payments. When used on unpaid
 
 845 invoices the whole amount is paid, with the skonto part automatically being
 
 846 booked according to the skonto chart configured in the tax settings for each
 
 847 tax key. If an amount is passed it is ignored and the actual configured skonto
 
 850 C<difference_as_skonto> can only be used after partial payments have been made,
 
 851 the whole specified amount is booked according to the skonto charts configured
 
 852 in the tax settings for each tax key.
 
 854 So passing amount doesn't have any effect for the cases C<with_skonto_pt> and
 
 855 C<difference_as_skonto>, as all necessary values are taken from the stored
 
 858 The skonto modes automatically calculate the relative amounts for a mix of
 
 859 taxes, e.g. items with 7% and 19% in one invoice. There is a helper method
 
 860 skonto_charts, which calculates the relative percentages according to the
 
 861 amounts in acc_trans (which are grouped by tax).
 
 863 There is currently no way of excluding certain items in an invoice from having
 
 864 skonto applied to them.  If this feature was added to parts the calculation
 
 865 method of relative skonto would have to be completely rewritten using the
 
 866 invoice items rather than acc_trans.
 
 868 The skonto modes also still don't automatically correct the tax, this still has
 
 869 to be done manually. Therefore all payments generated by pay_invoice have
 
 872 There is currently no way to directly pay an invoice via this method if the
 
 873 effective skonto differs from the skonto according to the payment terms
 
 874 configured for the invoice/vendor.
 
 876 In this case one has to pay in two steps: first the actual paid amount via
 
 877 "without skonto", and then the remainder via "difference_as_skonto". The user
 
 878 has to there actively decide whether to accept the differing skonto.
 
 880 Because of the way skonto_charts works the calculation doesn't work if there
 
 881 are negative values in acc_trans. E.g. one invoice with a positive value for
 
 882 19% tax and a negative value for the acc_trans line with 7%
 
 884 Skonto doesn't/shouldn't apply if the invoice contains credited items.
 
 886 If no amount is given the whole open amout is paid.
 
 888 If neither currency or currency_id are given as params, the currency of the
 
 889 invoice is assumed to be the payment currency.
 
 891 =item C<reference_account>
 
 893 Returns a chart object which is the chart of the invoice with link AR or AP.
 
 895 Example (1200 is the AR account for SKR04):
 
 896   my $invoice = invoice(invnumber => '144');
 
 897   $invoice->reference_account->accno
 
 900 =item C<percent_skonto>
 
 902 Returns the configured skonto percentage of the payment terms of an invoice,
 
 903 e.g. 0.02 for 2%. Payment terms come from invoice settings for ar, from vendor
 
 906 =item C<amount_less_skonto>
 
 908 If the invoice has a payment term (via ar for sales, via vendor for purchase),
 
 909 calculate the amount to be paid in the case of skonto.  This doesn't check,
 
 910 whether skonto applies (i.e. skonto doesn't wasn't exceeded), it just subtracts
 
 911 the configured percentage (e.g. 2%) from the total amount.
 
 913 The returned value is rounded to two decimals.
 
 917 The date up to which skonto may be taken. This is calculated from the invoice
 
 918 date + the number of days configured in the payment terms.
 
 920 This method can also be used to determine whether skonto applies for the
 
 921 invoice, as it returns undef if there is no payment term or skonto days is set
 
 924 =item C<within_skonto_period [DATE]>
 
 928 Checks whether the invoice has payment terms configured, and whether the date
 
 929 is within the skonto max date. If no date is passed the current date is used.
 
 931 You can also pass a dateref object as a parameter to check whether skonto
 
 932 applies for that date rather than the current date.
 
 934 =item C<valid_skonto_amount>
 
 936 Takes an amount as an argument and checks whether the amount is less than 10%
 
 937 of the total amount of the invoice. The value of 10% is currently hardcoded in
 
 938 the method. This method is currently used to check whether to offer the payment
 
 939 option "difference as skonto".
 
 942  if ( $invoice->valid_skonto_amount($invoice->open_amount) ) {
 
 946 =item C<skonto_charts [$amount]>
 
 948 Returns a list of chart_ids and some calculated numbers that can be used for
 
 949 paying the invoice with skonto. This function will automatically calculate the
 
 950 relative skonto amounts even if the invoice contains several types of taxes
 
 954   my $invoice = SL::DB::Manager::Invoice->find_by(invnumber => '211');
 
 955   my @skonto_charts = $invoice->skonto_charts;
 
 957 or with the total skonto amount as an argument:
 
 958   my @skonto_charts = $invoice->skonto_charts($invoice->open_amount);
 
 960 The following values are generated for each chart:
 
 966 The chart id of the skonto amount to be booked.
 
 968 =item C<skonto_amount>
 
 970 The total amount to be paid to the account
 
 972 =item C<skonto_percent>
 
 974 The relative percentage of that skonto chart. This can be useful if the actual
 
 975 ekonto that is paid deviates from the granted skonto, e.g. customer effectively
 
 976 pays 2.6% skonto instead of 2%, and we accept this. Then we can still calculate
 
 977 the relative skonto amounts for different taxes based on the absolute
 
 978 percentages. Used for case C<difference_as_skonto>.
 
 980 =item C<skonto_percent_abs>
 
 982 The absolute percentage of that skonto chart in relation to the total amount.
 
 983 Used to calculate skonto_amount for case C<with_skonto_pt>.
 
 987 If the invoice contains several types of taxes then skonto_charts can be used
 
 988 to calculate the relative amounts.
 
 990 Example in console of an invoice with 100 Euro at 7% and 100 Euro at 19% with
 
 993   my $invoice = invoice(invnumber => '144');
 
 996   $invoice->payment_terms->percent_skonto
 
 998   $invoice->skonto_charts
 
 999   pp $invoice->skonto_charts
 
1001   #               'chart_id'       => 128,
 
1002   #               'skonto_amount'  => '2.14',
 
1003   #               'skonto_percent' => '47.3451327433627'
 
1006   #               'chart_id'       => 130,
 
1007   #               'skonto_amount'  => '2.38',
 
1008   #               'skonto_percent' => '52.654867256637'
 
1011 C<skonto_charts> always returns positive values (abs) for C<skonto_amount> and
 
1014 C<skonto_charts> generates one entry for each acc_trans entry. ar and ap
 
1015 bookings only have one acc_trans entry for each taxkey (e.g. 7% and 19%).  This
 
1016 is because all the items are grouped according to the Buchungsgruppen mechanism
 
1017 and the totals are written to acc_trans.  For is and ir it is possible to have
 
1018 several acc_trans entries with the same tax. In this case skonto_charts
 
1019 generates a skonto booking for each acc_trans income/expense entry.
 
1021 In the future this function may also be used to calculate the corrections for
 
1024 =item C<open_amount>
 
1026 Unrounded total open amount of invoice (amount - paid).
 
1027 Doesn't take into account pending SEPA transfers.
 
1029 =item C<open_percent>
 
1031 Percentage of the invoice that is still unpaid, e.g. 100,00 if no payments have
 
1032 been made yet, 0,00 if fully paid.
 
1034 =item C<remaining_skonto_days>
 
1036 How many days skonto can still be taken, calculated from current day. Returns 0
 
1037 if current day is the max skonto date, and negative number if skonto date has
 
1040 Returns undef if skonto is not configured for that invoice.
 
1042 =item C<get_payment_suggestions %params>
 
1044 Creates data intended for an L.select_tag dropdown that can be used in a
 
1045 template. Depending on the rules it will choose from the options
 
1046 without_skonto, with_skonto_pt and difference_as_skonto, and select the most
 
1049 If the parameter "sepa" is passed, the SEPA export payments that haven't been
 
1050 executed yet are considered when determining the open amount of the invoice.
 
1052 The current rules are:
 
1056 =item * without_skonto is always an option
 
1058 =item * with_skonto_pt is only offered if there haven't been any payments yet and the current date is within the skonto period.
 
1060 =item * difference_as_skonto is only offered if there have already been payments made and the open amount is smaller than 10% of the total amount.
 
1062 with_skonto_pt will only be offered, if all the AR_amount/AP_amount have a
 
1063 taxkey with a configured skonto chart
 
1067 It will also fill $self->{invoice_amount_suggestion} with either the open
 
1068 amount, or if with_skonto_pt is selected, with amount_less_skonto, so the
 
1069 template can fill the input with the likely amount.
 
1072   my $ar = invoice( invnumber => '257');
 
1073   $ar->get_payment_suggestions;
 
1074   print $ar->{invoice_amount_suggestion} . "\n";
 
1076   pp $ar->{payment_select_options}
 
1079   #           'display' => 'ohne Skonto',
 
1080   #           'payment_type' => 'without_skonto'
 
1083   #           'display' => 'mit Skonto nach ZB',
 
1084   #           'payment_type' => 'with_skonto_pt',
 
1089 The resulting array $ar->{payment_select_options} can be used in a template
 
1090 select_tag using value_key and title_key:
 
1092 [% L.select_tag('payment_type_' _ loop.count, invoice.payment_select_options, value_key => 'payment_type', title_key => 'display', id => 'payment_type_' _ loop.count) %]
 
1094 It would probably make sense to have different rules for the pre-selected items
 
1095 for sales and purchase, and to also make these rules configurable in the
 
1096 defaults. E.g. when creating a SEPA bank transfer for vendor invoices a company
 
1097 might always want to pay quickly making use of skonto, while another company
 
1098 might always want to pay as late as possible.
 
1100 =item C<get_payment_select_options_for_bank_transaction $banktransaction_id %params>
 
1102 Make suggestion for a skonto payment type by returning an HTML blob of the options
 
1103 of a HTML drop-down select with the most likely option preselected.
 
1105 This is a helper function for BankTransaction/ajax_payment_suggestion.
 
1107 We are working with an existing payment, so difference_as_skonto never makes sense.
 
1109 If skonto is possible (skonto_date exists), add two possibilities:
 
1110 without_skonto and with_skonto_pt if payment date is within skonto_date,
 
1111 preselect with_skonto_pt, otherwise preselect without skonto.
 
1113 =item C<create_bank_transaction %params>
 
1115 Method used for testing purposes, allows you to quickly create bank
 
1116 transactions from invoices to have something to test payments against.
 
1118  my $ap = SL::DB::Manager::Invoice->find_by(id => 41);
 
1119  $ap->create_bank_transaction(amount => $ap->amount/2, transdate => DateTime->today->add(days => 5));
 
1121 To create a payment for 3 invoices that were all paid together, all with skonto:
 
1122  my $ar1 = SL::DB::Manager::Invoice->find_by(invnumber=>'20');
 
1123  my $ar2 = SL::DB::Manager::Invoice->find_by(invnumber=>'21');
 
1124  my $ar3 = SL::DB::Manager::Invoice->find_by(invnumber=>'22');
 
1125  $ar1->create_bank_transaction(amount  => ($ar1->amount_less_skonto + $ar2->amount_less_skonto + $ar2->amount_less_skonto),
 
1126                                purpose => 'Rechnungen 20, 21, 22',
 
1129 Amount is always relative to the absolute amount of the invoice, use positive
 
1130 values for sales and purchases.
 
1132 The following params can be passed to override the defaults:
 
1140 =item * chart_id (the chart the amount is to be paid to)
 
1146 =item C<exchangerate>
 
1148 Returns 1 immediately if the record uses the default currency.
 
1150 Returns the exchangerate in database format for the invoice according to that
 
1151 invoice's transdate, returning 'buy' for sales, 'sell' for purchases.
 
1153 If no exchangerate can be found for that day undef is returned.
 
1157 Returns 1 if record uses a different currency, 0 if the default currency is used.
 
1161 =head1 TODO AND CAVEATS
 
1167 when looking at open amount, maybe consider that there may already be queued
 
1168 amounts in SEPA Export
 
1174 G. Richardson E<lt>grichardson@kivitendo-premium.de<gt>