Endereco / endereco-shopware5-client

Endereco Addressvalidation Services for Shopware 5. Kundenstammdaten-Validierung und Korrekturvorschläge.
https://www.endereco.de/shopware
3 stars 1 forks source link

S5P-13: Falsche doAccounting Umsetzung #19

Closed iljamobilemojo closed 3 years ago

iljamobilemojo commented 3 years ago

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 commented 3 years ago

Gelöst damit: https://github.com/Endereco/endereco-shopware5-client/pull/20