Yii сложная валидация данных, как реализовать?

Question

[Максим]

1. На одну форму планирую создавать собственные сложные функции для проверки методом валидации. Подскажите как правильно реализовать в YII2? Начал делать прямо в модели, но правил валидации будет слишком много и все данные имеют запросы к базе. И если наследовать класс валидации, то в какую папку помещать правила валидации?
   
2. Подскажите как выводить ошибки валидаци вверху формы, а не под полем, которое проверяем на валидацию. Используя метод ниже получается ошибку вывести только вверху страницы, но не в форме. Форма в модальном окне ajax
   

if ( условие ) {
                Yii::$app->session->setFlash('success', 'При успешном выполнении.');
            } else {
                Yii::$app->session->setFlash('error', 'При ошибке в валидации.');
            }

Answer 1

[Дмитрий]

Добрый вечер.

Начал делать прямо в модели, но правил валидации будет слишком много и все данные имеют запросы к базе.

Показывайте, как Вы это делаете, почему много запросов к базе данных?

Подскажите как выводить ошибки валидаци вверху формы

Вот так:

$form = ActiveForm::begin([
    'id' => $idForm,
]);
echo $form->errorSummary($model);

p.s. Здесь прочтёте, как задать свой css класс для блока ошибок.

Comments

[Максим]

public function validateDateOpen()
    {
        $dateToday = date("d-m-Y");//Текущая дата

        //Если судья имеет категории, то проверяем на срок действия
        if ( $judgeComitets =  $this->idJudge->judgeComitets ) {

            foreach ( $judgeComitets as $judgeComitet ) {
                //Если у выбранного судьи имеется судейская категории выбранного комитета, то узнаём дату сдачи
                if ( $judgeComitet->comitet_id == $this->comitet_id ) {

                    $dateFrom = $judgeComitet->date_from;//Считываем дату сдачи
                    $date = date_create($dateFrom);//Создаем дату для рассчёта срока
                    date_modify($date, '1 year 2 month');//Прибавляем дату к текущей
                    $dateToCategory = date_format($date, 'd.m.Y');//Форматируем дату

                    //Если срок лицензии закончился, то выводим ошибку!
                    if ( $dateToday>$dateToCategory ) {
                        $this->addError('id_judge', 'У выбранного судьи закончился срок действия судейской категории! Срок действия выбранной категории до: '.$dateToCategory.'!');
                    }

                }
            }

            if ( empty($dateFrom) ) {
                $this->addError('comitet_id', 'У выбранного судьи нет судейской категории выбранного комитета!');
            }

        }
    }


    public function validateMaxJugle()
    {
        //Посчитаем общее колличество назначений на текущее событие с учетом статуса "проверяется"
        $aEventCount = Appointment::find()->where(['id_event' => $this->id_event,'status'=>'0','status'=>'1'])->count();
        //Максимальное кол-во судей для назначения по текущеме рангу
        $rangEvent = RangEvent::find()->where(['id' => $this->idEvent->id_rang_event])->one();
        $maxJugle = $rangEvent->col_judge;
        //Если назначений больше, то выдаем ошибку
        if ($aEventCount > $maxJugle) {
            $this->addError('id_event', 'На данное событие нельзя назначить больше '.$maxJugle.' судей!');
        }
    }

[Максим]

slo_nik это только часть данных. Ещё будет примерно 4 таких... Подскажите как улучшить всё это дело.. В принципе всё работает.

reules

['id_event', 'validateMaxJugle'],
            //['id_judge', 'validateActive'],
            ['id_judge', 'validateDateOpen'],

[Дмитрий]

Максим Ворожцов, мало что понял из Вашего кода.
Вы хотите проверить до какой даты действительна лицензия?
И хотите проверить максимальное количество судей?

[Максим]

slo_nik, вот что нужно для валидации

1. Нельзя назначить одного судью на одно и то же событие несколько раз
   
2. Назначить можно только судью, который имеет судейскую категорию текущего ранга
   
3. У судьи должна быть действующая категория (действует 1 год 2 мес) и статус активен
   
4. Судью можно назначать ее более 6 раз в течении в течении срока действия лицензии (1 года 2 месяца)
   
5. Нельзя назначить одного и тоже судью на одни и те же даты на разные события
   
6. Судью можно назначить на одно событие и с разными комитетами
   
7. Категории судей которые допускаются к назначениям привязанные к рангу событий
   
