Как работает наследование правил валидации в yii2?

Question

[Дмитрий]

Добрый день.
Возник такой вот вопрос, как работает наследование?
Есть две модели, родительская и дочерняя.
В дочерней модели определены дополнительные свойства - масса и объём. Так же объединяется массив правил валидации и массив сценариев с родительскими массивами.
Дочерняя модель.

public $mass;
    public $capacity;

    public function rules()
    {
        return ArrayHelper::merge(parent::rules(),[

            [['mass', 'capacity'], 'number', 'on' => self::SCENARIO_CALC_FORM],
            
        ]);
    }

    public function scenarios()
    {
        $scenarios = parent::scenarios();
        $scenarios[self::SCENARIO_CALC_FORM] = ['mass', 'capacity'];
        return $scenarios;
    }

Вот при таком раскладе не идёт ajax валидация для этих полей в форме, только после отправки формы или проверки других полей формы, в ответе сервера видно, что поля валидируются, но сообщения не выводятся.

Если же эти поля, массы и объёма, добавить в правила валидации родительской таблицы, то ajax валидация работает без проблем. Но эти поля не нужны в родительской, незачем их там определять.
Вот и вопрос, что не так с наследованием? Как правильно наследовать правила валидации?

p.s. В родительской модели никакие сценарии не используются.

p.s.s var_dump правил валидации дочерней модели

Comments

[Антон Иванов]

Покажите var_dump того, как мержутся массивы в rules().

[Дмитрий]

Антон, вот, все правила валидации дочерней модели.

Answer 1

[Boris Köln]

Еще надо в метод attributes() аналогично добавить ваши поля.

Comments

[Дмитрий]

Добрый день.
Пробовал - не помогает. Сообщения об ошибках вижу только в консоли браузера. В форму они не подставляются. Ни при отправки формы, ни при ajax валидации.

[Дмитрий]

Только если я добавлю эти поля в родительскую модель всё работает. И ajax и обычная валидация. Но эти поля там совершенно ни к чему.

[Boris Köln]

Покажите view.

[Дмитрий]

Boris Korobkov, view формы

[Boris Köln]

Проверил. Наследование валидаций работает.

Попробуйте временно убрать сценарии.

[Дмитрий]

Boris Korobkov, убрал, вроде работает. Но сценарии нужны, одна форма с набором разных свойств модели, в зависимости от выбора списка "cargoData". И поэтому надо сценарии применять.

[Boris Köln]

в зависимости от выбора списка "cargoData"

Тогда правильнее не сценарием, а 'when' и 'whenClient'

[Дмитрий]

Boris Korobkov, спасибо, попробую переписать код. О результатах доложу)))
Была мысль о том, что mass и capacity скрыты css и поэтому не валидируются как положено. Сделал эксперимент, скрыл поле message при помощи jquery, добавил в валидатор required и сообщение всё равно появлялось на месте скрытого поля.
Просто для меня загадка, почему идёт валидация только после отправки формы и не подставляются сообщения? А если эти два поля прописать в родительской модели - всё работает.

[Дмитрий]

Boris Korobkov, вообще-то это две формы, которые валидируются одна за другой.
В первой надо выбрать маршрут(откуда/куда) и в зависимости от выбора в списке провести валидацию полей, в данном случае mass и capacity(там есть и другие, но до них ход пока не дошёл). Тут используется первый сценарий - SCENARIO_CALC_FORM.
Если первая форма не выдаёт ошибок валидации, то открывается вторая, где надо провалидировать два поля - имя и способ связи с пользователем(телефон или email.) Тут используется второй сценарий - SCENARIO_TOP_FORM.
И всё бы ничего, но вот эти два поля, mass и capacity ломают всю картину.

[Дмитрий]

Тогда правильнее не сценарием, а 'when' и 'whenClient'

Boris Korobkov, и чем это будет правильней? Чем сценарий отличается от этих свойств валидатора?

[Boris Köln]

Если одно поле зависит от другого той же модели - "when", если другой - "scenario".
Тогда сценарий надо передавать (например, в GET-параметре URL) в валидацию.

[Дмитрий]

Boris Korobkov, значит, если одна и та же модель - when или whenClient, если другая модель $scenarios?
Упустил это в документации.
Проверил, вроде работает. Проверю ещё раз, надеюсь, что всё будет работать.
Пока правила сделал такими:

['mass', 'number', 'when' => function(){
                if($this->cargoData == 'mv'){
                    return $this->mass;
                }
                return false;
            }],

            ['capacity', 'number', 'when' => function(){
                if($this->cargoData == 'mv'){
                    return $this->capacity;
                }
                return false;
            }],

