Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fomular Zähler nicht gespeichert #570

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

lenilsas
Copy link

Der Zähler wurde beim erstellen von Formularen hochgezählt aber nicht gespeichert (Außer bei Rechnungen)

@lenilsas
Copy link
Author

Mir stellt sich die Frage, ob wir das wirklich so ändern wollen oder ein neues Konzept für den Zähler einführen sollen.
Der Zähler wird ja bei jedem Erstellen des Dokuments hochgezählt. Das heißt, wenn man es nochmal versendet oder ausdruckt hat es eine andere Nummer. Da der Fehler dass das Formular nicht gespeichert wird und der Zähler somit garnicht hochgezählt wird heißt, dass es auch nicht oder nur wenig genutzt wird.
Bei Rechnungen habe ich ja schon eine Rechnungsnummer eingeführt, die id aus der Tabelle. So könnten wir es auch bei Spendenbescheinigungen machen. Bei Pre-Notifications ist es auch über die id möglich. Denn bleiben nur noch die freien Formulare übrig, und da ist die Frage, ob mann da wirklich einen Zähler braucht.
Was haltet ihr davon?

@dippeal
Copy link
Member

dippeal commented Jan 15, 2025

Zähler bitte drin lassen. Wir z.B. nutzen den Zähler. Rechnungsnummern müssen ja fortlaufend sein und dürfen nicht unterbrochen sein.

@JohannMaierhofer JohannMaierhofer merged commit 60d9ff0 into openjverein:master Jan 15, 2025
2 checks passed
@lenilsas
Copy link
Author

@dippeal ich verstehe nicht ganz was du meinst. Bisher (bis Version 2.8.23) ist es ja so, dass bei jedem Aufruf einer Rechnung eine neue Nummer vergeben wird. Wenn mann die Rechnungen als PDF ausdruckt, speichert und per Post verschickt ist das kein Problem. Wenn man jedoch per Mail versendet und dann nochmal als PDF anschauen möchte was man verschickt hat, hat die Rechnung eine andere Rechnungsnummer. Dur die Änderung aus #365 gibt es eine eindeutige Rechnung in der DB, von der kann die id als Rechnungsnummer verwendet werden, diese ist eindeutig fortlaufend und hat keine Unterbrechungen sofern man keine Rechnungen löscht (Wobei meines Wissens nirgends gefordert wird, dass die Nummer ohne Unterbrechung fortlaufen ist).
Daher würde ich sehr dazu raten bei Rechnungen nicht mehr den Zähler sondern die eindeutige Rechungsnummer zu verwenden.
Ein Problem ist nur die Umstellung. Da bei der neuen Version wieder bei 1 losgezählt wird. Ich hatte schon überlegt die höchste bei Rechnungsformularen hinterlegte Zahl als minimale Rechnung-id per Update zu setzen. Ist das das Problem was du meinst?

@dippeal
Copy link
Member

dippeal commented Jan 16, 2025

Wir haben auch andere Formulare bei denen gezählt wird. Wir haben einen guten Workflow für unsere Rechnungsnummern (Jahreszahl abhängig). Leute welche die neue Rechnungsnummer verwenden wollen können dies ja machen.

@JohannMaierhofer
Copy link

Mir fällt grade auf, dass es ja auch Mahnungen gibt. Wird da auch gezählt. Die sind ja nicht wie die Rechnung in der DB. Da müsste dann das Formular auch gespeichert werden. Das war im PR nicht dabei.

@lenilsas
Copy link
Author

Mir fällt grade auf, dass es ja auch Mahnungen gibt. Wird da auch gezählt. Die sind ja nicht wie die Rechnung in der DB. Da müsste dann das Formular auch gespeichert werden. Das war im PR nicht dabei.

Das wird mit dem gleichen Code wie bei den Rechnungen gemacht, daher war das schon umgesetzt

@lenilsas
Copy link
Author

Wir haben auch andere Formulare bei denen gezählt wird. Wir haben einen guten Workflow für unsere Rechnungsnummern (Jahreszahl abhängig). Leute welche die neue Rechnungsnummer verwenden wollen können dies ja machen.

Wir können es natürlich auch lassen wenn es Nutzer wie dich gibt für die das besser passt. Es hat halt nur ein paar nachteile die einen richtigen Workflow erfordern. es ist auch flexiebler da man die Nummer anpassen kann. Eventuell wäre es ja auch hilfreich in der Rechnung eine von der Id unabhängige Rechnungsnummer zu erstellen...

@lenilsas lenilsas deleted the formular branch January 18, 2025 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants