Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Splitbuchungen editieren #536

Merged

Conversation

JohannMaierhofer
Copy link

Das ist eine Fix für #533

Es gab verschiedene Fehler in der vorhandenen Implementierung.

  • Wird eine Splitbuchung gespeichert wurde bei Steuersatz > 0 eine Steuerbuchung erzeugt wenn sie nicht schon existiert hat. Das gleiche passiert beim editieren. Es wurde aber auch beim editieren der Haupt- und Gegenbuchung eine Steuerbuchung erzeugt. Das passiert jetzt nur noch bei Splitbuchungen
  • Hat man eine Splitbuchung editiert wird geprüft ob es abhängige Steuerbuchungen gibt. Diese werden als zu löschen markiert wenn die Splitbuchung keinen Steuersatz > 0 mehr hat. Das Problem war aber wenn man eine Steuerbuchung editiert hat. Da ihr Steuersatz nicht > 0 ist wurde die zugehörige Splitbuchung zum löschen markiert. Beim editieren von Steuerbuchungen darf man also dependent Buchungen nicht löschen. Es gibt aber kein Flag in der Buchung woran man erkennen könnte ob es sich bei dependent Buchungen um die Buchung selbst oder um die zugehörigen Steuerbuchungen handelt. Ich leite dies jetzt implizit aus dem Betrag der Buchung ab. Die Beträge bei Steuerbuchungen sind ja immer absolut gesehen kleiner als der Betrag der zugehörigen Buchung. Damit scheint es zu funktionieren.

@JohannMaierhofer JohannMaierhofer added the fix This will fix a bug label Dec 17, 2024
@tolot27 tolot27 linked an issue Dec 17, 2024 that may be closed by this pull request
@@ -967,7 +982,9 @@ private void handleStore() throws ApplicationException
{
b.plausi();
Buchungsart b_art = b.getBuchungsart();
if (b_art.getSteuersatz() > 0) {
// Keine Steuer Buchungen erzeugen beim Speichern einer Haupt- bzw. Gegenbuchung
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Steuerbuchungen

@dippeal dippeal merged commit 07bc15a into openjverein:master Dec 18, 2024
2 checks passed
@JohannMaierhofer JohannMaierhofer deleted the fix_splitbuchung_steuer branch December 18, 2024 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix This will fix a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ausgeglichene Splitbuchung bearbeiten erzeugt weitere Splitbuchung
3 participants