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.
Mit diesem Feature lassen sich alte Buchungen z.B. deren Aufbewahrungsfrist nach 10 Jahren abgelaufen ist, löschen.
Gelöscht werden die Buchungen aller Konten vor einem vorgegebenen Datum.
Damit bei den Mitgliedern im Mitgliedskonto keine Fehlbeträge entstehen können zugeordnete Sollbuchungen mit gelöscht werden.
Der Button für das Löschen befindet sich im Buchungen View.
Der Dialog sieht so aus.
Buchungsdokument verweist auf eine Buchung. Diese werden automatisch gelöscht durch das Setzen der Referenz auf CASCADE.
Ich habe keine Beschränkung für das Datum eingebaut. Es steht nur der Hinweis im Dialog, die Aufbewahrungsfristen zu beachten. Mann kann also auch Buchungen löschen die jünger als die Aufbewahrungsfrist sind.
Ich sehe darin kein Problem, weil ja niemand gezwungen ist die Kontoauszüge (Buchungen) und Jahresabschlüsse in JVerein aufzubewahren. Die Pflicht ist ja auch erfüllt wenn man die Kontoauszüge und Jahresabschlüsse als Papier oder elektronisch z.B. PDF aufbewahrt.
In BuchungImpl habe ich beim deleteCheck den insertCheck auskommentiert. Es macht ja keinen Sinn auf die Insert Bedingungen zu prüfen. Da dürfte man älter als 10 Jahre nicht im Datum haben und könnte sie also auch nicht löschen.
PS: Wegen der Migration muss dieses Feature nach #213 übergeben werden.