1 package SL::Controller::BankTransaction;
3 # idee- möglichkeit bankdaten zu übernehmen in stammdaten
4 # erst Kontenabgleich, um alle gl-Einträge wegzuhaben
7 use parent qw(SL::Controller::Base);
9 use SL::Controller::Helper::GetModels;
10 use SL::Controller::Helper::ReportGenerator;
11 use SL::ReportGenerator;
13 use SL::DB::BankTransaction;
14 use SL::Helper::Flash;
15 use SL::Locale::String;
18 use SL::DB::PurchaseInvoice;
19 use SL::DB::RecordLink;
22 use SL::DB::AccTransaction;
23 use SL::DB::BankTransactionAccTrans;
25 use SL::DB::BankAccount;
26 use SL::DB::RecordTemplate;
27 use SL::DB::SepaExportItem;
28 use SL::DBUtils qw(like);
30 use List::UtilsBy qw(partition_by);
31 use List::MoreUtils qw(any);
32 use List::Util qw(max);
34 use Rose::Object::MakeMethods::Generic
36 scalar => [ qw(callback transaction) ],
37 'scalar --get_set_init' => [ qw(models problems) ],
40 __PACKAGE__->run_before('check_auth');
50 my $bank_accounts = SL::DB::Manager::BankAccount->get_all_sorted( query => [ obsolete => 0 ] );
52 $self->setup_search_action_bar;
53 $self->render('bank_transactions/search',
54 BANK_ACCOUNTS => $bank_accounts);
60 $self->make_filter_summary;
61 $self->prepare_report;
63 $self->setup_list_all_action_bar;
64 $self->report_generator_list_objects(report => $self->{report}, objects => $self->models->get);
70 if (!$::form->{filter}{bank_account}) {
71 flash('error', t8('No bank account chosen!'));
76 my $sort_by = $::form->{sort_by} || 'transdate';
77 $sort_by = 'transdate' if $sort_by eq 'proposal';
78 $sort_by .= $::form->{sort_dir} ? ' DESC' : ' ASC';
80 my $fromdate = $::locale->parse_date_to_object($::form->{filter}->{fromdate});
81 my $todate = $::locale->parse_date_to_object($::form->{filter}->{todate});
82 $todate->add( days => 1 ) if $todate;
85 push @where, (transdate => { ge => $fromdate }) if ($fromdate);
86 push @where, (transdate => { lt => $todate }) if ($todate);
87 my $bank_account = SL::DB::Manager::BankAccount->find_by( id => $::form->{filter}{bank_account} );
88 # bank_transactions no younger than starting date,
89 # including starting date (same search behaviour as fromdate)
90 # but OPEN invoices to be matched may be from before
91 if ( $bank_account->reconciliation_starting_date ) {
92 push @where, (transdate => { ge => $bank_account->reconciliation_starting_date });
95 my $bank_transactions = SL::DB::Manager::BankTransaction->get_all(
96 with_objects => [ 'local_bank_account', 'currency' ],
100 amount => {ne => \'invoice_amount'},
101 local_bank_account_id => $::form->{filter}{bank_account},
106 # credit notes have a negative amount, treat differently
107 my $all_open_ar_invoices = SL::DB::Manager::Invoice ->get_all(where => [ or => [ amount => { gt => \'paid' },
108 and => [ type => 'credit_note',
109 amount => { lt => \'paid' }
113 with_objects => ['customer','payment_terms']);
115 my $all_open_ap_invoices = SL::DB::Manager::PurchaseInvoice->get_all(where => [amount => { ne => \'paid' }], with_objects => ['vendor' ,'payment_terms']);
116 my $all_open_sepa_export_items = SL::DB::Manager::SepaExportItem->get_all(where => [chart_id => $bank_account->chart_id ,
117 'sepa_export.executed' => 0, 'sepa_export.closed' => 0 ], with_objects => ['sepa_export']);
119 my @all_open_invoices;
120 # filter out invoices with less than 1 cent outstanding
121 push @all_open_invoices, map { $_->{is_ar}=1 ; $_ } grep { abs($_->amount - $_->paid) >= 0.01 } @{ $all_open_ar_invoices };
122 push @all_open_invoices, map { $_->{is_ar}=0 ; $_ } grep { abs($_->amount - $_->paid) >= 0.01 } @{ $all_open_ap_invoices };
125 my %sepa_export_items_by_id = partition_by { $_->ar_id || $_->ap_id } @$all_open_sepa_export_items;
127 # first collect sepa export items to open invoices
128 foreach my $open_invoice (@all_open_invoices){
129 $open_invoice->{realamount} = $::form->format_amount(\%::myconfig,$open_invoice->amount,2);
130 $open_invoice->{skonto_type} = 'without_skonto';
131 foreach (@{ $sepa_export_items_by_id{ $open_invoice->id } || [] }) {
132 my $factor = ($_->ar_id == $open_invoice->id ? 1 : -1);
133 $open_invoice->{realamount} = $::form->format_amount(\%::myconfig,$open_invoice->amount*$factor,2);
135 $open_invoice->{skonto_type} = $_->payment_type;
136 $sepa_exports{$_->sepa_export_id} ||= { count => 0, is_ar => 0, amount => 0, proposed => 0, invoices => [], item => $_ };
137 $sepa_exports{$_->sepa_export_id}->{count}++;
138 $sepa_exports{$_->sepa_export_id}->{is_ar}++ if $_->ar_id == $open_invoice->id;
139 $sepa_exports{$_->sepa_export_id}->{amount} += $_->amount * $factor;
140 push @{ $sepa_exports{$_->sepa_export_id}->{invoices} }, $open_invoice;
144 # try to match each bank_transaction with each of the possible open invoices
148 foreach my $bt (@{ $bank_transactions }) {
149 ## 5 Stellen hinter dem Komma auf 2 Stellen reduzieren
150 $bt->amount($bt->amount*1);
151 $bt->invoice_amount($bt->invoice_amount*1);
153 $bt->{proposals} = [];
154 $bt->{rule_matches} = [];
156 $bt->{remote_name} .= $bt->{remote_name_1} if $bt->{remote_name_1};
158 if ( $bt->is_batch_transaction ) {
160 foreach ( keys %sepa_exports) {
161 if ( abs(($sepa_exports{$_}->{amount} * 1) - ($bt->amount * 1)) < 0.01 ) {
163 @{$bt->{proposals}} = @{$sepa_exports{$_}->{invoices}};
164 $bt->{sepa_export_ok} = 1;
165 $sepa_exports{$_}->{proposed}=1;
166 push(@proposals, $bt);
172 # batch transaction has no remotename !!
174 next unless $bt->{remote_name}; # bank has no name, usually fees, use create invoice to assign
177 # try to match the current $bt to each of the open_invoices, saving the
178 # results of get_agreement_with_invoice in $open_invoice->{agreement} and
179 # $open_invoice->{rule_matches}.
181 # The values are overwritten each time a new bt is checked, so at the end
182 # of each bt the likely results are filtered and those values are stored in
183 # the arrays $bt->{proposals} and $bt->{rule_matches}, and the agreement
184 # score is stored in $bt->{agreement}
186 foreach my $open_invoice (@all_open_invoices) {
187 ($open_invoice->{agreement}, $open_invoice->{rule_matches}) = $bt->get_agreement_with_invoice($open_invoice,
188 sepa_export_items => $all_open_sepa_export_items,
190 $open_invoice->{realamount} = $::form->format_amount(\%::myconfig,
191 $open_invoice->amount * ($open_invoice->{is_ar} ? 1 : -1), 2);
195 my $min_agreement = 3; # suggestions must have at least this score
197 my $max_agreement = max map { $_->{agreement} } @all_open_invoices;
199 # add open_invoices with highest agreement into array $bt->{proposals}
200 if ( $max_agreement >= $min_agreement ) {
201 $bt->{proposals} = [ grep { $_->{agreement} == $max_agreement } @all_open_invoices ];
202 $bt->{agreement} = $max_agreement; #scalar @{ $bt->{proposals} } ? $agreement + 1 : '';
204 # store the rule_matches in a separate array, so they can be displayed in template
205 foreach ( @{ $bt->{proposals} } ) {
206 push(@{$bt->{rule_matches}}, $_->{rule_matches});
212 # separate filter for proposals (second tab, agreement >= 5 and exactly one match)
213 # to qualify as a proposal there has to be
214 # * agreement >= 5 TODO: make threshold configurable in configuration
215 # * there must be only one exact match
216 # * depending on whether sales or purchase the amount has to have the correct sign (so Gutschriften don't work?)
217 my $proposal_threshold = 5;
218 my @otherproposals = grep {
219 ($_->{agreement} >= $proposal_threshold)
220 && (1 == scalar @{ $_->{proposals} })
221 && (@{ $_->{proposals} }[0]->is_sales ? abs(@{ $_->{proposals} }[0]->amount - $_->amount) < 0.01
222 : abs(@{ $_->{proposals} }[0]->amount + $_->amount) < 0.01)
223 } @{ $bank_transactions };
225 push @proposals, @otherproposals;
227 # sort bank transaction proposals by quality (score) of proposal
228 if ($::form->{sort_by} && $::form->{sort_by} eq 'proposal') {
229 if ($::form->{sort_dir}) {
230 $bank_transactions = [ sort { $a->{agreement} <=> $b->{agreement} } @{ $bank_transactions } ];
232 $bank_transactions = [ sort { $b->{agreement} <=> $a->{agreement} } @{ $bank_transactions } ];
236 # for testing with t/bank/banktransaction.t :
237 if ( $::form->{dont_render_for_test} ) {
238 return ( $bank_transactions , \@proposals );
241 $::request->layout->add_javascripts("kivi.BankTransaction.js");
242 $self->render('bank_transactions/list',
243 title => t8('Bank transactions MT940'),
244 BANK_TRANSACTIONS => $bank_transactions,
245 PROPOSALS => \@proposals,
246 bank_account => $bank_account,
247 ui_tab => scalar(@proposals) > 0?1:0,
251 sub action_assign_invoice {
254 $self->{transaction} = SL::DB::Manager::BankTransaction->find_by(id => $::form->{bt_id});
256 $self->render('bank_transactions/assign_invoice',
258 title => t8('Assign invoice'),);
261 sub action_create_invoice {
263 my %myconfig = %main::myconfig;
265 $self->transaction(SL::DB::Manager::BankTransaction->find_by(id => $::form->{bt_id}));
267 my $vendor_of_transaction = SL::DB::Manager::Vendor->find_by(iban => $self->transaction->{remote_account_number});
268 my $use_vendor_filter = $self->transaction->{remote_account_number} && $vendor_of_transaction;
270 my $templates_ap = SL::DB::Manager::RecordTemplate->get_all(
271 where => [ template_type => 'ap_transaction' ],
272 with_objects => [ qw(employee vendor) ],
274 my $templates_gl = SL::DB::Manager::RecordTemplate->get_all(
275 query => [ template_type => 'gl_transaction',
276 chart_id => SL::DB::Manager::BankAccount->find_by(id => $self->transaction->local_bank_account_id)->chart_id,
278 with_objects => [ qw(employee record_template_items) ],
281 # pre filter templates_ap, if we have a vendor match (IBAN eq IBAN) - show and allow user to edit this via gui!
282 $templates_ap = [ grep { $_->vendor_id == $vendor_of_transaction->id } @{ $templates_ap } ] if $use_vendor_filter;
284 $self->callback($self->url_for(
286 'filter.bank_account' => $::form->{filter}->{bank_account},
287 'filter.todate' => $::form->{filter}->{todate},
288 'filter.fromdate' => $::form->{filter}->{fromdate},
292 'bank_transactions/create_invoice',
294 title => t8('Create invoice'),
295 TEMPLATES_GL => $use_vendor_filter && @{ $templates_ap } ? undef : $templates_gl,
296 TEMPLATES_AP => $templates_ap,
297 vendor_name => $use_vendor_filter && @{ $templates_ap } ? $vendor_of_transaction->name : undef,
301 sub action_ajax_payment_suggestion {
304 # based on a BankTransaction ID and a Invoice or PurchaseInvoice ID passed via $::form,
305 # create an HTML blob to be used by the js function add_invoices in templates/webpages/bank_transactions/list.html
306 # and return encoded as JSON
308 my $bt = SL::DB::Manager::BankTransaction->find_by( id => $::form->{bt_id} );
309 my $invoice = SL::DB::Manager::Invoice->find_by( id => $::form->{prop_id} ) || SL::DB::Manager::PurchaseInvoice->find_by( id => $::form->{prop_id} );
311 die unless $bt and $invoice;
313 my @select_options = $invoice->get_payment_select_options_for_bank_transaction($::form->{bt_id});
316 $html = $self->render(
317 'bank_transactions/_payment_suggestion', { output => 0 },
318 bt_id => $::form->{bt_id},
319 prop_id => $::form->{prop_id},
321 SELECT_OPTIONS => \@select_options,
324 $self->render(\ SL::JSON::to_json( { 'html' => "$html" } ), { layout => 0, type => 'json', process => 0 });
327 sub action_filter_templates {
330 $self->{transaction} = SL::DB::Manager::BankTransaction->find_by(id => $::form->{bt_id});
332 my (@filter, @filter_ap);
334 # filter => gl and ap | filter_ap = ap (i.e. vendorname)
335 push @filter, ('template_name' => { ilike => '%' . $::form->{template} . '%' }) if $::form->{template};
336 push @filter, ('reference' => { ilike => '%' . $::form->{reference} . '%' }) if $::form->{reference};
337 push @filter_ap, ('vendor.name' => { ilike => '%' . $::form->{vendor} . '%' }) if $::form->{vendor};
338 push @filter_ap, @filter;
339 my $templates_gl = SL::DB::Manager::RecordTemplate->get_all(
340 query => [ template_type => 'gl_transaction',
341 chart_id => SL::DB::Manager::BankAccount->find_by(id => $self->transaction->local_bank_account_id)->chart_id,
342 (and => \@filter) x !!@filter
344 with_objects => [ qw(employee record_template_items) ],
347 my $templates_ap = SL::DB::Manager::RecordTemplate->get_all(
348 where => [ template_type => 'ap_transaction', (and => \@filter_ap) x !!@filter_ap ],
349 with_objects => [ qw(employee vendor) ],
351 $::form->{filter} //= {};
353 $self->callback($self->url_for(
355 'filter.bank_account' => $::form->{filter}->{bank_account},
356 'filter.todate' => $::form->{filter}->{todate},
357 'filter.fromdate' => $::form->{filter}->{fromdate},
360 my $output = $self->render(
361 'bank_transactions/_template_list',
363 TEMPLATES_AP => $templates_ap,
364 TEMPLATES_GL => $templates_gl,
367 $self->render(\to_json({ html => $output }), { type => 'json', process => 0 });
370 sub action_ajax_add_list {
373 my @where_sale = (amount => { ne => \'paid' });
374 my @where_purchase = (amount => { ne => \'paid' });
376 if ($::form->{invnumber}) {
377 push @where_sale, (invnumber => { ilike => like($::form->{invnumber})});
378 push @where_purchase, (invnumber => { ilike => like($::form->{invnumber})});
381 if ($::form->{amount}) {
382 push @where_sale, (amount => $::form->parse_amount(\%::myconfig, $::form->{amount}));
383 push @where_purchase, (amount => $::form->parse_amount(\%::myconfig, $::form->{amount}));
386 if ($::form->{vcnumber}) {
387 push @where_sale, ('customer.customernumber' => { ilike => like($::form->{vcnumber})});
388 push @where_purchase, ('vendor.vendornumber' => { ilike => like($::form->{vcnumber})});
391 if ($::form->{vcname}) {
392 push @where_sale, ('customer.name' => { ilike => like($::form->{vcname})});
393 push @where_purchase, ('vendor.name' => { ilike => like($::form->{vcname})});
396 if ($::form->{transdatefrom}) {
397 my $fromdate = $::locale->parse_date_to_object($::form->{transdatefrom});
398 if ( ref($fromdate) eq 'DateTime' ) {
399 push @where_sale, ('transdate' => { ge => $fromdate});
400 push @where_purchase, ('transdate' => { ge => $fromdate});
404 if ($::form->{transdateto}) {
405 my $todate = $::locale->parse_date_to_object($::form->{transdateto});
406 if ( ref($todate) eq 'DateTime' ) {
407 $todate->add(days => 1);
408 push @where_sale, ('transdate' => { lt => $todate});
409 push @where_purchase, ('transdate' => { lt => $todate});
413 my $all_open_ar_invoices = SL::DB::Manager::Invoice ->get_all(where => \@where_sale, with_objects => 'customer');
414 my $all_open_ap_invoices = SL::DB::Manager::PurchaseInvoice->get_all(where => \@where_purchase, with_objects => 'vendor');
416 my @all_open_invoices = @{ $all_open_ar_invoices };
417 # add ap invoices, filtering out subcent open amounts
418 push @all_open_invoices, grep { abs($_->amount - $_->paid) >= 0.01 } @{ $all_open_ap_invoices };
420 @all_open_invoices = sort { $a->id <=> $b->id } @all_open_invoices;
422 my $output = $self->render(
423 'bank_transactions/add_list',
425 INVOICES => \@all_open_invoices,
428 my %result = ( count => 0, html => $output );
430 $self->render(\to_json(\%result), { type => 'json', process => 0 });
433 sub action_ajax_accept_invoices {
436 my @selected_invoices;
437 foreach my $invoice_id (@{ $::form->{invoice_id} || [] }) {
438 my $invoice_object = SL::DB::Manager::Invoice->find_by(id => $invoice_id) || SL::DB::Manager::PurchaseInvoice->find_by(id => $invoice_id);
439 push @selected_invoices, $invoice_object;
443 'bank_transactions/invoices',
445 INVOICES => \@selected_invoices,
446 bt_id => $::form->{bt_id},
453 return 0 if !$::form->{invoice_ids};
455 my %invoice_hash = %{ delete $::form->{invoice_ids} }; # each key (the bt line with a bt_id) contains an array of invoice_ids
457 # e.g. three partial payments with bt_ids 54, 55 and 56 for invoice with id 74:
470 # or if the payment with bt_id 44 is used to pay invoices with ids 50, 51 and 52
472 # '44' => [ '50', '51', 52' ]
475 $::form->{invoice_skontos} ||= {}; # hash of arrays containing the payment types, could be empty
477 # a bank_transaction may be assigned to several invoices, i.e. a customer
478 # might pay several open invoices with one transaction
484 if ( $::form->{proposal_ids} ) {
485 foreach (@{ $::form->{proposal_ids} }) {
486 my $bank_transaction_id = $_;
487 my $invoice_ids = $invoice_hash{$_};
488 push @{ $self->problems }, $self->save_single_bank_transaction(
489 bank_transaction_id => $bank_transaction_id,
490 invoice_ids => $invoice_ids,
491 sources => ($::form->{sources} // {})->{$_},
492 memos => ($::form->{memos} // {})->{$_},
494 $count += scalar( @{$invoice_ids} );
497 while ( my ($bank_transaction_id, $invoice_ids) = each(%invoice_hash) ) {
498 push @{ $self->problems }, $self->save_single_bank_transaction(
499 bank_transaction_id => $bank_transaction_id,
500 invoice_ids => $invoice_ids,
501 sources => [ map { $::form->{"sources_${bank_transaction_id}_${_}"} } @{ $invoice_ids } ],
502 memos => [ map { $::form->{"memos_${bank_transaction_id}_${_}"} } @{ $invoice_ids } ],
504 $count += scalar( @{$invoice_ids} );
507 my $max_count = $count;
508 foreach (@{ $self->problems }) {
509 $count-- if $_->{result} eq 'error';
511 return ($count, $max_count);
514 sub action_save_invoices {
516 my ($success_count, $max_count) = $self->save_invoices();
518 if ($success_count == $max_count) {
519 flash('ok', t8('#1 invoice(s) saved.', $success_count));
521 flash('error', t8('At least #1 invoice(s) not saved', $max_count - $success_count));
524 $self->action_list();
527 sub action_save_proposals {
530 if ( $::form->{proposal_ids} ) {
531 my $propcount = scalar(@{ $::form->{proposal_ids} });
532 if ( $propcount > 0 ) {
533 my $count = $self->save_invoices();
535 flash('ok', t8('#1 proposal(s) with #2 invoice(s) saved.', $propcount, $count));
538 $self->action_list();
542 sub save_single_bank_transaction {
543 my ($self, %params) = @_;
547 bank_transaction => SL::DB::Manager::BankTransaction->find_by(id => $params{bank_transaction_id}),
551 if (!$data{bank_transaction}) {
555 message => $::locale->text('The ID #1 is not a valid database ID.', $data{bank_transaction_id}),
559 my $bank_transaction = $data{bank_transaction};
564 my $bt_id = $data{bank_transaction_id};
565 my $sign = $bank_transaction->amount < 0 ? -1 : 1;
566 my $not_assigned_amount = $bank_transaction->not_assigned_amount;
567 my $payment_received = $bank_transaction->amount > 0;
568 my $payment_sent = $bank_transaction->amount < 0;
571 foreach my $invoice_id (@{ $params{invoice_ids} }) {
572 my $invoice = SL::DB::Manager::Invoice->find_by(id => $invoice_id) || SL::DB::Manager::PurchaseInvoice->find_by(id => $invoice_id);
577 message => $::locale->text("The ID #1 is not a valid database ID.", $invoice_id),
580 push @{ $data{invoices} }, $invoice;
583 if ( $payment_received
584 && any { ( $_->is_sales && ($_->amount < 0))
585 || (!$_->is_sales && ($_->amount > 0))
586 } @{ $data{invoices} }) {
590 message => $::locale->text("Received payments can only be posted for sales invoices and purchase credit notes."),
595 && any { ( $_->is_sales && ($_->amount > 0))
596 || (!$_->is_sales && ($_->amount < 0) && ($_->invoice_type eq 'purchase_invoice'))
597 } @{ $data{invoices} }) {
601 message => $::locale->text("Sent payments can only be posted for purchase invoices and sales credit notes."),
605 my $max_invoices = scalar(@{ $data{invoices} });
608 foreach my $invoice (@{ $data{invoices} }) {
609 my $source = ($data{sources} // [])->[$n_invoices];
610 my $memo = ($data{memos} // [])->[$n_invoices];
615 if (!$not_assigned_amount && $invoice->open_amount) {
619 message => $::locale->text("A payment can only be posted for multiple invoices if the amount to post is equal to or bigger than the sum of the open amounts of the affected invoices."),
624 if ( defined $::form->{invoice_skontos}->{"$bt_id"} ) {
625 $payment_type = shift(@{ $::form->{invoice_skontos}->{"$bt_id"} });
627 $payment_type = 'without_skonto';
630 # TODO rewrite this: really booked amount should be a return value of Payment.pm
631 # also this controller shouldnt care about how to calc skonto. we simply delegate the
632 # payment_type to the helper and get the corresponding bank_transaction values back
634 my $open_amount = ($payment_type eq 'with_skonto_pt' ? $invoice->amount_less_skonto : $invoice->open_amount);
635 my $amount_for_booking = abs(($open_amount < $not_assigned_amount) ? $open_amount : $not_assigned_amount);
636 $amount_for_booking *= $sign;
637 $bank_transaction->invoice_amount($bank_transaction->invoice_amount + $amount_for_booking);
639 # ... and then pay the invoice
640 my @acc_ids = $invoice->pay_invoice(chart_id => $bank_transaction->local_bank_account->chart_id,
641 trans_id => $invoice->id,
642 amount => ($open_amount < $not_assigned_amount) ? $open_amount : $not_assigned_amount,
643 payment_type => $payment_type,
646 transdate => $bank_transaction->transdate->to_kivitendo);
647 # ... and record the origin via BankTransactionAccTrans
648 if (scalar(@acc_ids) != 2) {
652 message => $::locale->text("Unable to book transactions for bank purpose #1", $bank_transaction->purpose),
655 foreach my $acc_trans_id (@acc_ids) {
656 my $id_type = $invoice->is_sales ? 'ar' : 'ap';
658 acc_trans_id => $acc_trans_id,
659 bank_transaction_id => $bank_transaction->id,
660 $id_type => $invoice->id,
662 SL::DB::BankTransactionAccTrans->new(%props_acc)->save;
664 # Record a record link from the bank transaction to the invoice
666 from_table => 'bank_transactions',
668 to_table => $invoice->is_sales ? 'ar' : 'ap',
669 to_id => $invoice->id,
671 SL::DB::RecordLink->new(%props)->save;
673 # "close" a sepa_export_item if it exists
674 # code duplicated in action_save_proposals!
675 # currently only works, if there is only exactly one open sepa_export_item
676 if ( my $seis = $invoice->find_sepa_export_items({ executed => 0 }) ) {
677 if ( scalar @$seis == 1 ) {
678 # moved the execution and the check for sepa_export into a method,
679 # this isn't part of a transaction, though
680 $seis->[0]->set_executed if $invoice->id == $seis->[0]->arap_id;
685 $bank_transaction->save;
687 # 'undef' means 'no error' here.
692 my $rez = $data{bank_transaction}->db->with_transaction(sub {
694 $error = $worker->();
705 # Rollback Fehler nicht weiterreichen
707 # aber einen rollback von hand
708 $::lxdebug->message(LXDebug->DEBUG2(),"finish worker with ". ($error ? $error->{result} : '-'));
709 $data{bank_transaction}->db->dbh->rollback if $error && $error->{result} eq 'error';
712 return grep { $_ } ($error, @warnings);
720 $::auth->assert('bank_transaction');
727 sub make_filter_summary {
730 my $filter = $::form->{filter} || {};
734 [ $filter->{"transdate:date::ge"}, $::locale->text('Transdate') . " " . $::locale->text('From Date') ],
735 [ $filter->{"transdate:date::le"}, $::locale->text('Transdate') . " " . $::locale->text('To Date') ],
736 [ $filter->{"valutadate:date::ge"}, $::locale->text('Valutadate') . " " . $::locale->text('From Date') ],
737 [ $filter->{"valutadate:date::le"}, $::locale->text('Valutadate') . " " . $::locale->text('To Date') ],
738 [ $filter->{"amount:number"}, $::locale->text('Amount') ],
739 [ $filter->{"bank_account_id:integer"}, $::locale->text('Local bank account') ],
743 push @filter_strings, "$_->[1]: $_->[0]" if $_->[0];
746 $self->{filter_summary} = join ', ', @filter_strings;
752 my $callback = $self->models->get_callback;
754 my $report = SL::ReportGenerator->new(\%::myconfig, $::form);
755 $self->{report} = $report;
757 my @columns = qw(local_bank_name transdate valudate remote_name remote_account_number remote_bank_code amount invoice_amount invoices currency purpose local_account_number local_bank_code id);
758 my @sortable = qw(local_bank_name transdate valudate remote_name remote_account_number remote_bank_code amount purpose local_account_number local_bank_code);
761 transdate => { sub => sub { $_[0]->transdate_as_date } },
762 valutadate => { sub => sub { $_[0]->valutadate_as_date } },
764 remote_account_number => { },
765 remote_bank_code => { },
766 amount => { sub => sub { $_[0]->amount_as_number },
768 invoice_amount => { sub => sub { $_[0]->invoice_amount_as_number },
770 invoices => { sub => sub { my @invnumbers; for my $obj (@{ $_[0]->linked_invoices }) {
771 next unless $obj; push @invnumbers, $obj->invnumber } return \@invnumbers } },
772 currency => { sub => sub { $_[0]->currency->name } },
774 local_account_number => { sub => sub { $_[0]->local_bank_account->account_number } },
775 local_bank_code => { sub => sub { $_[0]->local_bank_account->bank_code } },
776 local_bank_name => { sub => sub { $_[0]->local_bank_account->name } },
780 map { $column_defs{$_}->{text} ||= $::locale->text( $self->models->get_sort_spec->{$_}->{title} ) } keys %column_defs;
782 $report->set_options(
783 std_column_visibility => 1,
784 controller_class => 'BankTransaction',
785 output_format => 'HTML',
786 top_info_text => $::locale->text('Bank transactions'),
787 title => $::locale->text('Bank transactions'),
788 allow_pdf_export => 1,
789 allow_csv_export => 1,
791 $report->set_columns(%column_defs);
792 $report->set_column_order(@columns);
793 $report->set_export_options(qw(list_all filter));
794 $report->set_options_from_form;
795 $self->models->disable_plugin('paginated') if $report->{options}{output_format} =~ /^(pdf|csv)$/i;
796 $self->models->set_report_generator_sort_options(report => $report, sortable_columns => \@sortable);
798 my $bank_accounts = SL::DB::Manager::BankAccount->get_all_sorted();
800 $report->set_options(
801 raw_top_info_text => $self->render('bank_transactions/report_top', { output => 0 }, BANK_ACCOUNTS => $bank_accounts),
802 raw_bottom_info_text => $self->render('bank_transactions/report_bottom', { output => 0 }),
806 sub init_problems { [] }
811 SL::Controller::Helper::GetModels->new(
816 dir => 0, # 1 = ASC, 0 = DESC : default sort is newest at top
818 transdate => t8('Transdate'),
819 remote_name => t8('Remote name'),
820 amount => t8('Amount'),
821 invoice_amount => t8('Assigned'),
822 invoices => t8('Linked invoices'),
823 valutadate => t8('Valutadate'),
824 remote_account_number => t8('Remote account number'),
825 remote_bank_code => t8('Remote bank code'),
826 currency => t8('Currency'),
827 purpose => t8('Purpose'),
828 local_account_number => t8('Local account number'),
829 local_bank_code => t8('Local bank code'),
830 local_bank_name => t8('Bank account'),
832 with_objects => [ 'local_bank_account', 'currency' ],
836 sub load_ap_record_template_url {
837 my ($self, $template) = @_;
839 return $self->url_for(
840 controller => 'ap.pl',
841 action => 'load_record_template',
843 'form_defaults.amount_1' => $::form->format_amount(\%::myconfig, -1 * $self->transaction->amount, 2),
844 'form_defaults.transdate' => $self->transaction->transdate_as_date,
845 'form_defaults.duedate' => $self->transaction->transdate_as_date,
846 'form_defaults.no_payment_bookings' => 1,
847 'form_defaults.paid_1_suggestion' => $::form->format_amount(\%::myconfig, -1 * $self->transaction->amount, 2),
848 'form_defaults.AP_paid_1_suggestion' => $self->transaction->local_bank_account->chart->accno,
849 'form_defaults.callback' => $self->callback,
853 sub load_gl_record_template_url {
854 my ($self, $template) = @_;
856 return $self->url_for(
857 controller => 'gl.pl',
858 action => 'load_record_template',
860 'form_defaults.amount_1' => abs($self->transaction->not_assigned_amount), # always positive
861 'form_defaults.transdate' => $self->transaction->transdate_as_date,
862 'form_defaults.callback' => $self->callback,
863 'form_defaults.bt_id' => $self->transaction->id,
864 'form_defaults.bt_chart_id' => $self->transaction->local_bank_account->chart->id,
868 sub setup_search_action_bar {
869 my ($self, %params) = @_;
871 for my $bar ($::request->layout->get('actionbar')) {
875 submit => [ '#search_form', { action => 'BankTransaction/list' } ],
876 accesskey => 'enter',
882 sub setup_list_all_action_bar {
883 my ($self, %params) = @_;
885 for my $bar ($::request->layout->get('actionbar')) {
889 submit => [ '#filter_form', { action => 'BankTransaction/list_all' } ],
890 accesskey => 'enter',
905 SL::Controller::BankTransaction - Posting payments to invoices from
906 bank transactions imported earlier
912 =item C<save_single_bank_transaction %params>
914 Takes a bank transaction ID (as parameter C<bank_transaction_id> and
915 tries to post its amount to a certain number of invoices (parameter
916 C<invoice_ids>, an array ref of database IDs to purchase or sales
919 This method handles already partly assigned bank transactions.
921 This method cannot handle already partly assigned bank transactions, i.e.
922 a bank transaction that has a invoice_amount <> 0 but not the fully
923 transaction amount (invoice_amount == amount).
925 If the amount of the bank transaction is higher than the sum of
926 the assigned invoices (1 .. n) the bank transaction will only be
929 The whole function is wrapped in a database transaction. If an
930 exception occurs the bank transaction is not posted at all. The same
931 is true if the code detects an error during the execution, e.g. a bank
932 transaction that's already been posted earlier. In both cases the
933 database transaction will be rolled back.
935 If warnings but not errors occur the database transaction is still
938 The return value is an error object or C<undef> if the function
939 succeeded. The calling function will collect all warnings and errors
940 and display them in a nicely formatted table if any occurred.
942 An error object is a hash reference containing the following members:
946 =item * C<result> — can be either C<warning> or C<error>. Warnings are
947 displayed slightly different than errors.
949 =item * C<message> — a human-readable message included in the list of
950 errors meant as the description of why the problem happened
952 =item * C<bank_transaction_id>, C<invoice_ids> — the same parameters
953 that the function was called with
955 =item * C<bank_transaction> — the database object
956 (C<SL::DB::BankTransaction>) corresponding to C<bank_transaction_id>
958 =item * C<invoices> — an array ref of the database objects (either
959 C<SL::DB::Invoice> or C<SL::DB::PurchaseInvoice>) corresponding to
968 Niclas Zimmermann E<lt>niclas@kivitendo-premium.deE<gt>,
969 Geoffrey Richardson E<lt>information@richardson-bueren.deE<gt>