Spendenbescheinigung auf Zahler ausstellen #493
Conversation
|
Ich habe noch geändert, dass über den Sollbuchung Zuordnen Dialog nach dem Zahler gesucht wird. Dieser hat ja die Buchung bezahlt. Damit findet man Sollbuchungen die er selbst bezahlt und auch die, die er für andere bezahlt. |
Müssen das hier zwei Felder sein? könnte man nicht auch gleichzeitig in beiden Felder Suchen? Ich denke das könnte verwirren wenn nur das eine oder das andere Feld verwendet werden darf.
Ich frage mich, wie Sinnvoll das Bearbeiten und manuelle erstellen von Sollbuchungen insgesamt ist. Normalerweise sollten Sollbuchungen über einen Abrechnungslauf erstellt werden, alle angaben dazu Liegen in den Mitgliedsdaten. Nur in Ausnahmefällen müssen die Sollbuchungen manuell angepasst werden. Vor allem bei Lastschriften kann das zu Verwirrungen führen da diese ja nur beim Abrechnungslauf generiert werden jedoch nicht beim anlegen und bearbeiten von Sollbuchungen. Eventuell wäre ein Freischalten in den Einstellungen Sinnvoll.
Beim Löschen von Mitgliedern müsste dann die Meldung geändert werden, hier steht noch mit drin das Spendenbescheinigungen gelöscht werden.
Das denke ich wäre der beste Weg. So oft sollte das ja nicht wechseln. In der Doku sollten wir darauf hinweisen. Auch beim automatischen erstellen sollte der Anwender immer prüfen ob die Ausgabe korrekt ist, alle fälle können wir nicht mit Tests abdecken.
Ja das stimmt. Hier würde ich es dann auch so machen, dass der Zahler verwendet wird, bei einem abweichenden Kontoinhaber aber nur seine Daten in der Rechnung verwendet werden und es trotzdem dem Mitglied zugeordnet bleibt. |
Ja, es sind zwei Felder. Ich hätte schreiben sollen "der Zahler zusätzlich angezeigt"
Nachdem man aber Sollbuchungen auch manuell erstellen kann muss mann editieren können. Da sollten wir weiter nachdenken. Evtl. Ein Flag in der Sollbuchung ob manuell oder automatisch erzeugt.
Ja, das muss ich ändern.
Doch, man kann Sachspendenbescheinigungen in der Liste mit dem Button unten ohne Mitglied erzeugen. Aber ja, bei Geldspenden haben wir das nicht. Eigentlich ist das inkonsistent. Sachspenden sollten auch durch eine Buchung hinterlegt sein. Mit Spendenbescheinigung ohne Mitglied sehe ich kein Problem. Der Name steht ja immer noch drinnen. Das ist wie bei Buchung, die bleiben ja auch erhalten. Und wegen der Aufbewahrungsfristen macht es ja Sinn diese zu behalten.
|
|
Bezüglich editieren des Zahler, Spendenbescheinigungen ohne Mitglied Referenz und der Mail Adresse des abweichenden Kontoinhaber habe ich mir auch Gedanken gemacht und dann letztendlich bei meinem Vorschlag gelandet. Alles hängt damit zusammen, dass man den abweichenden Kontoinhaber oder auch den Vollzahler wechseln kann bevor man die Spendenbescheinigung erzeugt. Nehmen wir an wir haben ein Mitglied mit Vollzahler. Aus irgendeinem Grund muss er selbst etwas außerordentlich zahlen per Überweisung. Anderes Beispiel Vollzahler, es kann ja sein, dass jemand aus irgendeinem Grund keine Spendenbescheinigungen braucht und jetzt ausgetreten ist. Jetzt braucht er plötzlich doch noch eine Spendenbescheinigung für die letzte Zahlung. Er ist nicht mehr in der Sollbuchung eingetragen. Ich kann aber jetzt trotzdem eine Spendenbescheinigung ohne Verweis auf das Mitglied erstellen und seinen Namen in die Bescheinigung eintragen. Das spricht dafür auch Spendenbescheinigungen ohne Referenz auf ein Mitglied zu erlauben und in diesem Fall auch erzeugen zu können. Damit wäre es aber auch nicht inkonsistent bei Löschen des Zahler die Spendenbescheinigungen bestehen zu lassen,da man sie auch ohne Zahler Mitglied erzeugen kann. Zu dem Mail Problem. Man hat den alternativen Kontoinhaber gewechselt und vorher vergessen die Spendenbescheinigung zu erstellen oder wie oben genannt den Fall, dass man ursprünglich keine gebraucht hat aber plötzlich doch. Erstellt man sie jetzt ist der eingetragene Zahler falsch. Wenn ich jetzt per Mail verschicke bekommt auch noch der falsche die Bescheinigung. Da dachte ich mir wenn man die Mail Adresse nicht nimmt muss man diese ausdrucken und da fällt es einem auf jeden Fall auf, das da der falsche Empfänger drauf steht. Für den Fall, dass der Zahler beim Erstellen der Spendenbescheinigung schon ausgetreten und gelöscht wurde hätte man da natürlich auch keine Mail Adresse. Hier bräuchte man dann doch noch die Mail Adresse als Attribut in der Spendenbescheinigung. Die würde dann aber nur manuell eingetragen. Man könnte aber auch deine Idee umsetzen und eine Mail an das Mitglied und den alternativen Kontoinhaber schicken. Dann muss ich in der Spendenbescheinigung doch auch bei alternativen Kontoinhaber das Mitglied in die Spendenbescheinigung eintragen. Beim versenden muss dann eine Mail an das Mitglied gehen und falls eine in der Spendenbescheinigung eingetragen ist auch an diese. |
ja, das sollte gehen
Ja, das sehe ich auch so dass Sollbuchungen bearbeitbar sein sollen. Aber ich denke eine Checkobx unter Einstellungen "Sollbuchungen bearbeiten zulassen" würde helfen damit unerfahrene Nutzer den richtigen Workflow nutzen ohne inkosistenzen zu erzeugen.
So meinte ich das nicht. ich denke man sollte konsequenter weise das erstellen von Geldspendenbescheinigungen ohne Mitglied und Sollbuchung ermöglichen, da sie eigentlich nicht für die Spendenbescheinigung nötig sind. |
|
Zur Mailadresse des Kontoinhabers: |
Das wäre einfach, man bräuchte einfach im Kontextmenü der Buchung die Option Spendenbescheinigung und erzeugt dann eine ohne Mitglied Referenz. Dann muss man einfach manuell die Spender Daten eintragen, im Prinzip so wie bei der Sachspende wenn es über den Button passiert. Das könnte ich mit extra PR machen. |
Das klingt sehr gut. Mit der Mailadresse können wir dann da noch mal schauen... |
|
Habe Mergekonflikt behoben |
|
Ich das Blockieren der Editierbarkeit von Sollbuchungen wieder rückgängig gemacht. Das könnte man nach #547 evtl. nochmals besprechen. |
|
Ich habe das Bug Flag gesetzt weil es hat ein Fehler ist die Spendenbescheinigung auf die falsche Person auszustellen. |
JohannMaierhofer
added
fix
| Object o = (Object) super.getAttribute("zahlerid"); | ||
| if (o == null) | ||
| return null; | ||
|
|
||
| if(o instanceof Mitglied) | ||
| return (Mitglied)o; | ||
|
|
||
| Cache cache = Cache.get(Mitglied.class,true); | ||
| return (Mitglied) cache.get(o); | ||
| } |
There was a problem hiding this comment.
Bitte hier noch Code Formatting anwenden.
| Object o = (Object) super.getAttribute("zahler"); | ||
| if (o == null) | ||
| { | ||
| return null; | ||
| } | ||
|
|
||
| if(o instanceof Mitglied) | ||
| { | ||
| return (Mitglied)o; | ||
| } | ||
|
|
||
| Cache cache = Cache.get(Mitglied.class,true); | ||
| return (Mitglied) cache.get(o); |
There was a problem hiding this comment.
CF. Außerdem ist das wieder redundanter Code, welchen man mittels eines Interfaces und default methods vermeiden könnte.
Das können wir jedoch auch später aufräumen. ;-)
There was a problem hiding this comment.
Ich bin ja auch dafür redundanten Code zu vermeiden. Ich habe auch damals den FilterControl und DruckMailControl eingeführt weil jeder Control der sie jetzt benutzt für die gleichen Filter Attribute die gleichen Methoden implementiert hatte. Das hat sehr viel redundanten Code entfernt.
Ich verstehe allerdings den Vorschlag mit Interfaces nicht ganz. Ein Interface definiert doch nur die Methoden Signatur. Jeder der es implementiert muss es doch auscodieren. Wie spare ich da redundanten Code.
Wie geht das mit defaults?
There was a problem hiding this comment.
There was a problem hiding this comment.
Danke, ich muss doch langsam anfangen Java zu lernen.
There was a problem hiding this comment.
Ich würde das dann separat angehen. Die gleiche Verdoppelung gibt es ja bei den DB Klassen öfters.
|
Migration ist bei 459. Der PR muss also nach der Mittelverwendung übergeben werden. |
Implementiert das Konzept aus #467.
Ich habe jetzt ein Attribut Zahler in die Sollbuchung aufgenommen. Beim Abrechnungslauf wird dort der Zahler eingetragen. Beim Generieren der Spendenbescheinigung wird nach dem Zahler gesucht und dieser in die Spendenbescheinigung eingetragen.
Hat der Zahler einen alternativen Kontoinhaber werden die Daten dieses Kontoinhabers in die Spendenbescheinigung übernommen, der Zahler wird nicht in der Spendenbescheinigung gesetzt da er ja der alternative Kontoinhaber als Mitglied nicht existiert. Es gibt ja auch jetzt schon Spendenbescheinigungen wo kein Mitglied eingetragen ist (Sachspenden).
Folgende GUI Änderungen habe ich gemacht:
Da man einen Zahler löschen kann habe ich geändert, dass die zugehörigen Spendenbescheinigungen nicht mehr gelöscht werden. Ist der Zahler nicht das Mitglied bleibt ja die Sollbuchung bestehen und man könnte eine neue Spendenbescheinigung auf ein anderes Mitglied ausstellen weil ja die Buchung und Sollbuchung erhalten bleiben. Es schadet auch nicht wenn sie bestehen bleiben, es gilt ja auch bei elektonischen Spendenbescheinigungen eine Aufbewahrungsfrist.
Das Problem, dass man den alternativen Kontoinhaber ändern kann und dann bei der Generierung der Spendenbescheinigung evtl. den falschen kopiert ist hier nicht gelöst. Für eine Lösung müsste man die Daten des alternativen Kontoinhabers in der Sollbuchung speichern. Das wäre #439.
Es gibt folgende Workarounds:
Es ist so, dass man nach Generierung der Spendenbescheinigung noch den Spender in der Sollbuchung ändern kann. Ich habe das nicht verhindert, evtl. gibt es ja einen Anwendungsfall. Für bestehende Spendenbescheinigungen ist das ja egal weil dort ja der Zahler gespeichert ist. Darum sehe ich es auch als kein Problem an, eine Buchung mit Spendenbescheinigung einer andern Sollbuchung zuzuordnen.
Als letztes stellt sich mir noch die Frage wie das eigentlich bei den Rechnungen sein soll. Sollten die dann nicht auch auf den Zahler ausgestellt werden? Der muss ja eine Betrag zahlen und nicht das Mitglied und es wird ja auch bei ihm abgebucht. Wenn also zwei Beträge beim Zahle abgebucht werden sollten auch beide auf der Rechnung stehen. Das sollte man dann mit einem eigenen PR machen.