search

openjverein / jverein

Open JVerein - Open Source Vereinsverwaltung
https://openjverein.github.io
GNU General Public License v3.0
45 stars 17 forks source link

Fenster SEPA-XML-Version entfernen #442

Closed fkuersch closed 2 weeks ago

fkuersch commented 3 weeks ago

Bei jedem Abrechnungslauf wird ein Fenster angezeigt, über das man die SEPA-XML-Version auswählen kann.

Dass es die Option jetzt gibt, ist super. Ich glaube aber nicht, dass sie bei jeder Lastschrift neu eingestellt werden muss. Es reicht, wenn sie einmalig unter Einstellungen -> Abrechnung eingestellt wird (und dann ggf. in ein paar Jahren noch mal, wenn die Bank ihre Standards geändert hat). Dafür braucht es kein separates Fenster.

Falls die XML-Version noch nicht in den Einstellungen ausgewählt wurde, sollte diese Info unter "Fehler/Warnungen/Hinweise" angezeigt werden (s. auch #441). Achtung: Es kann auch Vereine geben, die gar keine Lastschriften erstellen bzw. Abrechnungsläufe ohne Lastschriften.

JohannMaierhofer commented 3 weeks ago

Das eigene Fenster kommt aber doch gar nicht wenn man in den Einstellungen etwas gesetzt hat. Eine Warnung finde ich auch übertrieben. Ich schlage vor in den Dialog eine Checkbox einzubauen mit dem Text "Als Standard übernehmen". Das setzt dann den Wert in den Einstellungen. So macht das ja auch andere Software üblicherweise. Danach kommt die Meldung dann auch nicht mehr.

JohannMaierhofer commented 2 weeks ago

Achtung: Es kann auch Vereine geben, die gar keine Lastschriften erstellen bzw. Abrechnungsläufe ohne Lastschriften.

Wenn man keine Lastschriften erstellt kann man in der Abrechnung statt Hibiskus oder XML File Ausgabe einfach "Keine" angeben. Dann sollte auch kein Dialog kommen.

fkuersch commented 2 weeks ago

@JohannMaierhofer So eine Checkbox wäre eine Lösung. Ich glaube aber, dass es für diese einmalige Einstellung nicht zwei verschiedene Orte geben muss. Aus Gründen der Wartbarkeit wäre ich eher dafür, die Codebasis zu reduzieren und nur die Option in den zentralen Einstellungen zu behalten

JohannMaierhofer commented 2 weeks ago

Und wenn in den Einstellungen nichts gesetzt ist muss ich ohne diesen Dialog einen anderen öffnen der eine Fehlermeldung ausgibt. Da muss der User in die Einstellungen wechseln, die Version setzen und die Abrechnung neu starten. Das geht mit dem jetzigen Dialog einfacher.

fkuersch commented 2 weeks ago

@JohannMaierhofer Bei anderen Fehlern (z. B. fehlende Gläubiger-Identifikationsnummer, fehlende IBAN etc.) wird unten im Log einfach ein roter Hinweis angezeigt. Dafür gibt es kein separates Fenster

JohannMaierhofer commented 2 weeks ago

Ok, ich werde es entfernen. Ich habe sowieso gesehen, dass der defaultmäßig vorgeschlagene Wert im Dialog keine gültige SEPA Version ist. Ich werde jetzt gleich einen Defaultwert in den Einstellungen setzen. Dann ist immer etwas da.