GuV-Rechnung falsch bei EÜR und teilgezahlten Rechnungen

[lx-office.bugzilla@…]

Hallo,

ich habe bei der GuV-Rechnung (Bericht GuV) festgestellt, daß bei EÜR teilgezahlte Rechnungen voll hinzugerechnet werden. Dies führte bei mir zu einem zu hoch ausgewiesenen Gewinn. Bei der UStVa werden teilgezahlte Rechnungen hingegen korrekt nur entsprechend der Zahlungseingänge gewertet.

Als Bugfix habe ich habe daher den SQL-Query für die GuV-Rechnung analog zur UStVa angepaßt. Der Patch ist angehängt.

Jetzt kriege ich auch für die GuV-Rechnung ein korrektes Ergebnis.

Korrigiert SQL-Query für GuV

(In reply to comment #0)

Als Bugfix habe ich habe daher den SQL-Query für die GuV-Rechnung analog zur
UStVa angepaßt. Der Patch ist angehängt.

Jetzt kriege ich auch für die GuV-Rechnung ein korrektes Ergebnis.

Ohne das ganze jetzt eingespielt und getestet zu haben, hast Du das wirklich ausprobiert?

C-style /* */ Kommentare im qq

Operator sollte wenn überhaupt eher nen SQL Fehler geben.

• Teilung durch Rechnungssumme ist nicht abgesichert (Rechnungssumme kann durchaus 0 sein)
• Wo ist der Aufruf zu chart_category_to_sgn hin?
• Was passiert wenn $fromdate oder $todate leer sind?

(In reply to comment #1)

(In reply to comment #0)

Als Bugfix habe ich habe daher den SQL-Query für die GuV-Rechnung analog zur
UStVa angepaßt. Der Patch ist angehängt.

Jetzt kriege ich auch für die GuV-Rechnung ein korrektes Ergebnis.

Ohne das ganze jetzt eingespielt und getestet zu haben, hast Du das wirklich
ausprobiert?

Ja, habe ich. Es hat mit meiner Konfiguration wunderbar funktioniert und hat nun die gleichen Werte wie die UstVa?. Ich hab das immer mit den Ergebnissen der jährlichen UStVa verglichen.

C-style /* */ Kommentare im qq

Operator sollte wenn überhaupt eher nen SQL

Fehler geben.

Beides ist nicht der Fall. Das kannst du auch leicht testen:

perl -e 'print qq|test /* comment */ test\n|;'

psql -c 'select /* comment test */ 1 /* another comment */, 2;' postgres

• Teilung durch Rechnungssumme ist nicht abgesichert (Rechnungssumme kann

durchaus 0 sein)

Tritt dieses Problem dann nicht auch bei der UStVa auf? Ich habe den SQL-Code ja von der UStVa kopiert.

• Wo ist der Aufruf zu chart_category_to_sgn hin?

Was genau macht der? Bei meiner Konfiguration scheint der nicht erforderlich zu sein. Ich habe nur einen Teil des GuV-SQL-Queries auskommentiert durch jenen Teil der UStVA-SQL-Queries ersetzt, der die richtigen Werte bei ermittelt hat. Das Ganze habe ich mir dabei natürlich in einem SQL-Editor angesehen und darüber aus herausgefunden welche Teile der Queries was zurückliefern. Die Queries selbst habe ich aber im Detail nie wirklich nachvollzogen.

• Was passiert wenn $fromdate oder $todate leer sind?

Das muß wohl noch abgefangen werden. Das kann ich gerne noch einbauen, falls es gewünscht ist.

C-style /* */ Kommentare im qq

Operator sollte wenn überhaupt eher nen SQL

Fehler geben.

Beides ist nicht der Fall. Das kannst du auch leicht testen:

perl -e 'print qq|test /* comment */ test\n|;'

psql -c 'select /* comment test */ 1 /* another comment */, 2;' postgres

Gerade getan und auch nachgeschaut, ist tatsächlich Bestandteil von SQL'99, bitte um Verzeihung.

• Teilung durch Rechnungssumme ist nicht abgesichert (Rechnungssumme kann

durchaus 0 sein)

Tritt dieses Problem dann nicht auch bei der UStVa auf? Ich habe den SQL-Code
ja von der UStVa kopiert.

Kann durchaus sein.

• Was passiert wenn $fromdate oder $todate leer sind?

Das muß wohl noch abgefangen werden. Das kann ich gerne noch einbauen, falls es
gewünscht ist.

Gerne.

Korrigierter SQL-Query für GuV mit Datumscheck

Moin!

Ich hab es leider nicht früher geschafft, den korrigierten Patch zu schicken. Nun klappt auch alles, wenn das Datum nicht gesetzt ist.

Ich habe auch versucht, einen Testcase zu erstellen, bei dem im GuV-Zeitraum nur eine einzige Rechnung mit Rechnungssumme 0 existiert. Dies hat allerdings keinen Fehler ausgelöst. Ich kenne allerdings die DB-Struktur zu wenig, um den Query wirklich nachvollziehen zu können.

Ahjo, pflege ich so ein (mit leichten Korrekturen).

Ich würde alle bitten, die diesen Bug verfolgen das einmal auszutesten.

Commit acd5752aff6f10f3d19b33da176a07895efcb964

keine Beschwerden bisher. schliesse den Bug.

Wie in 1802 herausgefunden führt dieser Patch zu fehlerhaften BWA Ausgaben, weil get_accounts_g für guv und bwa geteilt wird.

#1802 ist mit Commit 4805ef3469753b79cfd5f474bdeac2a326a33a10 behoben.

Ich habe einen kleinen Test gemacht, und für Istversteuerung bei einer Rechnung mit einem Zahlungseingang in einer anderen Periode klappt das für BWA und GuV. Ob das mit Jahreswechsel/EB/SB-Buchungen noch funktioniert habe ich aber nicht getestet, wäre schön wenn das hier jemand mit echten Daten testet.

Und mit f7866ad sollte auch die kaputte behandlung von gl-Buchungen behoben sein.

Schließe den wieder.