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 zusatzbetrag #410

Merged
merged 4 commits into from
Nov 1, 2024
Merged

Conversation

JohannMaierhofer
Copy link

@JohannMaierhofer JohannMaierhofer commented Oct 28, 2024

Der Filter "Active" im Zusatzbeiträge View sollte Einträge anzeigen die eine nächste Fälligkeit in der Zukunft haben. Jedenfalls wäre das mein Verständnis.

Dafür war aber die Abfrage falsch und so war meine Liste immer leer.

Dann habe ich auch noch den View Titel angepasst.

@JohannMaierhofer JohannMaierhofer added the fix This will fix a bug label Oct 28, 2024
@lenilsas
Copy link

Für mein Verständnis war es richtig. Es werden die aktuell auszuführenden Zusatzberäge angezeigt, also alle die fällig sind und noch nicht ausgeführt wurden (bzw. bei Intervall auch die aktuell fälligen).
Bei "Noch nicht ausgeführt" kann man ja alle zukünftige (und aktuellen) anzeigen lassen.

@JohannMaierhofer
Copy link
Author

Ja, so kann man es auch interpretieren. Dann wäre es richtig.

Ich verstehe aber dann den Workflow nicht. Da würde ich ja nur Einträge sehen wenn die Fälligkeit schon vorbei ist. Wenn ich dann den Abrechnungslauf mache würde eine Abbuchung ja nach der Fälligkeit passierten.
Damit die Abbuchung zum Fälligkeitsdatum passiert muss ich den Abrechnungslauf vor dem Fälligkeitsdatum machen. Wenn ich das mache wird die nächste Fälligkeit einen Monat weiter gesetzt. Ich werde also nie einen Eintrag darin sehen weil das aktuelle Datum immer vor der nächsten Fälligkeit ist.

Damit ist bei mir der View per default immer leer. Ich muss immer erst den default Filter auf "Alle" umstellen um zu sehen was es gibt. Das sind dann aber auch welche dabei die schon beendet sind.

Auch der "Noch nicht ausgeführt" Filter zeigt nichts an weil er auf Ausführung == null prüft. Bei periodischen Zahlungen wurde aber schon mal etwas ausgeführt und damit steht ab der ersten Buchung auch nichts mehr in der Liste.

Somit habe ich keinen Filter der mir laufende Zusatzbeiträge anzeigen würde. Ich bekomme mur bei "Alle" etwas zu sehen oder bei den generierten Datum Einträgen.

Da müsste ich mir dann wohl einen Filter "Laufend" implementieren der mir zeigt ob es noch Zusatzbeiträge gibt die abzurechnen sind. Und den hätte ich den in meinem Workflow als Defaultwert. Da bräuchte ich wohl eine Auswahl Möglichkeit in den Einstellungen was der Default Filter sein soll.

@JohannMaierhofer
Copy link
Author

JohannMaierhofer commented Oct 28, 2024

Bei "Noch nicht ausgeführt" kann man ja alle zukünftige (und aktuellen) anzeigen lassen.

Der zeigt heute nichts an. Mit meinem letzten Kommentar ganz unten hätte ich da einen anderen Vorschlag.

@JohannMaierhofer
Copy link
Author

Für mein Verständnis war es richtig. Es werden die aktuell auszuführenden Zusatzberäge angezeigt, also alle die fällig sind und noch nicht ausgeführt wurden (bzw. bei Intervall auch die aktuell fälligen).

Für mich zeigt fällig, dass ich etwas machen muss. Also einen Abrechnungslauf vor/zur nächsten Fälligkeit.

Bei der jetzigen Implementierung werden "überfällige" Einträge angezeigt und die sollte es nicht geben.

Wir könnten auch Aktive laut meiner Implementierung nehmen und ein "Überfällig" implementieren was dem heutigen Aktiv entspricht.

@JohannMaierhofer
Copy link
Author

JohannMaierhofer commented Oct 28, 2024

Noch eine andere Idee, die überfälligen, also die heute aktiven könnte man in "Noch nicht ausgeführt" aufnehmen. Sie hätten ja schon ausgeführt gehört, sind es aber nicht weil das Fälligkeitsdatum abgelaufen ist. Dann könnte es auch bei periodischen Zahlungen hier Einträge geben.
Und Aktive sind dann die laufenden mit einem Fälligkeitsdatum in der Zukunft.

Alternativ dazu dein Vorschlag von oben. Aktive bleiben die überfälligen und in "Noch nicht ausgeführt" nimmt man die laufenden auf. Dann würde ich das "Laufend" oder irgend wie anders nennen.

@lenilsas
Copy link

