Почему не работает валидатор Yii2?

Question

[Vladislav]

Модель: Setting

<?php

namespace app\modules\panel\models;

use app\models\User;
use Yii;
use yii\base\Model;

/**
 * Login the model behind the login form.
 */
class Setting extends Model
{
    public $login;
    public $name;
    public $vk;
    public $password;
    public $passwordTwo;

    public function rules()
    {
        return [
            [['login', 'name', 'vk'], 'string'],
            ['password', 'validatePassword'],
        ];
    }
    public function validatePassword($attribute, $params)
    {
        $this->addError($attribute, 'сразу ошибка без проверок');
    }
    
}

Экшен в контроллере:

public function actionSetting()
    {
        if (Yii::$app->request->isPost) {
            $model = new Setting();
            $model->load(Yii::$app->request->post());
            $model->updateUser();
        }
        $dbUser = User::findOne(Yii::$app->user->identity->id);
        $user = new Setting();
        $user->login = $dbUser->login;
        $user->name = $dbUser->name;
        $user->vk = $dbUser->vk;

        return $this->render('setting', compact('user'));
    }

Просто ради проверки поставил встроенный валидатор:

['passwordTwo', 'compare', 'compareAttribute' => 'password']

Он работает. А кастомный не работает... Он вообще ничего не выдает. Не зеленую ни красную обводку:

Comments

[Дмитрий]

Добрый день.
Потому, что он у Вас ничего не делает, а только добавляет ошибку.
Кстати, это тоже бред.

if (Yii::$app->request->isPost) {
            $model = new Setting();
            $model->load(Yii::$app->request->post());
            $model->updateUser();
        }
        $dbUser = User::findOne(Yii::$app->user->identity->id);
        $user = new Setting();
        $user->login = $dbUser->login;
        $user->name = $dbUser->name;
        $user->vk = $dbUser->vk;

[Vladislav]

Дополню тестами:
если сделать:

var_dump($model->validate());
            die;

там будет false
Если закомментировать:

$this->addError($attribute, 'сразу ошибка без проверок');

то там будет true. Все прошел. Но на фронте почему то не выводит ошибок, о том что валидация не прошла

[Vladislav]

Дмитрий, Расскажите пожалуйста как сделать правильно. Мне нужно сделать на сайте настройки пользователя. Пробовал через метод load он не заполняет модель...
И как вывести ошибку на фронт ?

[Дмитрий]

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

[Дмитрий]

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

[Vladislav]

Дмитрий,

$user = new Setting();
        $user->load(User::findOne(Yii::$app->user->id));

[Дмитрий]

Владислав, зачем?

[Vladislav]

Дмитрий, Пользовательский: мне нужно проверить что старый пароль совпадает с текущем паролем. И в поле passwordTwo указывается новый. Крч сделать смену пароля. При это в начале проверить старый пароль что он правильный

[Дмитрий]

Владислав, почитайте сначала документацию и блог, ссылки я дал. Потому будете писать свой код.

[Дмитрий]

Владислав, вот это зачем?

$user = new Setting();
        $user->load(User::findOne(Yii::$app->user->id));

[Дмитрий]

Владислав, Вам надо принять новые данные, а не те, что хранятся в базе.

[Vladislav]

Дмитрий, Но для начала нужно вывести старые. И их менять вот такой вид:

[Дмитрий]

Владислав, ну и что? Выводите. Но так как Вы делаете - ничего не получится.

$user = new Setting();
        $user->load(User::findOne(Yii::$app->user->id));

[Дмитрий]

Владислав, через gii сгенерируйте crud и посмотрите, как правильно будет.

[Gennady S]

Владислав, пользовательский валидатор должен вернуть true / false вместе с добавлением ошибки. Не думаю, что стоит перезаписывать пароль и вообще выводить пользователю старый пароль, его стоит захешировать в одностороннем порядке. И затем обработать ввод / отсутствие нового пароля, что можно сделать разными способами, в том числе в beforeValidate или в контроллере, например: 1) загружаете модель пользователя $user = User::findOne, 2) заменяете данными формы $user->load (если модель формы отдельная, то $user->setAttributes, т.к. load берет вложенную структуру данных, 3) если пользователь не новый, и пароль пуст — заменяете с getOldAttribute, в противном случае сгенерируете хэш для нового пароля — для валидатора пароль будет заполнен.