[Дмитрий]

Boris Korobkov, переписал правило таким образом, но не уверен, что допустимо так возвращать результат(я о return)

[['mass', 'capacity'], 'number', 'when' => function(){
                if($this->cargoData == 'mv'){
                    return $this->mass || $this->capacity;
                }
                return false;
            }],

Тогда сценарий надо передавать (например, в GET-параметре URL) в валидацию.

Пример можете показать, если это Вас не затруднит.

[Boris Köln]

when или whenClient

Не "или", а "и".
when - для серверной валидации, whenClient - для клиентской валидации.

Но для конкретной проверки "number" не нужно не when, ни scenario. А для "required" - нужно.

[Дмитрий]

Boris Korobkov, и есть ещё один вопрос, который связан с этим вопросом. Посмотрите, может есть идея, как это реализовать? Сейчас я это реализовал дичайшим говнокодом, что аж показывать стыдно)))

[Boris Köln]

[['mass', 'capacity'], 'number'],

[['mass', 'capacity'], 'required',
'when' => function(){ return $this->cargoData === 'mv'; },
'whenClient' => "function (attribute, value) { return $('#cargo-data').val() == 'mv'; }",
],

[Дмитрий]

Но для конкретной проверки "number" не нужно не when, ни scenario. А для "required" - нужно.

Boris Korobkov, значит, для всех полей, которые я проверяю на корректность ввода пользователем, нужно убрать сценарии совсем? Применять только если я проверяю required?

[Дмитрий]

Boris Korobkov, и ещё вопрос, всё для этой же формы.
Пока все проверки касались только одной модели. Но при значении cargoData == ''xyz" выводятся поля другой модели, где надо указать параметры груза(ширина, длина, высота, масса, количество). Это уже другая модель - Dimension. Этих параметров может быть несколько, добавляются динамически. В этом случае уже надо использовать валидацию динамических полей? Как её увязать с ajaxValidation для модели Calculate?

[Boris Köln]

В этом случае уже надо использовать валидацию динамических полей?

Почему динамические поля то? Они же статические (ширина, длина, высота, масса, количество).
Если этих моделей много - это ничего не меняет. Каждую валидируйте отдельно.

https://github.com/yiisoft/yii2/blob/master/docs/g...

[Дмитрий]

Boris Korobkov, раз не динамические, то проблем с обычной валидацией нет. А вот как быть с ajajx?
В post запросе придут две модели, одна из которых будет массив моделей. Как это всё разрулить в одном действии контролера?
Если бы были просто две модели, то можно бы было записать так
ActiveForm::validate($model1, $model2);
Но если вторая модель является массивом, то надо же отдельно, в цикле её валидировать, а потом объединить каким-то образом с валидацией первой модели. А вот как?

[Дмитрий]

а потом объединить каким-то образом с валидацией первой модели

Boris Korobkov, с этим разобрался, использовать надо ArrayHelper::merge(), с его помощью собрать массив ошибок двух моделей и вернуть клиенту.

Почему динамические поля то? Они же статические (ширина, длина, высота, масса, количество).

А вот с этим проблемы. Первый набор параметров всегда в форме, только скрыт при помощи css. Но есть возможность добавить ещё несколько раз этот набор, при помощи jquery(кнопка +/-), и значит, что это уже динамические поля?

[Boris Köln]

Это динамические строки, а не динамические поля (столбцы).

[Дмитрий]

Boris Korobkov, но обычная валидация не подойдёт?
По тому принципу, что Вы написали, валидация не работает как надо. Если я делаю два набора параметров, то при возникновении ошибок текст ошибок подставляется для первого и второго набора. Хотя валидирую только первый набор.

Answer 2

[LAV45]

А чем простой array_merge() не устраивает?

public function rules()
{
    return array_merge(parent::rules(), [
// ...
    ]);
}

Comments

[Дмитрий]

Можно и array_merge(), но это не влияет на работу валидации. А меня интересует, в первую очередь, вопрос валидации, как она работает и как происходит наследование правил и сценариев.
Так что в данной ситуации без разницы, что я использую.

[LAV45]

slo_nik, ты можешь описать список полей которые будут валидироватся в методе `scenarios()` или добавить к некоторым валидаторам параметр `'on' => self::SCENARIO_CALC_FORM`
это уже как тебе удобнее
Но на практике проще описать список полей в `scenarios()`

[Дмитрий]

LAV45, что я и сделал. Посмотрите код в вопросе.