yakamara / yform

YForm für REDAXO 5 – Formulare im Frontend und Backend mit Verwaltung von Datenbank-Tabellen.
MIT License
77 stars 55 forks source link

Umgang mit Fehlermeldungen anpassen #1380

Open dergel opened 1 year ago

dergel commented 1 year ago

Das ist im Moment nicht ideal gelöst.

WarningMessages können sich gegenseitig überschreiben wenn sie z.B. auf eine ObjektId gemappt sind. d.h. am besten Warnmeldungen auslagern und einen eigenen Handler dafür bauen.

Da dies aber nicht BC ist, kommt das erst in 5.0

dergel commented 1 year ago

siehe auch https://github.com/yakamara/redaxo_yform/issues/623

Rückgabewerte ändern

dergel commented 1 year ago

https://github.com/yakamara/redaxo_yform/issues/1369

dergel commented 1 year ago

https://github.com/yakamara/redaxo_yform/issues/11

alxndr-w commented 1 year ago

Betrifft auch #591 nach meinem Verständnis

tbaddade commented 1 year ago

Betrifft auch https://github.com/yakamara/redaxo_yform/issues/591 nach meinem Verständnis

jein, dort ist der Fall des Überschreibens gewollt. Man müsste die Reihenfolge der Validatoren anpassen.

$form->setValueField('text', ['email', 'E-Mail']);
$form->setValidateField('unique', ['email', '{{ form.error.email.unique }}']);
$form->setValidateField('email', ['email', '{{ form.error.email.email }}']);
$form->setValidateField('empty', ['email', '{{ form.error.email.empty }}']);

Ist die Frage, ob man an einem E-Mail-Feld dann drei Meldungen haben möchte oder eine Einzige.

alxndr-w commented 1 year ago

Empty braucht man gar nicht validieren, das ist schon in unique bspw. dabei. Ansonsten finde ich es richtig, wenn jede Validierung eine neue Fehlermeldung erzeugt, weil ja mehrere Dinge falsch sein können.