Так же, есть смысл избежать = new Setting(); дважды. Вы так и так загружаете данные пользователя (не identity ли?), так и так создаете форму Setting. Делаете это в шапке обработчика, проверяете наличие POST, обновляете, что нужно, и возвращайте тот же Setting.

[Vladislav]

Gennady S, В общем рассказываю, всю суть.
Это страница с настройками аккаунта пользователя уже авторизованного т.е он identity.
Там одна форма, модель Setting заполнена моделью User. Чтобы предварительно было видно старые данные.
Там можно поменять логин имя, почту, соц сеть и т.д
Если поле password и passwordTwo пусто. Значит мы не меняем пароль.
Если заполнены. Нужно моим валидатором проверить пароль у меня в модели User есть метод validatePassword:

public function validatePassword($pass)
    {
        return \Yii::$app->security->validatePassword($pass, $this->hashPass);
    }

И получается если старый пароль верный (как бы подтвердили что это вы, а не краденная сессия) то тогда меняем пароль на новый.
И для этого мне нужен был свой валидатор.
___
На счет контроллера я делал сперва:

$user = new Setting();
$user->load(User::findOne(Yii::$app->user->id));

Но он её не заполнял данными, если я правильно понял из ваших слов мне нужно делать: setAttributes вместо load да ?
Потому что делать через $model->prop = $val; да... тупо. но это первое что пришло в голову. Так как не все методы Yii знаю. Пару месяцев на нем пишу, так по тихому и изучаю

[Gennady S]

В общем и целом трудно что-либо сказать. Повторный пароль можно проверить отдельно, в контроллере, остальное штатным образом в модели User. Да, я ошибся, валидатор не должен ничего возвращать. Кстати, почему $user = new Setting(); а не $settings = new Settings? И не понадобится "костыль" $dbUser.

[Дмитрий]

Владислав, да не нужен Вам собственный валидатор!!!
Вы документацию, раздел "Валидация" читали?!! На фиг Вы лепите велосипед?!!

[Дмитрий]

Повторный пароль можно проверить отдельно, в контроллере, остальное штатным образом в модели User.

Gennady S, не советуйте подобного никогда.

[Gennady S]

Дмитрий, почему же, вполне посоветую. А еще посоветую не использовать гроб Yii2 никогда, при всей любви к нему. И никогда не писать вручную велосипед, а развернуть готовый сервис авторизации. И вообще, не мучиться, а перейти на ASP .NET Core / Kestrel.

Вот обожаю такие комментарии знатоков паттернов и подходов. Помилуйте, человек пишет что получится, и вполне может в контроллере хоть крейсер развернуть. И плюнуть на Yii, уложить свой сайт в один PHP файл. Мир не пострадает, опыт получит. А своими руками больше авторизации для визитки ничего не слепит.

А писать можно скромнее и не шуметь про документацию с кучей восклицательных знаков. Может кому-то и захочется зайти в пропиаренный выше блог.

[Дмитрий]

Gennady S, читайте и Вы документацию и блог, ссылки я дал выше.
В документации есть, как проверить данные на совпадение.
В блоге есть рабочие примеры.
Если Владислав лень читать, то это его проблемы. Пусть и дальше изобретает велосипед.

Answer 1

[Богдан Хвалько]

Это $this->addError($attribute, 'сразу ошибка без проверок') сработает только в случае валидацыи формы на сервере. Для динамической валыдацыи ищи в доках инфу.

Comments

[Vladislav]

Ну должно же как работать... Я отправил форму. Нажал сабмит.
На сервере проверка не прошла, и на странице под input появилась ошибка а там ничего не появляется.
я НЕ имею ввиду про ошибки на клиентской стороне. их я и сам на js могу написать. а вот именно после отправки должно появится ошибка