Почему не работает $user->save();?

Question

[Ivan Yakushenko]

Код:

public function editdata()
    {

        $user = User::findOne($this->username);
        if($this->email != null)$user->email = $this->email;
        if($this->fio != null)$user->fio = $this->fio;
        if($this->apartment != null)$user->apartment = $this->apartment;
        if($this->house != null)$user->house = $this->house;
        if($this->housing != null)$user->housing = $this->housing;
        if($this->street != null)$user->street = $this->street;
        if($this->phone != null)$user->phone = $this->phone;
        if($this->password != null)$user->setPassword($this->password);

        return $user->save();

    }

Ошибка:

Call to a member function save() on null

Answer 1

[Максим Тимофеев]

findOne() ищет по первичному ключу, по id. поэтому и не находит
надо так:

$user = User::find()->where(['username'=>$this->username])->one()

а что бы не было таких ошибок, добавлять if

public function editdata()
    {
if($user = User::find()->where(['username'=>$this->username])->one()){
        if($this->email != null)$user->email = $this->email;
        if($this->fio != null)$user->fio = $this->fio;
        if($this->apartment != null)$user->apartment = $this->apartment;
        if($this->house != null)$user->house = $this->house;
        if($this->housing != null)$user->housing = $this->housing;
        if($this->street != null)$user->street = $this->street;
        if($this->phone != null)$user->phone = $this->phone;
        if($this->password != null)$user->setPassword($this->password);

        return $user->save();
}
return false;
    }

Comments

[Ivan Yakushenko]

Getting unknown property: app\models\EditdataForm::username

[Максим Тимофеев]

Ivan Yakushenko: Значит у Вас нет в $this атрибута username.

[Ivan Yakushenko]

Максим Тимофеев: перепробовал кучу вариантов - ни один не работает. В некоторых других кусках программы используется $this->getUser(), $this->getId() и функции identity, но тут ни одна из них не работает.

[Максим Тимофеев]

Ivan Yakushenko: $this - это переменная. В ней может содержаться что угодно. В зависимости где Вы ее используете. Если Вы хотите получить username пользователя в моделе USER, то $this->username вернет значение аттрибута username. Но по Вашему вопросу не понятно что это и где это. Распишите и получите ответ.

[Ivan Yakushenko]

Максим Тимофеев: Getting unknown property: app\models\EditdataForm::username

[Максим Тимофеев]

Ivan Yakushenko: в EditdataForm нет атрибута username

Answer 2

[Алексей Уколов]

Потому что User::findOne возвращает null. Смотрите, что у вас в $this->username и действительно ли есть такой пользователь. Судя по коду, вы получаете с клиента изменённый username и пытаетесь найти его в БД.

Comments

[Ivan Yakushenko]

Действительно, у меня же есть форма изменения username и $this->username возвращает именно эти данные. А как тогда можно записать в $user данные о текущем пользователе из таблицы для их изменения? Вариант с User::findOne($this->id); вызывает ошибку Getting unknown property: app\models\EditdataForm::id.

[Ivan Yakushenko]

Удалил форму изменения username, в итоге в username должен храниться текущий залогиненый пользователь и по нему искать в бд информацию запросом $user = User::findOne($this->username);, но ошибка все равно Call to a member function save() on null

[holfza]

$user = User::findOne(Yii::$app->user->getId());
Как-то так наверное

[Ivan Yakushenko]

holfza: Argument 1 passed to yii\web\User::login() must implement interface yii\web\IdentityInterface, boolean given, called in D:\OpenServer\domains\yii2sp\controllers\SiteController.php on line 152 and defined. На 152 строке SiteController
if (Yii::$app->getUser()->login($user))

[Кирилл Арутюнов]

Ivan Yakushenko: а так:
$user = User::findOne(['username' => $this->username]); тоже null возвращает?

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

[Lander]

На сколько я знаю, то findOne (если ему передан не ассоциативный массив) ищет по значению поля, которое указано как PrimaryKey. В стандартной модели User - это поле id, а поле username просто уникальное. Так что либо User::findOne(['username' => $this->username]), либо User::findOne($userId). (Только $userId надо откуда то получать.)

[Lander]

А, да... ещё после вызова $user->save() можно сделать var_dump($user->errors), чтобы посмотреть ошибки валидации, если таковые имеются :)