Abrechnungslauf ohne SEPA File Abfrage

[JohannMaierhofer]

Erlaube einen Abrechnungslauf ohne eine XML Datei zu generieren.

Ich habe Abrechnungsläufe wo alle Mitglieder per Überweisung bezahlen. Da brauche ich die Abfrage nach dem Dateinamen für die XML Datei nicht. Ich erlaube darum als Abrechnungsausgabe neben Datei und Hibiscus auch "Keine". Damit wird der Lauf durchgeführt aber kein XML erzeugt.
Meine erste Idee war die Abfrage nach dem Filenamen erst zu machen wenn bekannt ist ob eine SEPA Lastschrift enthalten ist oder nicht. Da die Berechnung aber in eine Hintergrund Task stattfindet dachte ich, dass man da wohl keine GUI Abfrage einfügen sollte.
Da ich weiß, dass keine Lastschriften enthalten sind klappt das auch mit der expliziten Auswahl.
Damit wäre es auch möglich kein XML zu erzeugen obwohl es SEPA Lastschriften gibt. Das wäre der Fall wenn jemand den Lauf machen will aber die Lastschriften anderweitig generiert z.B. anderes Tool.
Da das Feature sonst nicht stört sollte es so ok sein.

[NicoB77]

Zusammenfassung: gewünschtes Verhalten ließ sich vorher schon mit Ausgabe "Hibiscus" erreichen, Code unnötig aufgebläht. Ich bin dafür, die Änderungen zurückzunehmen.

Diese Änderungen kommen mir nicht sinnvoll vor. Für Deinen Anwendungsfall (nur Lastschriften) hättest Du einfach als Ausgabe Hibiscus wählen können. Dann musst Du keinen Dateinamen für die Ausgabe angeben. Wenn Dich die Log-Meldung, dass die Hibiscus-Lastschrift geschrieben wurde, stört (es wurde ja keine erzeugt), hätte man nur diese Meldung unterbinden können.

Den von Dir beschriebenen Anwendungsfall, dass die erzeugte XML-Datei/Hibiscus-Lastschrift nicht verwendet werden soll, weil die Lastschrift extern erzeugt wird, halte ich für relativ selten. Falls man das wirklich will, sollte es kein Problem sein, die Hibiscus-Lastschrift (wahlweise die XML-Datei) nach dem Abrechnungslauf zu löschen.

Es dürfte häufiger vorkommen, dass jemand versehentlich "Keine Datei" auswählt, und dann den Abrechnungslauf löschen und neu erzeugen muss.

Die zusätzlich nötigen Änderungen an der Pre-Notification haben blähen den Code unnötig auf.

[JohannMaierhofer]

Ich persönlich würde halt gerne das auswählen können was ich haben will anstatt nach Workarounds zu suchen. Aber bei meinem Anwendungsfall würde das wohl gehen. Ich muss halt auch noch eine Fake Gläubiger ID eingeben weil ich in dem Verein überhaupt keine Lastschriften mache. Ohne Gläubiger ID gibt es eine Fehlermeldung wenn Hibiscus ausgewählt ist.

Deine Anmerkung zum beschriebenen Anwendungsfall verstehe ich aber jetzt nicht ganz. Der Workaround funktioniert nicht weil ja sonst etwas an Hibiscus übergeben werden würde. Die Auswahl Datei fragt jetzt wieder den Dateinamen ab. Genau das sollte ja das Feature unterbinden. In dem Fall geht es also nur mit dem Feature und nicht mit dem Workaround.

[JohannMaierhofer]

Wenn das neue File Ct1Ausgabe.java den Code aufbläht kann ich diese Datei auch entfernen. Ich entferne das unnötige Keine dann beim Anzeigen des Input Dialogs. So sollte es auch gehen. Ob das dann schöner ist können andere entscheiden.

[JohannMaierhofer]

Ein Vorschlag:
Ich mache die Ct1Ausgabe rückgängig und entferne auch den Wert "Keine" aus dem Enum der Abrechnungsausgabe.
Im Abrechnungsdialog füge ich dafür im Dialog eine Sucheingabe "Bitte auswählen" hinzu. Dann sollte man es auch nicht so leicht aus Versehen ohne auszuwählen benutzen.
Wäre das ok für dich?

[NicoB77]

Wenn es keine Lastschrift gibt, wird auch nichts an Hibiscus übergeben. Die Log-Meldung stimmt in diesem Fall nicht (was man natürlich korrigieren könnte). Daher halte ich das für die einfachste Lösung, auch wenn man dann eine Gläubiger-ID eintragen muss (daran hatte ich nicht gedacht, ist aber nur einmalig).