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

Abrechnungslauf fix #28

Closed
wants to merge 9 commits into from
Closed

Conversation

sbuerg
Copy link

@sbuerg sbuerg commented Feb 25, 2023

Hi,

in der Vergangenheit hatte ich den Abrechnungslauf mit Ausgabe DATEI auf hbci4java umgestellt. So war/ist es möglich durch hbci4java aktuelle SEPA-XML Schematas zu verwenden.

Im Forum kam es allerdings zu folgenden Rückmeldungen
https://www.jverein-forum.de/viewtopic.php?t=7206

Die Problem ist leicht zu erklären. Bei der Ausgabeart DATEI wurden die XMLs sauber geschrieben. Aber die Gegenbuchungen nicht mehr. Bei der Ausgabeart HIBISCUS wurde ebenfalls nach der SEPA Schemata gefragt, was unnötig ist - die Abrechnung funktionierte aber nicht mehr sauber. Wählte man zusätzlich SEPA Datei ausdrucken, kam es zu einem weiteren abnormalen Verhalten. (Liegt am neuen Format der XML-Dateien), welche als Basis für die PDF Ausgabe verwendet wird.

Mit diesem PR habe ich nun folgendes erstmal behoben:

  • Beide Ausgabearten führen wieder wie gewohnt zu Gegenbuchungen
  • Wird Hibiscus a gewählt so erscheint keine SEPA-XML Versionsabfrage mehr
  • Wird zusätzlich SEPA ausdrucken gewählt, wird zwar nach der Ausgabedatei für PDF gefragt (es wird allerdings keine Ausgabe erzeugt, weil die PDF Ausgabe die neue XML auslesen will, was fehl schlägt).

Dies ist nur ein Bugfix. Später muß man den Abrechnungslauf nochmal komplett redesignen vom Code. Die Code-Struktur ist leider sehr tief /komplex verschachelt um das von jetzt auf gleich zu beheben

@NicoB77
Copy link

NicoB77 commented Sep 25, 2023

Ich habe meine Änderungen aus dem JVerein-Pull-Request (SEPA RCUR) an den aktuellen Stand von OpenJVerein angepasst und werde in den nächsten Tagen einen Pull-Request dafür erstellen. Das ergibt aber Konflikte mit diesem Pull-Request. Daher schlage ich vor, diesen zunächst zurückzustellen, und sich dann für einen der beiden zu entscheiden. Die hier behobenen Fehler sind in meinem Code ebenfalls behoben.

@NicoB77
Copy link

NicoB77 commented Sep 28, 2023

Pull-Request #42 enthält meine Änderungen. Wenn ich den Code richtig gelesen habe, funktionieren auch mit diesem Fix weder das Erzeugen der PDF-Datei noch eine Lastschrift mit kompakter Abbuchung (bei Ausgabe in eine Datei werden alle Buchungen erzeugt, aber nicht zusammengefasst; bei Ausgabe Hibiscus enthält die Lastschrift keine Buchungen).

@MSchmalzl
Copy link

Nachdem die Änderungen inzwischen mit #42 im Repo sind, schließe ich diesen PR hier.

@MSchmalzl MSchmalzl closed this Oct 16, 2023
auto-merge was automatically disabled October 16, 2023 14:58

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants