Почему не работает метод save()?

Question

[Vladislav]

У меня есть такой метод:

public function beforeSave($insert)
    {
        if (parent::beforeSave($insert)) {

            $this->password = Yii::$app->getSecurity()->generatePasswordHash($this->password);

            return true;
        }
        return false;
    }
public function afterSave($insert, $changedAttributes)
    {
        # Добавляем пользователю группу пользователя
        $auth = Yii::$app->authManager;
        $user_group = $auth->getRole('user');
        $auth->assign($user_group, $this->id);
        parent::afterSave($insert, $changedAttributes);
    }

Перед сохранением генерируем пароль для пользователя.

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

$model = Users::findOne(['id'=>1]);
$model->name = 'New name';
$model->save();

$model->save() - возвращает false.
$model->validate() - возвращает false;
$model->getErrors() - возвращает пустой массив.

Вот сам процесс изменения поля, изменяю баланс:

//$user = Users::findOne(['id'=>Yii::$app->user->identity->id]);
$user = Users::find()->where(['id' => Yii::$app->user->identity->id])->one();
        $user->balance = ($user->balance - $cost);
        
        if (!$order->save() || !$user->update()) {
            $response = ['code' => -99, 'text' => 'Произошла неизвестная ошибка, обратитесь в тех поддержку, код ошибки: -99'];
            $this->printAPI($response);
        } else {
            $response = ["code" => 5, "text" => "Заказ №" . $order->id . " успешно создан! ", 'balance' => $user->balance];
            $this->printAPI($response);
        }

В чем может быть проблема? Почему не сохраняет, я грешу на методы after и berfore save.

Comments

[v3shin]

Возможно, это все не боевая версия скрипта, но на всякий случай замечу: при каждом успешном обновлении данных пользователя у него меняется пароль и происходит попытка добавить к нему роль user. Оно так и задумывалось?

[Vladislav]

v3shin, так у меня стоит проверка, если это новая запись, то генерируем пароль:
if (parent::beforeSave($insert)) {

[v3shin]

Владислав, параметр $insert в beforeSave() указывает на добавление новой записи, проверять следует его. =)

Answer 1

[Максим]

У вас не проходит валидация. Смотрите правила валидации. Выполните в коде var_dump($model->errors) перед формой. И смотрите ошибки. Если у вас нет ошибок одной модели - смотрите другую. Проверить валидация ли это или нет можно легко. $model->save(false)

Comments

[Vladislav]

Там пусто, говорю же...
$model->errors - пусто
$model->save() - false
$model->update() - false
$model->getErrors() - пусто
$model->validate() - false

[Максим]

Владислав, Проверить валидация ли это или нет можно легко. $model->save(false)

[Vladislav]

Максим, Все получилось, передав false в save.
но методы after и before все равно срабатывают. И получается такая ошибка:
https://vk.cc/awwu60

[Максим]

Владислав, пользователь с id 1 уже существует. Читайте ошибки

[Vladislav]

Максим, можно сделать костыль, типа если текущий пользователь не гость. то не выполняет after и before а просто выходим из него

[Vladislav]

Максим, Максим, я в курсе. он пытается выполнить метод afterSave. и получается ошибка. т.к права такому пользователю уже выданы.
надо просто изменить поле у записи, без вызова after & before

[Максим]

Владислав, after и before save и так являются костылем) Можете сделать, но Лучше вынести все из этих методов и не использовать их.

[Vladislav]

Максим, почему ? А как мне тогда при регистрации делать дефолтную роль пользователю?
или вместо after вызывать свой метод после записи, передав туда id к примеру и там назначать роль да ?

[Максим]

Владислав, ну либо в контроллере, либо в сервисе при регистрации.

Можно и так)

[Vladislav]

Максим, Понял спасибо, а теперь с основной проблемой, почему при false запись обновляется, а без false ошибка валидации. И в массиве ошибок пусто...

[Максим]

Владислав, я же говорю, что где-то есть ошибка валидации. False отключает проверку валидации. Смотрите обязательные поля. Тип полей.

[Vladislav]

Максим, я нашел ошибку.
у меня есть такой валидатор:
rules:
[['password', 'passwordTwo'], 'validatePassword'],
валидатор:

public function validatePassword($attribute, $params)
    {
        
        if (mb_strlen($this->password) < 8) {
            $this->addError($attribute, 'Длина пароля должна быть более 8 символов');
        }
        if ($this->password != $this->passwordTwo) {
            $this->addError($attribute, 'Пароли не совпадают');
        }
    }

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

[Максим]

Владислав, ну поздравляю) Я вам с самого начала сказал ошибка валидации)

[Vladislav]

Максим, так почему он срабатывает, если я не меняю пароль. мне только 1 поле обновить нужно при чем тут пароль ((

[Максим]

Владислав, я же говорю. Избавляйтесь от этих методов. Вы пароль сохраняете каждый раз при любом сохранении/изменении модели. Изменим имя - пароль уже Новый.

[Vladislav]

Максим, Понятно спасибо, только как тогда продумать другой вариант валидации пароля на lenght и сравнение. js не вариант, можно обойти. а вызвать из контроллера этот метод можно ?

[Максим]

Владислав, создайте сервис PasswordHasher и вызывайте его в контроллере или UseCase

В контроллере это может выглядеть так:

$user->password =  $this->hasher->hash($form->password)

Внутри хэшера валидируйте и вкидывайте исключения.

Answer 2

[Иван Орлов]

1. Нельзя писать

$this->password = Yii::$app->getSecurity()->generatePasswordHash($this->password);

В методе after/beforeSave(), потому что первый раз предположим $password будет нормальным, а потом вытащив юзера из базы его password уже будет хешем, и сохранив юзера повторно - уже сгенерируется новый хеш пароля из старого хеша пароля, и так далее, короче после второго сохранения пароль пользователя будет совершенно непригодным.

2. Нельзя использовать $user->update() в конструкции if, потому что $user->update() возвращает не true/false, а количество изменённых строк в БД, но в Yii если модель не изменилась - ничего и не будет обновлено в базе(зачем лишний запрос), просто пишите $user->save()