Fix Buchungsklassensaldo #648
Conversation
| @@ -49,7 +49,9 @@ public class BuchungsklasseSaldoList extends TablePart implements Part | |||
| private Date datumvon = null; | |||
|
|
|||
| private Date datumbis = null; | |||
There was a problem hiding this comment.
Wieso sind die Einrückungen unterschiedlich? Ist das Konvention?
There was a problem hiding this comment.
Die unterschiedlichen Einrückungen sehe ich nicht bei Github, obwohl ich im Diff View die Whitspaces nicht ausgeblendet habe. Hier an dieser Stelle kommen sie ja nicht von diesem PR, sondern von "Formatierungsfehler" von früher.
| if (Math.abs(einnahmen) >= LIMIT || Math.abs(ausgaben) >= LIMIT | ||
| || Math.abs(umbuchungen) >= LIMIT | ||
| || !Einstellungen.getEinstellung().getUnterdrueckungOhneBuchung()) |
There was a problem hiding this comment.
Kanst du das in ein Prädikat überführen? Das wäre einfacher zu lesen und zu verstehen.
So in der Art
if (!buchungUnterdruecken(einnahmen, ausgaben, umbuchung))
Oder wenn es etwas ohne Negation gibt, noch besser.
There was a problem hiding this comment.
Und den Einstellungscheck am Anfang durchführen, damit die anderen Checks gar nicht erst durchgeführt werden, falls die Unterdrückung aktiv ist.
There was a problem hiding this comment.
Kanst du das in ein Prädikat überführen? Das wäre einfacher zu lesen und zu verstehen.
Grundsätzlich hast du recht. Nur deswegen extra eine Funktion einführen, wo dann der gleiche Check drin steht, halte ich für einmalige Verwendung nicht für nötig.
| } | ||
| suEinnahmen += suBukEinnahmen; | ||
| suAusgaben += suBukAusgaben; | ||
| suUmbuchungen += suBukUmbuchungen; | ||
| if (!ausgabe | ||
| if (Math.abs(suBukEinnahmen) < LIMIT && Math.abs(suBukAusgaben) < LIMIT |
Trotz der Einstellung Buchungsarten ohne Buchung unterdrücken, wurden im Buchungsklassensaldo Buchungsarten angezeigt bei denen sowohl Einnahmen, Ausgaben als auch Umbuchungen 0.00 waren.
Formal ist das korrekt, weil es Umbuchungen gab. Diese heben sich aber gegenseitig auf, wodurch dann der Betrag 0.00 ist. So ist es aber wohl nicht gewollt.
Mit dem Fix werden jetzt auch keine Zeilen angezeigt bei denen alle drei Werte 0.00 sind.