Комментарии на: Ko3: добавляем условные правила валидации

От: Random

Random — Fri, 22 Jan 2010 05:50:28 +0000

По мне так вполне логичных подход. Ведь любом учебнике “для чайников” русским по белому обязательно написано: “Всегда проверяйте данные, получаемые от пользователей”. И если кое-где можно схалявить, то благодаря Woody Gilk в Kohana это делать обязательно

От: alweb

alweb — Fri, 22 Jan 2010 02:10:25 +0000

04:57 22.01.2010, Woody Gilk, Kohana v3.x: Activity
The way validation works is clearly defined… As I said numerous times:

Any field that does not have a filter, rule, or callback is removed.

This is the way it works and is the way it has always worked. The reasons why have been clearly stated… I’m sorry you think it is wrong, but tough luck.

Вот ведь неугомонный какой

От: alweb

alweb — Tue, 12 Jan 2010 14:27:32 +0000

Спасибо, Иван
читал уже, и не раз
и Sprig разобрал уже “по винтикам”…

От: BIakaVeron

BIakaVeron — Tue, 12 Jan 2010 06:39:03 +0000

Sprig использует другой принцип работы, там не требуется запускать валидацию вручную (читаем). Просто берем create() или update() и ловим Validate_Exception с ошибками.

От: alweb

alweb — Mon, 11 Jan 2010 16:57:03 +0000

в целом вижу выход только один…
использовать функции работы с массивами после валидации, дабы воссоздать целостность данных перед передачей в модель перед сохранением данных.

От: alweb

alweb — Mon, 11 Jan 2010 15:09:35 +0000

не получается что-то…
если данные не проходят валидацию, то имеем
Validate_Exception [ 0 ]: Failed to validate array
я работаю со Sprig, а там у Создателя по-проще малость чем в ORM… при этом метод check() возвращает массив данных (зачем??? ). Ну а если валидация не проходит, то:
$data = Validate::factory($data);
…
…
…
if ( ! $data->check())
{
throw new Validate_Exception($data);
}
мда…

От: alweb

alweb — Mon, 11 Jan 2010 11:46:16 +0000

Спасибо, Иван. Я попробую.

От: BIakaVeron

BIakaVeron — Mon, 11 Jan 2010 09:58:22 +0000



// загружаем данные в модель
$model->values($_POST);
// проверяем данные
if ($model->check()) {...}

От: alweb

alweb — Mon, 11 Jan 2010 09:55:00 +0000

очень просто…
у меня как раз такой случай.
форма передает данные для 3х моделей.
у некоторых полей в модели нет правил, фильтров, колбэков.

Я считаю, что передача данных в модель должна происходить с использованием всего лишь одного метода.
Или должен быть механизм автоматической передачи данных в модель, причем как с правилами, так и без правил…

Иначе придется писать дополнительную обработку по добавлению переменных в объект модели, использовать всякие array_intersect’ы и тд и тп…
а это уже как-то не swift…

От: BIakaVeron

BIakaVeron — Mon, 11 Jan 2010 05:51:34 +0000

Не Вы первый негодуете, и не Вы последний… На самом деле я есть смысл фильтровать лишние переменные при проверке. Представьте себе, что форма содержит данные для нескольких моделей (такое бывает). Зачем загружать все переменные в объект Validate и пытаться проверять их? И даже если поле существует в модели, зачем неявно загружать его в нее? Каждый должен заниматься своим делом, и Validate всего лишь проверяет на соответствие правилам. Объясните, почему данный класс должен обрабатывать какие-то посторонние переменные, которые в нем не описаны?