Vielleicht sollte man ein Filterfeld mit dem Stichtag machen, dann ist das Verhalten gleich wie bei der Abrechnung.

@JohannMaierhofer
Copy link
Author

JohannMaierhofer commented Oct 28, 2024

Vielleicht sollte man ein Filterfeld mit dem Stichtag machen, dann ist das Verhalten gleich wie bei der Abrechnung.

Die Idee ist auch nicht schlecht.

Dann sollte aber der Stichtag beim Anzeigen des Views automatisch auf das Datum der nächsten Fälligkeit gesetzt werden. Sonst ist die Liste wieder leer wenn man das aktuelle Datum nimmt.

Ich verstehe das dann so, dass Aktiv alle anzeigt die zu diesem Stichtag abgerechnet werden würden.

Wenn ich dann den Abrechnungslauf mit diesem Datum mache, dann steht heute in letzter Ausführung das Datum an dem ich den Abrechnungslauf gemacht habe. Die Ausführung ist aber eigentlich erst zum SEPA Datum des Abrechnungslauf. Sollte dann nicht auch dieses als letzte Ausführung eingetragen werden. Die wäre dann zwar in der Zukunft, aber die Abbuchungen (echte Ausführung) ist es ja auch. Und die nächste Fälligkeit wird ja auch einen Monat weiter gesetzt.

Was machen wir dann mit dem "Noch nicht ausgeführt". Da könnte man man die anzeigen, bei denen die letzte Ausführung in der Zukunft liegt, wenn wir oben das so machen. Das ergäbe einen Sinn. Sie sind abgerechnet aber die Abbuchung ist noch nicht erfolgt.

So würde mir das gefallen.

PS: Das "Letzte Ausführung" in der Tabelle müsste man dann in "Ausführung" ändern.

@JohannMaierhofer
Copy link
Author

Ehrlich gesagt bin ich mir aber nicht sicher ob ein normaler Anwender all die neuen Vorschläge verstehen würde. Wir selbst haben ja schon Probleme zu verstehen was "Akive" bedeutet.

Wäre es da nicht doch einfacher den PR zu nehmen wie er ist?

Aktive sind halt alle die nicht beendet sind. Die Tabelle ist dann nicht mehr leer und man sieht auch gleich wann die nächste fällig ist. Dann muss man dafür sorgen vorher den Abrechnungslauf zu machen.

Das ist einfach zu verstehen und sollte nicht verwirren.

Vielleicht können andere das auch noch kommentieren bevor wir weiter machen.

@JohannMaierhofer JohannMaierhofer added help wanted Extra attention is needed and removed fix This will fix a bug labels Oct 29, 2024
@JohannMaierhofer
Copy link
Author

Ich sehe aber gerade, dass ich auch für die einfache Lösung noch etwas ändern muss. Die nächste Fälligkeit kann ja auch in der Vergangenheit sein.
Bei Aktive müssen alle nicht geschlossen angezeigt werden.

@JohannMaierhofer
Copy link
Author

JohannMaierhofer commented Oct 29, 2024

Ich habe es jetzt so geändert, dass alle Aktiv sind die ein Fälligkeitsdatum vor dem Ende Datum haben.

Man könnte jetzt noch die Spalte "Letzte Ausführung" in "Letzte Abrechnung" ändern. Da wird ja das Datum des Abrechnungslauf eingetragen und nicht das Abbuchungsdatum welches der letzten Fälligkeit entsprechen würde. Es ist halt nicht so klar was da mit "Ausführung" gemeint ist.

Man könnte natürlich auch hier das Abbuchungsdatum eintragen und die Spalte "Vorhergehende Fälligkeit" nennen. Es wäre die vor dem Eintrag "Nächste Fälligkeit" und könnte auch in der Zukunft sein aber eben vor der nächsten Fälligkeit. Die wird ja beim Abrechnungslauf auch schon hochgezählt.

Da war auch noch ein Fehler auch nach alter Implementiereung. Bisher wurde ein Eintrag nicht mehr angezeigt wenn das aktuelle Datum nach dem Ende Datum war, auch wenn noch eine Fälligkeit vor dem Ende Datum existiert hat. Das hätte eigentlich Active sein müssen.

Copy link

@lenilsas lenilsas left a comment

Choose a reason for hiding this comment

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

So finde ich es gut umgesetzt

@dippeal
Copy link
Member

dippeal commented Nov 1, 2024

Im nächsten PR dann gerne die Anpassung der View Spalten, falls benötigt.

@dippeal dippeal merged commit 98b398e into openjverein:master Nov 1, 2024
@JohannMaierhofer JohannMaierhofer deleted the fix_zusatzbetrag branch November 1, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants