yform icon indicating copy to clipboard operation
yform copied to clipboard

Umgang mit Fehlermeldungen anpassen

Open dergel opened this issue 2 years ago • 6 comments

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 avatar Feb 24 '23 11:02 dergel

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

Rückgabewerte ändern

dergel avatar Feb 24 '23 11:02 dergel

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

dergel avatar Feb 24 '23 11:02 dergel

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

dergel avatar Feb 24 '23 16:02 dergel

Betrifft auch #591 nach meinem Verständnis

AWqxKAWERbXo avatar Feb 28 '23 21:02 AWqxKAWERbXo

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.

tbaddade avatar Mar 01 '23 09:03 tbaddade

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.

AWqxKAWERbXo avatar Mar 01 '23 09:03 AWqxKAWERbXo