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

S5P-13: Falsche doAccounting Umsetzung #19

Closed
iljamobilemojo opened this issue May 5, 2021 · 1 comment
Closed

S5P-13: Falsche doAccounting Umsetzung #19

iljamobilemojo opened this issue May 5, 2021 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@iljamobilemojo
Copy link
Collaborator

iljamobilemojo commented May 5, 2021

Was genau ist das Problem?

Aktuell werden doAccounting erst beim Anlegen oder Speichern eine Adresse abgesendet werden. Falls beim Absenden der Form jedoch ein Validationsfehler kommt, dann wird keine Adresse gespeichert und somit geht die Session verloren, obwohl die Adresse geprüft worden war.

Wieso ist das wichtig?

Dadurch ist unser Konzept der erfolgsbasierten Abrechnung nicht korrekt umgesetzt und unsere statistische Auswertung der Nutzung liefert falsche Zahlen.

Was wäre das erwartete Verhalten?

Die doAccounting sollen beim Absenden der Form produziert werden, unabhängig davon, ob sonstige Werte in der Form korrekt oder falsch sind. Da eine richtig eingegebene Adresse das zweite Mal nicht geändert wird, besteht keine Gefahr der doppelten Abrechnung.

Wie kann man das Problem reproduzieren?

Beim Absenden der Form soll ein Fehler gemacht werden, der die Frontendvalidierung besteht, bei der Server Validierung jedoch einen Fehler verursacht. Man landet somit nach dem Absenden auf der gleichen Seite und die vorhin übermittelte Session geht verloren.

@iljamobilemojo iljamobilemojo added the bug Something isn't working label May 5, 2021
@iljamobilemojo iljamobilemojo self-assigned this May 5, 2021
@iljamobilemojo
Copy link
Collaborator Author

Gelöst damit: #20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant