Баг в CHtml

[sashaYY]

https://github.com/yiisoft/yii/pull/2109

[anton44eg]

в каких целях вы передаете array?

[sashaYY]

Я не передаю, но сегодня зашел в гости на сайт ко мне сканер какой-то и начал пихать в $_POST значения массивы.
Форма валидацию не прошла, но вот notice кидаются когда вызывается CHtml::activeTextField($model,$attribute) и т.п.

[sashaYY]

Молодец Sam, тоже думает что я array пихаю.

[anton44eg]

ну тут проблема совсем не в CHtml, а в фильтрации данных

[sashaYY]

Подскажи тогда встроенный в framework фильтр.

[anton44eg]

CStringValidator? Как у вас массив вообще валидацию прошел

[sashaYY]

Я не передаю, но сегодня зашел в гости на сайт ко мне сканер какой-то и начал пихать в $_POST значения массивы.
Форма валидацию не прошла, но вот notice кидаются когда вызывается CHtml::activeTextField($model,$attribute) и т.п.

Валидатор на то и валидатор, что он проверяет, а не фильтрует.

[sashaYY]

да и еще валидаторы тоже notice кидают, если массив пришел, проверил на CNumberValidator,CStringValidator возможно другие тоже.

[Антон Смирнов]

Я не очень понял, почему решение проблемы:
$text = is_array($text)?null:$text;

Мне кажется лучше так:
$text = (string) $text;

[sashaYY]

Включи вывод всех ошибок в php.ini

error_reporting = E_ALL & ~E_DEPRECATED & ~E_STRICT
display_errors = On
display_startup_errors = On

и посмотри что будет, если сделать так
$text = array();
$text = (string) $text;





и посмотри , будет notice.

[samdark]

Надо валидаторы фиксить, а не CHtml.

Кастовать в строку нельзя потому как 5.4 будет ругаться.

[sashaYY]

Согласен, просто сразу не них не обратил внимания.

P.s у меня php 5.4.10

[sashaYY]

Кинуть pull или сам?

[samdark]

Лучше pull. У меня релиз Stay.com завтра-послезавтра.

[Антон Смирнов]

Не знал, что в php 5.4 будет ругаться. Сам до сих пор на 5.3.

[sashaYY]

Закинул https://github.com/yiisoft/yii/pull/2111
CHtml::encode тоже поправил, т.к при ошибках валидации он выводить полученное значение.