X-Git-Url: http://wagnertech.de/gitweb/gitweb.cgi/mfinanz.git/blobdiff_plain/eeaa5e92707a1d6be8e63bede49b1868d49eacf8..13f40529221ea3f6d8afbd5a047be7df7fa2a93c:/SL/BackgroundJob/SelfTest/Transactions.pm diff --git a/SL/BackgroundJob/SelfTest/Transactions.pm b/SL/BackgroundJob/SelfTest/Transactions.pm index ca7e278c9..1f706e540 100644 --- a/SL/BackgroundJob/SelfTest/Transactions.pm +++ b/SL/BackgroundJob/SelfTest/Transactions.pm @@ -15,7 +15,7 @@ sub run { $self->_setup; - $self->tester->plan(tests => 21); + $self->tester->plan(tests => 23); $self->check_konten_mit_saldo_nicht_in_guv; $self->check_bilanzkonten_mit_pos_eur; @@ -38,6 +38,8 @@ sub run { $self->calc_saldenvortraege; $self->check_missing_tax_bookings; $self->check_bank_transactions_overpayments; + $self->check_ar_paid_acc_trans; + $self->check_ap_paid_acc_trans; } sub _setup { @@ -474,9 +476,9 @@ sub check_ap_acc_trans_amount { my $query = qq| select sum(ac.amount) as amount, ap.invnumber,ap.netamount from acc_trans ac left join ap on (ac.trans_id = ap.id) - WHERE ac.chart_link like 'AR_amount%' + WHERE ac.chart_link like '%AP_amount%' AND ac.transdate >= ? AND ac.transdate <= ? - group by invnumber,netamount having sum(ac.amount) <> ap.netamount*-1|; + group by invnumber,trans_id,netamount having sum(ac.amount) <> ap.netamount*-1|; my $ap_amount_not_ac_amount = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate); @@ -509,11 +511,12 @@ sub check_missing_tax_bookings { my $missing_tax_bookings = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate); if ( scalar @{ $missing_tax_bookings } > 0 ) { - $self->tester->ok(0, "Folgende Konten weisen Buchungen ohne Steuer auf:"); + $self->tester->ok(0, "Folgende Konten weisen Buchungen ohne Steuerverknüpfung auf:"); for my $acc_trans_nok (@{ $missing_tax_bookings } ) { $self->tester->diag("Kontonummer: $acc_trans_nok->{accno} Belegdatum: $acc_trans_nok->{transdate} Trans-ID: $acc_trans_nok->{trans_id}. - Kann über System -> Korrekturen im Hauptbuch bereinigt werden."); + Kann über System -> Korrekturen im Hauptbuch bereinigt werden. Falls es ein Zahlungskonto ist, wurde + ggf. ein Brutto-Skonto-Konto mit einer Netto-Rechnung verknüpft. Kann nur per SQL geändert werden."); } } else { $self->tester->ok(1, "Hauptbuch-Nettowert und Nebenbuch-Nettowert stimmen überein."); @@ -541,6 +544,54 @@ sub check_bank_transactions_overpayments { } } +sub check_ar_paid_acc_trans { + my ($self) = @_; + + my $query = qq| + select sum(ac.amount) as paid_amount, ar.invnumber,ar.paid + from acc_trans ac left join ar on (ac.trans_id = ar.id) + WHERE ac.chart_link like '%AR_paid%' + AND ac.transdate >= ? AND ac.transdate <= ? + group by invnumber, paid having sum(ac.amount) <> ar.paid*-1|; + + my $ar_amount_not_ac_amount = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate); + + if ( scalar @{ $ar_amount_not_ac_amount } > 0 ) { + $self->tester->ok(0, "Folgende Ausgangsrechnungen haben einen falschen Bezahl-Wert im Nebenbuch:"); + + for my $ar_ac_amount_nok (@{ $ar_amount_not_ac_amount } ) { + $self->tester->diag("Rechnungsnummer: $ar_ac_amount_nok->{invnumber} Hauptbuch-Wert: $ar_ac_amount_nok->{paid_amount} + Nebenbuch-Wert: $ar_ac_amount_nok->{paid}"); + } + } else { + $self->tester->ok(1, "Hauptbuch-Bezahlwert und Debitoren-Nebenbuch-Bezahlwert stimmen überein."); + } +} + +sub check_ap_paid_acc_trans { + my ($self) = @_; + + my $query = qq| + select sum(ac.amount) as paid_amount, ap.invnumber,ap.paid + from acc_trans ac left join ap on (ac.trans_id = ap.id) + WHERE ac.chart_link like '%AP_paid%' + AND ac.transdate >= ? AND ac.transdate <= ? + group by trans_id,invnumber,paid having sum(ac.amount) <> ap.paid|; + + my $ap_amount_not_ac_amount = selectall_hashref_query($::form, $self->dbh, $query, $self->fromdate, $self->todate); + + if ( scalar @{ $ap_amount_not_ac_amount } > 0 ) { + $self->tester->ok(0, "Folgende Eingangsrechnungen haben einen falschen Bezahl-Wert im Nebenbuch:"); + + for my $ap_ac_amount_nok (@{ $ap_amount_not_ac_amount } ) { + $self->tester->diag("Rechnungsnummer: $ap_ac_amount_nok->{invnumber} Hauptbuch-Wert: $ap_ac_amount_nok->{paid_amount} + Nebenbuch-Wert: $ap_ac_amount_nok->{paid}"); + } + } else { + $self->tester->ok(1, "Hauptbuch Bezahl-Wert und Kreditoren-Nebenbuch-Bezahlwert stimmen überein."); + } +} + 1;