syntax icon indicating copy to clipboard operation
syntax copied to clipboard

Erstellen von Splitbuchungen

Open lenilsas opened this issue 1 year ago • 4 comments

neue Funktion um Buchungen zu Spliten

lenilsas avatar Apr 19 '24 19:04 lenilsas

Vielen Dank für das Patch. Werde aber einige Tage brauchen, um mir das in Ruhe anzuschauen, da es ein komplexes Thema ist. Insbesondere zwei Bereiche sind mir da wichtig:

  • Wenn eine Buchung gesplittet wird, existiert danach sowohl die ursprüngliche Buchung als auch die darauf verweisenden Splits. Ist hier sichergestellt, dass alle Berechnungen (Salden, Summen, Auswertungen, etc.) noch korrekt sind oder erscheinen dann dort zu hohe Summen, da die Beträge in Summe doppelt einfliessen?
  • Ist die Performance beim Laden von Buchungslisten noch gewährleistet. Gerade in den letzten Releases habe ich daran gearbeitet, das Laden der Buchungsliste soweit zu optimieren, dass im Wesentlichen nur 3-4 SQL-Statements ausgeführt werden und die Anzahl der SQL-Queries nicht abhängig von der Anzahl der Buchungen ist. Gerade wenn die Datenbank nicht im LAN sondern remote per VPN angebunden wurde, ist das wichtig. Hier muss sichergestellt sein, dass beim Laden von Listen nicht irgendwo für jeden einzelnen Datensatz nochmal ein extra SQL-Query ausgelöst wird. Das kann man gut sehen, wenn man das Loglevel unter Datei->Einstellungen auf DEBUG stellt. Wenn es sich per SQL nicht lösen lässt, muss man da ggf. mit Caches arbeiten.

Hattest du schon Gelegenheit, die beiden Themen zu betrachten?

willuhn avatar Apr 20 '24 06:04 willuhn

  • Bei den Gesplitteten Buchungen werden immer die Hauptbuchungen rausgefiltert und nur die aufgespitteten Teile Summiert. Dass habe ich in allen Listen und Auswertungen geprüft.
  • Im Moment werden noch mehrere SQL Querries pro Buchung durchgeführt. Das schaue ich mir nochmal an, ob ich das mit einem Querry oder mit Cache lösen kann.

lenilsas avatar Apr 22 '24 11:04 lenilsas

Ich habe ein paar Anpassungen vorgenommen, damit es nicht so Viele SQL Queries gibt. So sollte es von der Performance her wesentlich besser sein.

lenilsas avatar Apr 24 '24 11:04 lenilsas

Danke für die Änderungen. Ich schau's mir. Kann aber noch ein paar Tage dauern.

willuhn avatar Apr 25 '24 08:04 willuhn

Danke für das Patch. Habe es übernommen.

willuhn avatar May 13 '24 11:05 willuhn