Rundung bei Debitorenbuchung, Kreditorenbuchung und Dialogbuchung
authorG. Richardson <information@kivitendo-premium.de>
Tue, 4 Nov 2014 11:04:43 +0000 (12:04 +0100)
committerG. Richardson <information@kivitendo-premium.de>
Tue, 4 Nov 2014 11:04:43 +0000 (12:04 +0100)
commit0e68056cbc17b531266c16454f3a74ae1e57dea6
treea6212cdac741e1c7bf312bc397c10a6c133dd847
parent71233175a9443a11e4918a876c5069b105786180
Rundung bei Debitorenbuchung, Kreditorenbuchung und Dialogbuchung

Zwei neue Hilfsfunktionen für Form eingeführt die von ap/ar/gl genutzt
werden:

* calculate_tax wird zur Berechnung der Steuer bei
** update in ar, ap und gl
** post_transaction in AR.pm und AP.pm innerhalb von calculate_arap

* calculate_arap berechnet netamount, amount und totaltax anhand einer
vorhandenen $form in ar/ap, und formatiert und rundet die amount_$i and
tax_$i in der $form.

Das Ziel war es, daß diese drei Belege mit den gleichen Formeln zur
Steuerberechnung bei Steuer inkl./exkl. arbeiten. Ein Vorteil im
Vergleich zur Einkaufs- und Verkaufsrechnung ist, daß es hier keine
Positionen/Mengen/Rabatte/etc gibt, sondern direkt Werte auf Konten
bebucht werden, man muß nur Währungskurse berücksichtigen und beim
Schreiben in die Datenbank auf die Vorzeichen achten. Einkaufs- und
Verkaufsrechnungen werden hiervon nicht beeinflußt.

Vor allem bei der Einstellung "MwSt. inkl" wird die Steuer nun immer auf
zwei Stellen gerundet in die Datenbank geschrieben, und nicht (wie
bisher in manchen Fällen) mit 5 Nachkommastellen, was zu mehreren Bugs
geführt hat.

Es gibt einen neuen Test t/form/arap.t der die Berechnungen dieser
beiden Funktionen testet, nicht aber das eigentliche Buchen der Belege.

Mit diesem Commit sollten für zukünftige Buchungen folgende Bugs behoben
sein:

1691 - Rundung bei Berichten bei Buchungen mit MwSt inkl.
2029 - Rundungsfehler bei Dialogbuchung
2033 - Unterschiede in Rundungen durch taxincluded
2094 - Rundungsprobleme in Kreditorenbuchungen: Cent "kippt" bei Zahlungseinbuchung
2435 - Rundungsfehler in Kreditorenbuchungen (Netto vs. Brutto)

Dieser Commit greift in die Eingeweide von Uraltcode ein, wo die
diversen Stellen alle mal mit copy&paste entstanden und dann langsam
divergiert sind, hat also hohes Fehlerpotential. Es lohnt sich, den
DATEV-Check anzuschalten.
SL/AP.pm
SL/AR.pm
SL/Form.pm
bin/mozilla/ap.pl
bin/mozilla/ar.pl
bin/mozilla/gl.pl
t/form/arap.t [new file with mode: 0644]