Möglicher Fehler bei Familienzahler #257
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dippeal
approved these changes
Jul 9, 2024
dippeal
added
documentation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ich bin mir nicht sicher ob hier ein Fehler oder Unschönheit vorliegt und bitte um euere Meinung.
Beim Speichern eines Mitglieds wird bei Austritt und Kündigung geprüft ob bei Beitragsart FAMILIE_ZAHLER noch Familienmitglieder existieren die noch nicht ausgetreten sind. Es wird aber nicht geprüft ob Familienmitglieder gekündigt haben.
Das ist inkonsistent. Man kann Austritt nur setzen wenn alle Familienmitglieder auch ausgetreten sind und man kann Kündigung nur setzten wenn alle Familienmitglieder ausgetreten sind. Kündigung der Familienmitglieder reicht nicht.
Allgemein scheint es so zu sein, dass Kuendigung keine Auswirkung auf den Austritt hat. Gekündigte Mitglieder tauchen im Mitglieder View weiter unter dem Filter Angemeldet auf. Auch werden im Abrechnungslauf weiter die Mitgliedsbeiträge eingezogen.
Mit meiner Änderung ergibt sich jetzt folgendes:
Eine ander Lösung wäre gewesen einen eigenen Check für Kündigung zu erstellen und Kündigung nur erlauben wenn alle Familienmitglieder auch gekündigt haben. Das würde aber bezüglich des Abrechnungslaufes nichts ändern und wäre eine künstliche Einschränkung.