8. Нельзя назначать одного и того же судью на одно и тоже событие 2 года подряд
   
9. VIP судей может быть не более чем указано в настройках
   
10. Максимальное кол-во судей может быть только столько, сколько указано в настройках
    
11. Присутсвие городов и округов считывается из настроек и проверяется. Например, на Рейтинговые события могут быть только 5 судей из разных городов и Округов. Если один из округа уже имеет назначения - выдаём ошибку
    
    В кратце написал сколько валидаций нужно, но возможно добавится. В будущем планирую сделать форму, которая автоматически будет выдавать данные по текущим критериям (связанные списки). Но пока что нужно выдать через валидацию
    
    

[Дмитрий]

Максим Ворожцов,
1) валидатор unique, проверяете по id или ещё по какому-то параметру.
2) валидатор range.
3) в форме не показывать тех судей, у которых срок действия категории истёк.
4) опять же, перед выводом формы проверять этот параметр и не показывать в форме.
....
Это первое, что пришло на ум, чтобы не писать кучу запросов в validator-ах.
Делайте сразу с таблицами и связями, меньше мороки будет с валидацией.

[Максим]

slo_nik Выводить ошибку получилось. А как при этом сделать поля с ошибками красными? Чтобы пользователь видел где исправить данные...

echo $form->errorSummary($model);

[Максим]

slo_nik ещё вопрос: "Как выводить все ошибки в сумме. В моём случае валидация проверяется, но доходит до первого вывода сообщения об ошибки и прекращает валидировать данные. Хочется чтобы сразу все ошибки выводились исходя из выбранных параметров.
Вывожу с помощью

$this->addError('', 'У выбранного судьи нет судейской категории выбранного комитета!');

Возможно использовать нужно "addErrors"

$this->addErrors('У выбранного судьи нет судейской категории выбранного комитета!');

Только как это использовать в $form->errorSummary($model)

[Максим]

slo_nik,

1) валидатор unique, проверяете по id или ещё по какому-то параметру.
2) валидатор range.
3) в форме не показывать тех судей, у которых срок действия категории истёк.
4) опять же, перед выводом формы проверять этот параметр и не показывать в форме.
....
Это первое, что пришло на ум, чтобы не писать кучу запросов в validator-ах.
Делайте сразу с таблицами и связями, меньше мороки будет с валидацией.

Часть функций и так будет реализовано, но почему пошёл путем валидации, потому что для того, кто имеет права Админа валидация не будет произведена. Если запрашивать сразу с такими данными, то не будет возможности исключений для админа, кроме как через саму базу

Сейчас некоторые данные валидируются в рулез

public function rules()
    {
        return [
            [['id_event', 'comitet_id', 'id_judge'], 'required'],
            [['id_event', 'comitet_id', 'id_judge', 'created_at', 'updated_at'], 'integer'],
            [['status'], 'string'],
            [['status'], 'default', 'value' => '0'],
            [['id_event'], 'exist', 'skipOnError' => true, 'targetClass' => Event::className(), 'targetAttribute' => ['id_event' => 'id']],
            [['id_judge'], 'exist', 'skipOnError' => true, 'targetClass' => Judge::className(), 'targetAttribute' => ['id_judge' => 'id']],
            [['comitet_id'], 'exist', 'skipOnError' => true, 'targetClass' => Comitet::className(), 'targetAttribute' => ['comitet_id' => 'id']],

            ['id_event', 'validateMaxJugle'],
            //['id_judge', 'validateActive'],
            ['id_judge', 'validateDateOpen'],
            [['id_event','comitet_id', 'id_judge'], 'unique', 'targetClass' => '\backend\models\Appointment', 'targetAttribute' => ['id_event', 'comitet_id', 'id_judge'], 'message' => 'Такое назначение уже существует!'],
        ];
    }

[Дмитрий]

Максим Ворожцов, используйте scenarios.

[Максим]

slo_nik, это на какой вопрос ответили?) Для чего scenarios использовать? Вместо валидации?

[Дмитрий]

Максим Ворожцов, вместе с валидацией. За счёт сценариев Вы сможете разделить валидацию для админа и для пользователя по отдельности

[Дмитрий]

Максим Ворожцов, и модели разделите, что ещё лучше будет. Одна модель для админки, вторая для пользовательской части. Тогда со сценариями можно не заморачиваться.