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 Familienzahler #256

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

JohannMaierhofer
Copy link

@JohannMaierhofer JohannMaierhofer commented Jul 4, 2024

Bei der Implementierung der Funktion für Familienbeitrag gab es einige Fehler.

  • Ändert man die Beitragsgruppe eines Familienzahlers wurde nicht geprüft ob dieser aktuell für andere zahlt.
  • Löscht man einen Familienzahler wurde nicht geprüft ob er für andere zahlt. In diesem Fall zeigen bei denen die Zahler IDs auf ein nicht existierendes Mitglied. Beim Öffnen des Mitgliedsdialogs kommt es dann zu einem Fehler der Anzeige.

Ich habe jetzt den Plausi Check von MitgliedImpl geändert. Falls die Beitragsart nicht FAMILIE_ZAHLER ist wird jetzt neu geprüft ob das Mitglied gerade für andere zahlt. Die Beitragsart könnte ja gerade geändert worden sein.

Beim deleteCheck() prüfe ich auch ob das Mitglied noch für andere zahlt. In diesem Fall kann man nicht löschen.

@JohannMaierhofer JohannMaierhofer added the fix This will fix a bug label Jul 4, 2024
@JohannMaierhofer
Copy link
Author

Ich habe hier den filter austritt entfernt. Bei Ändern der Beitragsgruppe oder Löschen dürfen überhaupt keine Familienmitglieder vorhanden sein.

@JohannMaierhofer
Copy link
Author

Ich habe noch einen weiteren Fehler gefunden.
Man kann bei einem Familien Zahler keinen Austritt setzen wenn nicht alle Familien Mitglieder ausgetreten sind.
Allerdings war es möglich, bei einem Familien Mitglied das Austrittsdatum wieder zu löschen obwohl der zugehörige Familien Zahler schon ausgetreten ist. Das prüfe ich jetzt auch ab.

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.

None yet

1 participant