problem with server and client validation #1511

Closed
aohorodnyk opened this Issue Oct 1, 2012 · 8 comments

Projects

None yet

4 participants

@aohorodnyk

Здравствуйте.

Есть такая форма:
https://gist.github.com/3813044

Во первых не понятно как определить класс для input-error, это я решил костылем (если есть встроенные средства сообщите плз).

Во время клиентской валидации в строке 23 добавляется parent-error, всё хорошо.
Во время серверной валидации отмечается строка 25 классом error (или как я переопределил в проблеме 1).
Но когда человек отправил на сервер и получил в ответ ошибку в поле EMail, после чего исправил ошибку и клиентская валидация уберает все классы и ошибки (то есть поле должно перестать быть отмечено крассным и текст ошибки должен быть убран), текст ошибки уберается - всё хорошо, а поле остается быть красным.

Я уверен что это бага, увы прошлый тикет вы по какой-то причине закрыли... (#1509)

@creocoder
Contributor

Да, баг подтверждается. Только править тут нужно js, а не CHtml.

@samdark samdark was assigned Oct 1, 2012
@cebe
Member
cebe commented Oct 1, 2012
  1. есть CHtml::$errorCss и CHtml::$errorMessageCss. э́то поня́тно.
@aohorodnyk

ну как-то уныло... всётаки удобней будет, если передаешь как параметр ActiveForm...

@aohorodnyk

спасибо за подтвержденный баг, то что не в CHtml это я знаю, я там сделал одну удобную вещь и одну правку (как я думал будет лучше).

По поводу JS я смотрел, но как-то красивого решения пока не нашел, если смогу найти то вышлю пул реквест. Отпишите плз здесь как исправите.

заранее благодарен!

@creocoder
Contributor

но как-то красивого решения пока не нашел

Вам и не нужно этого делать, думать над красивым решением позвольте разработчикам фреймворка.

@creocoder
Contributor

@cebe I will make normal description for this bug tomorrow.

@aohorodnyk

Вам и не нужно этого делать, думать над красивым решением позвольте разработчикам фреймворка.

грубовато, но в целом правда...

@creocoder
Contributor

@DropSQL При детальном рассмотрении стало ясно, что проблемы никакой нет. Всё решается на уровне css. Проблема исправлена.

@cebe On closer examination, it became clear that the issue does not exists. Problem can be corrected at css level.

@samdark samdark closed this Oct 3, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment