openjverein / jverein

Open JVerein - Open Source Vereinsverwaltung
https://openjverein.github.io
GNU General Public License v3.0
42 stars 15 forks source link

Feature Familienbeitrag View Update #258

Closed JohannMaierhofer closed 1 month ago

JohannMaierhofer commented 2 months ago

Mit diesem Feature erweitere ich den Familienbeitrag View.

Mit dem Feature kann man sich jetzt auch die Familien Einträge von ausgetretenen Mitgliedern anschauen. Das ist nützlich wenn man z.B. den Familienzahler löschen will. Das geht erst wenn alle Familienmitglieder gelöscht sind. Mit der Anzeige hier kann man leicht sehen welche Familienmitglieder da sind und leicht zu ihnen wechseln um sie zu löschen. Natürlich könnte man das auch im jeweiligen Mitglied View unter dem Tab Familienverband sehen, aber dann nicht so leicht die Familienmitglieder löschen.

Ich habe auch noch einen Bug behoben. Wenn kein Geburtsdatum beim Mitglied gesetzt war hat der View das Datum 01.01.1900 an den Namen anghängt. Jetzt zeigt er keines an wenn kein Geburtsdatum konfiguriert ist.

Der Radiergummi kennzeichnet abgemeldete Mitglieder.

Bildschirmfoto_20240704_181828

lenilsas commented 1 month ago

Du prüfst ob das Mitglied ein Austrittsdatum hinterlegt hat, aber nicht ob es bereits ausgetreten ist. Das heißt das Mitglied wird mit einem Radiergummi gekennzeichnet, auch wenn es erst in einem Jahr oder so austreten wird. Sollten hier nicht auf das Aktuelle Datum hin geprüft werden ob das Mitglied schon ausgetreten ist?

JohannMaierhofer commented 1 month ago

Im Prinzip hast du Recht. Ich habe aber gesehen, dass bei den Checks beim Speichern auch immer nur geprüft wird ob ein Datum vorhanden ist. Ein richtiges Implementieren müsste z.B. erlauben, dass das Austrittsdatum eines Familienmitglieds vor dem des Familienzahlers ist aber nicht umgekehrt. Das wird wahrscheinlich relativ aufwendig. Mein Vorschlag wäre, hinter dem Namen auch das Austrittsdatum anzuzeigen. Dann wüsste man an dem Icon, dass eines hinterlegt ist und kann ablesen wann.

lenilsas commented 1 month ago

Ja, den Vorschlag finde isch gut. Sonst würde es wirklich sehr kompliziert mit allen Prüfungen.

JohannMaierhofer commented 1 month ago

Jetzt mit Austrittsdatum statt Geburtdatum. Bildschirmfoto_20240729_195433