0333ec9b36588bf93be6c597ca77c2e62c1e21ae
[kivitendo-erp.git] / SL / BackgroundJob / SelfTest / Transactions.pm
1 ackage SL::BackgroundJob::SelfTest::Transactions;
2
3 use utf8;
4 use strict;
5 use parent qw(SL::BackgroundJob::SelfTest::Base);
6
7 use SL::DBUtils;
8
9 use Rose::Object::MakeMethods::Generic (
10   scalar => [ qw(dbh fromdate todate) ],
11 );
12
13 sub run {
14   my ($self) = @_;
15
16   $self->_setup;
17
18   $self->tester->plan(tests => 20);
19
20   $self->check_konten_mit_saldo_nicht_in_guv;
21   $self->check_bilanzkonten_mit_pos_eur;
22   $self->check_balanced_individual_transactions;
23   $self->check_verwaiste_acc_trans_eintraege;
24   $self->check_verwaiste_invoice_eintraege;
25   $self->check_ar_acc_trans_amount;
26   $self->check_ap_acc_trans_amount;
27   $self->check_netamount_laut_invoice_ar;
28   $self->check_invnumbers_unique;
29   $self->check_summe_stornobuchungen;
30   $self->check_ar_paid;
31   $self->check_ap_paid;
32   $self->check_ar_overpayments;
33   $self->check_ap_overpayments;
34   $self->check_paid_stornos;
35   $self->check_stornos_ohne_partner;
36   $self->check_overpayments;
37   $self->check_every_account_with_taxkey;
38   $self->calc_saldenvortraege;
39   $self->check_missing_tax_bookings;
40 }
41
42 sub _setup {
43   my ($self) = @_;
44
45   # TODO FIXME calc dates better, unless this is wanted
46 #  my $year =
47   #$self->fromdate(DateTime->new(day => 1, month => 1, year => DateTime->today->year));
48   #$self->todate($self->fromdate->clone->add(years => 1)->add(days => -1));
49   $self->fromdate(DateTime->new(day => 1, month => 1, year => 2015));
50   $self->todate($self->fromdate->clone->add(years => 1)->add(days => -1));
51   $main::lxdebug->message(0, 'hier und dort:' . $self->fromdate . ' mit ' . $self->todate);
52   $self->dbh($::form->get_standard_dbh);
53 }
54
55 sub check_konten_mit_saldo_nicht_in_guv {
56   my ($self) = @_;
57
58   my $query = qq|
59     SELECT c.accno, c.description, c.category, SUM(a.amount) AS Saldo
60     FROM chart c,
61          acc_trans a
62     WHERE c.id = a.chart_id
63      and  (c.category like 'I' or c.category like 'E')
64      and  amount != 0
65      and  pos_eur is null
66          and  a.transdate >= ? and a.transdate <= ?
67     GROUP BY c.accno,c.description,c.category,c.pos_bilanz,c.pos_eur
68     ORDER BY c.accno|;
69
70   my $konten_nicht_in_guv =  selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
71
72   my $correct = 0 == scalar grep { $_->{Saldo} } @$konten_nicht_in_guv;
73
74   $self->tester->ok($correct, "Erfolgskonten mit Saldo nicht in GuV (Saldenvortragskonten können ignoriert werden, sollten aber 0 sein)");
75   if (!$correct) {
76     for my $konto (@$konten_nicht_in_guv) {
77       $self->tester->diag($konto);
78     }
79   }
80 }
81
82 sub check_bilanzkonten_mit_pos_eur {
83   my ($self) = @_;
84
85   my $query = qq|SELECT accno, description FROM chart WHERE (category = 'A' OR category = 'L' OR category = 'Q') AND (pos_eur IS NOT NULL OR pos_eur != 0)|;
86
87   my $bilanzkonten_mit_pos_eur = selectall_hashref_query($::form, $self->dbh, $query);
88   if (@$bilanzkonten_mit_pos_eur) {
89      $self->tester->ok(0, "Es gibt Bilanzkonten die der GuV/EÜR zugeordnet sind)");
90      $self->tester->diag("$_->{accno}  $_->{description}") for @$bilanzkonten_mit_pos_eur;
91   } else {
92      $self->tester->ok(1, "Keine Bilanzkonten in der GuV");
93   }
94 }
95
96 sub check_balanced_individual_transactions {
97   my ($self) = @_;
98
99   my $query = qq|
100     select sum(ac.amount) as amount,trans_id,ar.invnumber as ar,ap.invnumber as ap,gl.reference as gl
101       from acc_trans ac
102       left join ar on (ar.id = ac.trans_id)
103       left join ap on (ap.id = ac.trans_id)
104       left join gl on (gl.id = ac.trans_id)
105     where ac.transdate >= ? AND ac.transdate <= ?
106     group by trans_id,ar.invnumber,ap.invnumber,gl.reference
107     having sum(ac.amount) != 0;|;
108
109   my $acs = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
110   if (@$acs) {
111     $self->tester->ok(0, "Es gibt unausgeglichene acc_trans-Transaktionen:");
112     for my $ac (@{ $acs }) {
113       $self->tester->diag("trans_id: $ac->{trans_id},  amount = $ac->{amount}, ar: $ac->{ar} ap: $ac->{ap} gl: $ac->{gl}");
114     }
115   } else {
116     $self->tester->ok(1, "Alle acc_trans Transaktionen ergeben in Summe 0, keine unausgeglichenen Transaktionen");
117   }
118 }
119
120 sub check_verwaiste_acc_trans_eintraege {
121   my ($self) = @_;
122
123   my $query = qq|
124       select trans_id,amount,accno,description from acc_trans a
125     left join chart c on (c.id = a.chart_id)
126     where trans_id not in (select id from gl union select id from ar union select id from ap order by id)
127       and a.transdate >= ? and a.transdate <= ? ;|;
128
129   my $verwaiste_acs = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
130   if (@$verwaiste_acs) {
131      $self->tester->ok(0, "Es gibt verwaiste acc-trans Einträge! (wo ar/ap/gl-Eintrag fehlt)");
132      $self->tester->diag($_) for @$verwaiste_acs;
133   } else {
134      $self->tester->ok(1, "Keine verwaisten acc-trans Einträge (wo ar/ap/gl-Eintrag fehlt)");
135   }
136 }
137
138 sub check_verwaiste_invoice_eintraege {
139   # this check is always run for all invoice entries in the entire database
140   my ($self) = @_;
141   my $query = qq|
142      select * from invoice i
143       where trans_id not in (select id from ar union select id from ap order by id) |;
144
145   my $verwaiste_invoice = selectall_hashref_query($::form, $self->dbh, $query);
146   if (@$verwaiste_invoice) {
147      $self->tester->ok(0, "Es gibt verwaiste invoice Einträge! (wo ar/ap-Eintrag fehlt)");
148      for my $invoice ( @{ $verwaiste_invoice }) {
149         $self->tester->diag("invoice: id: $invoice->{id}  trans_id: $invoice->{trans_id}   description: $invoice->{description}  itime: $invoice->{itime}");
150      };
151   } else {
152      $self->tester->ok(1, "Keine verwaisten invoice Einträge (wo ar/ap-Eintrag fehlt)");                                                                                       }
153 }
154
155 sub check_netamount_laut_invoice_ar {
156   my ($self) = @_;
157   my $query = qq|
158     select sum(round(cast(i.qty*(i.fxsellprice * (1-i.discount)) as numeric), 2))
159     from invoice i
160     left join ar a on (a.id = i.trans_id)
161     where a.transdate >= ? and a.transdate <= ?;|;
162   my ($netamount_laut_invoice) =  selectfirst_array_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
163
164   $query = qq| select sum(netamount) from ar where transdate >= ? and transdate <= ? AND invoice; |;
165   my ($netamount_laut_ar) =  selectfirst_array_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
166
167   # should be enough to get a diff below 1. We have currently the following issues:
168   # verkaufsbericht berücksichtigt keinen rabatt
169   # fxsellprice ist mit mwst-inklusive
170   my $correct = abs($netamount_laut_invoice - $netamount_laut_ar) < 1;
171
172   $self->tester->ok($correct, "Summe laut Verkaufsbericht sollte gleich Summe aus Verkauf -> Berichte -> Rechnungen sein");
173   if (!$correct) {
174     $self->tester->diag("Netto-Summe laut Verkaufsbericht (invoice): $netamount_laut_invoice");
175     $self->tester->diag("Netto-Summe laut Verkauf -> Berichte -> Rechnungen: $netamount_laut_ar");
176   }
177 }
178
179 sub check_invnumbers_unique {
180   my ($self) = @_;
181
182   my $query = qq| select  invnumber,count(invnumber) as count from ar
183                where transdate >= ? and transdate <= ?
184                group by invnumber
185                having count(invnumber) > 1; |;
186   my $non_unique_invnumbers =  selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
187
188   if (@$non_unique_invnumbers) {
189     $self->tester->ok(0, "Es gibt doppelte Rechnungsnummern");
190     for my $invnumber (@{ $non_unique_invnumbers }) {
191       $self->tester->diag("invnumber: $invnumber->{invnumber}    $invnumber->{count}x");
192     }
193   } else {
194     $self->tester->ok(1, "Alle Rechnungsnummern sind eindeutig");
195   }
196 }
197
198 sub check_summe_stornobuchungen {
199   my ($self) = @_;
200
201   my $query = qq|
202     select sum(amount) from ar a JOIN customer c ON (a.customer_id = c.id)
203     WHERE storno is true
204       AND a.transdate >= ? and a.transdate <= ?|;
205   my ($summe_stornobuchungen_ar) = selectfirst_array_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
206
207   $query = qq|
208     select sum(amount) from ap a JOIN vendor c ON (a.vendor_id = c.id)
209     WHERE storno is true
210       AND a.transdate >= ? and a.transdate <= ?|;
211   my ($summe_stornobuchungen_ap) = selectfirst_array_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
212
213   $self->tester->ok($summe_stornobuchungen_ap == 0, 'Summe aller Einkaufsrechnungen (stornos + stornierte) soll 0 sein');
214   $self->tester->ok($summe_stornobuchungen_ar == 0, 'Summe aller Verkaufsrechnungen (stornos + stornierte) soll 0 sein');
215   $self->tester->diag("Summe Verkaufsrechnungen (ar): $summe_stornobuchungen_ar") if $summe_stornobuchungen_ar;
216   $self->tester->diag("Summe Einkaufsrechnungen (ap): $summe_stornobuchungen_ap") if $summe_stornobuchungen_ap;
217 }
218
219 sub check_ar_paid {
220   my ($self) = @_;
221
222   my $query = qq|
223       select invnumber,paid,
224            (select sum(amount) from acc_trans a left join chart c on (c.id = a.chart_id) where trans_id = ar.id and c.link like '%AR_paid%') as accpaid ,
225            paid+(select sum(amount) from acc_trans a left join chart c on (c.id = a.chart_id) where trans_id = ar.id and c.link like '%AR_paid%') as diff
226     from ar
227     where
228           (select sum(amount) from acc_trans a left join chart c on (c.id = a.chart_id) where trans_id = ar.id and c.link like '%AR_paid%') is not null
229             AND storno is false
230       AND transdate >= ? and transdate <= ?
231     order by diff |;
232
233   my $paid_diffs_ar = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
234
235   my $errors = scalar grep { $_->{diff} != 0 } @$paid_diffs_ar;
236
237   $self->tester->ok(!$errors, "Vergleich ar paid mit acc_trans AR_paid");
238
239   for my $paid_diff_ar (@{ $paid_diffs_ar }) {
240     next if $paid_diff_ar->{diff} == 0;
241     $self->tester->diag("ar invnumber: $paid_diff_ar->{invnumber} : paid: $paid_diff_ar->{paid}    acc_paid= $paid_diff_ar->{accpaid}    diff: $paid_diff_ar->{diff}");
242   }
243 }
244
245 sub check_ap_paid {
246   my ($self) = @_;
247
248   my $query = qq|
249       select invnumber,paid,
250             (select sum(amount) from acc_trans a left join chart c on (c.id = a.chart_id) where trans_id = ap.id and c.link like '%AP_paid%') as accpaid ,
251             paid-(select sum(amount) from acc_trans a left join chart c on (c.id = a.chart_id) where trans_id = ap.id and c.link like '%AP_paid%') as diff
252      from ap
253      where
254            (select sum(amount) from acc_trans a left join chart c on (c.id = a.chart_id) where trans_id = ap.id and c.link like '%AP_paid%') is not null
255        AND transdate >= ? and transdate <= ?
256      order by diff |;
257
258   my $paid_diffs_ap = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
259
260   my $errors = scalar grep { $_->{diff} != 0 } @$paid_diffs_ap;
261
262   $self->tester->ok(!$errors, "Vergleich ap paid mit acc_trans AP_paid");
263   for my $paid_diff_ap (@{ $paid_diffs_ap }) {
264      next if $paid_diff_ap->{diff} == 0;
265      $self->tester->diag("ap invnumber: $paid_diff_ap->{invnumber} : paid: $paid_diff_ap->{paid}    acc_paid= $paid_diff_ap->{accpaid}    diff: $paid_diff_ap->{diff}");
266   }
267 }
268
269 sub check_ar_overpayments {
270   my ($self) = @_;
271
272   my $query = qq|
273        select invnumber,paid,amount,transdate,c.customernumber,c.name from ar left join customer c on (ar.customer_id = c.id)
274        where abs(paid) > abs(amount)
275        AND transdate >= ? and transdate <= ?
276        order by invnumber;|;
277
278   my $overpaids_ar =  selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
279
280   my $correct = 0 == @$overpaids_ar;
281
282   $self->tester->ok($correct, "Keine Überzahlungen laut ar.paid");
283   for my $overpaid_ar (@{ $overpaids_ar }) {
284     $self->tester->diag("ar invnumber: $overpaid_ar->{invnumber} : paid: $overpaid_ar->{paid}    amount= $overpaid_ar->{amount}  transdate = $overpaid_ar->{transdate}");
285   }
286 }
287
288 sub check_ap_overpayments {
289   my ($self) = @_;
290
291   my $query = qq|
292       select invnumber,paid,amount,transdate,vc.vendornumber,vc.name from ap left join vendor vc on (ap.vendor_id = vc.id)
293       where abs(paid) > abs(amount)
294       AND transdate >= ? and transdate <= ?
295       order by invnumber;|;
296
297   my $overpaids_ap =  selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
298
299   my $correct = 0 == @$overpaids_ap;
300
301   $self->tester->ok($correct, "Überzahlungen laut ap.paid:");
302   for my $overpaid_ap (@{ $overpaids_ap }) {
303     $self->tester->diag("ap invnumber: $overpaid_ap->{invnumber} : paid: $overpaid_ap->{paid}    amount= $overpaid_ap->{amount}  transdate = $overpaid_ap->{transdate}");
304   }
305 }
306
307 sub check_paid_stornos {
308   my ($self) = @_;
309
310   my $query = qq|
311     SELECT ar.invnumber,sum(amount - COALESCE((SELECT sum(amount)*-1
312                             FROM acc_trans LEFT JOIN chart ON (acc_trans.chart_id=chart.id)
313                             WHERE link ilike '%paid%' AND acc_trans.trans_id=ar.id ),0)) as "open"
314     FROM ar, customer
315     WHERE paid != amount
316       AND ar.storno
317       AND (ar.customer_id = customer.id)
318       AND ar.transdate >= ? and ar.transdate <= ?
319     GROUP BY ar.invnumber|;
320   my $paid_stornos = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
321
322   $self->tester->ok(0 == @$paid_stornos, "Keine bezahlten Stornos");
323   for my $paid_storno (@{ $paid_stornos }) {
324     $self->tester->diag("invnumber: $paid_storno->{invnumber}   offen: $paid_storno->{open}");
325   }
326 }
327
328 sub check_stornos_ohne_partner {
329   my ($self) = @_;
330
331   my $query = qq|
332     SELECT (SELECT cast ('ar' as text)) as invoice ,ar.id,invnumber,storno,amount,transdate,type,customernumber as cv_number
333     FROM ar
334     LEFT JOIN customer c on (c.id = ar.customer_id)
335     WHERE storno_id is null AND storno is true AND ar.id not in (SELECT storno_id FROM ar WHERE storno_id is not null AND storno is true)
336     AND ar.transdate >= ? and ar.transdate <= ?
337     UNION
338     SELECT (SELECT cast ('ap' as text)) as invoice,ap.id,invnumber,storno,amount,transdate,type,vendornumber as cv_number
339     FROM ap
340     LEFT JOIN vendor v on (v.id = ap.vendor_id)
341     WHERE storno_id is null AND storno is true AND ap.id not in (SELECT storno_id FROM ap WHERE storno_id is not null AND storno is true)
342     AND ap.transdate >= ? and ap.transdate <= ?|;
343
344   my $stornos_ohne_partner =  selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate,
345                                                                                    $self->fromdate, $self->todate);
346
347   $self->tester->ok(@$stornos_ohne_partner == 0, 'Es sollte keine Stornos ohne Partner geben');
348   if (@$stornos_ohne_partner) {
349     $self->tester->diag("Stornos ohne Partner, oder Storno über Jahreswechsel hinaus");
350   }
351   my $stornoheader = 0;
352   for my $storno (@{ $stornos_ohne_partner }) {
353     if (!$stornoheader++) {
354       $self->tester->diag(join "\t", keys %$storno);
355     }
356     $self->tester->diag(join "\t", map { $storno->{$_} } keys %$storno);
357   }
358 }
359
360 sub check_overpayments {
361   my ($self) = @_;
362
363   # Vergleich ar.paid und das was laut acc_trans bezahlt wurde
364   # "als bezahlt markieren" ohne sauberes Ausbuchen führt zu Differenzen bei offenen Forderungen
365   # geht nur auf wenn acc_trans Zahlungseingänge auch im Untersuchungszeitraum lagen
366   # Stornos werden rausgefiltert
367   my $query = qq|
368     SELECT
369     invnumber,customernumber,name,ar.transdate,ar.datepaid,
370     amount,
371     amount-paid as "open via ar",
372     paid as "paid via ar",
373     coalesce((SELECT sum(amount)*-1 FROM acc_trans LEFT JOIN chart ON (acc_trans.chart_id=chart.id)
374       WHERE link ilike '%paid%' AND acc_trans.trans_id=ar.id AND acc_trans.transdate <= ?),0) as "paid via acc_trans"
375     FROM ar left join customer c on (c.id = ar.customer_id)
376     WHERE
377      ar.storno IS FALSE
378      AND transdate >= ? AND transdate <= ?|;
379
380   my $invoices = selectall_hashref_query($::form, $self->dbh, $query, $self->todate, $self->fromdate, $self->todate);
381
382   my $count_overpayments = scalar grep {
383        $_->{"paid via ar"} != $_->{"paid via acc_trans"}
384     || (    $_->{"amount"} - $_->{"paid via acc_trans"} != $_->{"open via ar"}
385          && $_->{"paid via ar"} != $_->{"paid via acc_trans"} )
386   } @$invoices;
387
388   $self->tester->ok($count_overpayments == 0, 'Vergleich ar.paid und das was laut acc_trans bezahlt wurde');
389
390   if ($count_overpayments) {
391     for my $invoice (@{ $invoices }) {
392       if ($invoice->{"paid via ar"} != $invoice->{"paid via acc_trans"}) {
393         $self->tester->diag("paid via ar (@{[ $invoice->{'paid via ar'} * 1 ]}) !=   paid via acc_trans  (@{[ $invoice->{'paid via acc_trans'} * 1 ]}) (at least until transdate!)");
394         if (defined $invoice->{datepaid}) {
395           $self->tester->diag("datepaid = $invoice->{datepaid})");
396         }
397         $self->tester->diag("Überzahlung bei Rechnung: $invoice->{invnumber}") if $invoice->{"paid via acc_trans"} > $invoice->{amount};
398       } elsif ( $invoice->{"amount"} - $invoice->{"paid via acc_trans"} != $invoice->{"open via ar"} && $invoice->{"paid via ar"} != $invoice->{"paid via acc_trans"}) {
399         $self->tester->diag("amount - paid_via_acc_trans !=  open_via_ar");
400         $self->tester->diag("Überzahlung bei Rechnung: $invoice->{invnumber}") if $invoice->{"paid via acc_trans"} > $invoice->{amount};
401       } else {
402         # nothing wrong
403       }
404     }
405   }
406 }
407
408 sub calc_saldenvortraege {
409   my ($self) = @_;
410
411   my $saldenvortragskonto = '9000';
412
413   # Saldo Saldenvortragskonto 9000 am Jahresanfang
414   my $query = qq|
415       select sum(amount) from acc_trans where chart_id = (select id from chart where accno = ?) and transdate <= ?|;
416   my ($saldo_9000_jahresanfang) = selectfirst_array_query($::form, $self->dbh, $query, $saldenvortragskonto, DateTime->new(day => 1, month => 1, year => DateTime->today->year));
417   $self->tester->diag("Saldo 9000 am 01.01.@{[DateTime->today->year]}: @{[ $saldo_9000_jahresanfang * 1 ]}    (sollte 0 sein)");
418
419     # Saldo Saldenvortragskonto 9000 am Jahresende
420   $query = qq|
421       select sum(amount) from acc_trans where chart_id = (select id from chart where accno = ?) and transdate <= ?|;
422   my ($saldo_9000_jahresende) = selectfirst_array_query($::form, $self->dbh, $query, $saldenvortragskonto, DateTime->new(day => 31, month => 12, year => DateTime->today->year));
423   $self->tester->diag("Saldo $saldenvortragskonto am 31.12.@{[DateTime->today->year]}: @{[ $saldo_9000_jahresende * 1 ]}    (sollte 0 sein)");
424 }
425
426 sub check_every_account_with_taxkey {
427   my ($self) = @_;
428
429   my $query = qq|SELECT accno, description FROM chart WHERE id NOT IN (select chart_id from taxkeys)|;
430   my $accounts_without_tk = selectall_hashref_query($::form, $self->dbh, $query);
431
432   if ( scalar @{ $accounts_without_tk } > 0 ){
433     $self->tester->ok(0, "Folgende Konten haben keinen gültigen Steuerschlüssel:");
434
435     for my $account_without_tk (@{ $accounts_without_tk } ) {
436       $self->tester->diag("Kontonummer: $account_without_tk->{accno} Beschreibung: $account_without_tk->{description}");
437     }
438   } else {
439     $self->tester->ok(1, "Jedes Konto hat einen gültigen Steuerschlüssel!");
440   }
441 }
442
443 sub check_ar_acc_trans_amount {
444   my ($self) = @_;
445
446   my $query = qq|
447           select sum(ac.amount) as amount, ar.invnumber,ar.netamount
448           from acc_trans ac left join ar on (ac.trans_id = ar.id)
449           WHERE ac.chart_link like 'AR_amount%'
450           AND ac.transdate >= ? AND ac.transdate <= ?
451           group by invnumber,netamount having sum(ac.amount) <> ar.netamount|;
452
453   my $ar_amount_not_ac_amount = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
454
455   if ( scalar @{ $ar_amount_not_ac_amount } > 0 ) {
456     $self->tester->ok(0, "Folgende Ausgangsrechnungen haben einen falschen Netto-Wert im Nebenbuch:");
457
458     for my $ar_ac_amount_nok (@{ $ar_amount_not_ac_amount } ) {
459       $self->tester->diag("Rechnungsnummer: $ar_ac_amount_nok->{invnumber} Hauptbuch-Wert: $ar_ac_amount_nok->{amount}
460                             Nebenbuch-Wert: $ar_ac_amount_nok->{netamount}");
461     }
462   } else {
463     $self->tester->ok(1, "Hauptbuch-Nettowert und Debitoren-Nebenbuch-Nettowert  stimmen überein.");
464   }
465
466 }
467
468 sub check_ap_acc_trans_amount {
469   my ($self) = @_;
470
471   my $query = qq|
472           select sum(ac.amount) as amount, ap.invnumber,ap.netamount
473           from acc_trans ac left join ap on (ac.trans_id = ap.id)
474           WHERE ac.chart_link like 'AR_amount%'
475           AND ac.transdate >= ? AND ac.transdate <= ?
476           group by invnumber,netamount having sum(ac.amount) <> ap.netamount*-1|;
477
478   my $ap_amount_not_ac_amount = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
479
480   if ( scalar @{ $ap_amount_not_ac_amount } > 0 ) {
481     $self->tester->ok(0, "Folgende Eingangsrechnungen haben einen falschen Netto-Wert im Nebenbuch:");
482
483     for my $ap_ac_amount_nok (@{ $ap_amount_not_ac_amount } ) {
484       $self->tester->diag("Rechnungsnummer: $ap_ac_amount_nok->{invnumber} Hauptbuch-Wert: $ap_ac_amount_nok->{amount}
485                             Nebenbuch-Wert: $ap_ac_amount_nok->{netamount}");
486     }
487   } else {
488     $self->tester->ok(1, "Hauptbuch-Nettowert und Kreditoren-Nebenbuch-Nettowert stimmen überein.");
489   }
490
491 }
492
493
494 sub check_missing_tax_bookings {
495
496   my ($self) = @_;
497
498   # check tax bookings. all taxkey <> 0 should have tax bookings in acc_trans
499
500   my $query = qq| select trans_id, chart.accno,transdate from acc_trans left join chart on (chart.id = acc_trans.chart_id)
501                     WHERE taxkey <> 0 AND trans_id NOT IN
502                     (select trans_id from acc_trans where chart_link ilike '%tax%' and trans_id IN
503                     (SELECT trans_id from acc_trans where taxkey <> 0))
504                     AND transdate >= ? AND transdate <= ?|;
505
506   my $missing_tax_bookings = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate);
507
508   if ( scalar @{ $missing_tax_bookings } > 0 ) {
509     $self->tester->ok(0, "Folgende Konten weisen Buchungen ohne Steuer auf:");
510
511     for my $acc_trans_nok (@{ $missing_tax_bookings } ) {
512       $self->tester->diag("Kontonummer: $acc_trans_nok->{accno} Belegdatum: $acc_trans_nok->{transdate} Trans-ID: $acc_trans_nok->{trans_id}.
513                            Kann über System -> Korrekturen im Hauptbuch bereinigt werden.");
514     }
515   } else {
516     $self->tester->ok(1, "Hauptbuch-Nettowert und Nebenbuch-Nettowert stimmen überein.");
517   }
518 }
519
520
521 1;
522
523 __END__
524
525 =encoding utf-8
526
527 =head1 NAME
528
529 SL::BackgroundJob::SelfTest::Transactions - base tests
530
531 =head1 DESCRIPTION
532
533 Several tests for data integrity.
534
535 =head1 FUNCTIONS
536
537 =head1 BUGS
538
539 =head1 AUTHOR
540
541 G. Richardson E<lt>information@richardson-bueren.deE<gt>
542 Jan Büren E<lt>information@richardson-bueren.deE<gt>
543 Sven Schoeling E<lt>s.schoeling@linet-services.deE<gt>
544
545 =cut
546