Trying to get property 'user_id' of non-object?

Question

[Владислав]

Имеется экшен, который должен удалять запись из БД.
Он принимает id записи, посланный методом post.
После вызова, возвращает ошибку, однако метод delete() срабатывает.
В чём ошибка?

/**
     * @return string
     *
     * Delete vacancy action
     */
    public function actionDelete()
    {

        /* @var $currentUser User */
        $currentUser = Yii::$app->user->identity;

        if(Yii::$app->user->isGuest || $currentUser->isUser()) {
            return $this->goHome();
        }

        $model = new Vacancy();

        Yii::$app->response->format = Response::FORMAT_JSON;

        // получаем id поста, на который пожаловались
        // по методу post
        $id = Yii::$app->request->post('id');

        // находим вакансию
        $vacancy = $model->getVacancyById($id);

        // на всякий случай, проверяем, является ли
        // пользователь автором вакансии
        if($currentUser->getId() != $vacancy->user_id ) {
            return $this->goHome();
        }

        if($vacancy->delete()) {
            return $this->redirect(Url::to(['/user/profile/view', 'id' => $currentUser->getId()]));
        }

        return [
            'success' => false,
            'text' => 'Error',
        ];
    }

Метод getVacancyById

/**
     * @param $id
     * @return array|null|\yii\db\ActiveRecord
     *
     * get vacancy by id
     */
    public function getVacancyById($id)
    {
        return $this->find()->where(['id' => $id])->one();
    }

Answer 1

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

Trying to get property 'user_id' of non-object?

Вы пытаетесь обратиться к свойству объекта, но это не объект. Вероятно тут: $vacancy->user_id Скорее всего у Вас в $vacancy содержится null, и соответственно Вы пытаетесь сделать null->user_id Так что прежде чем обращаться к свойству стоит делать проверку, а объект ли в $vacancy или не равно ли оно null, а иногда и проверять инстанс какого именно класса в переменной.

Answer 2

[Дмитрий]

Добрый день.
return self::find()->where(['id' => $id])->one();

Comments

[Владислав]

Здравствуйте, возникает та же ошибка, ничего не поменялось

[Дмитрий]

Владислав, посмотрите, что у Вас вообще попадает в $vacancy?

[Дмитрий]

Владислав, где находится метод getVacancyById()?

[Владислав]

slo_nik, приходит null

[Дмитрий]

Владислав, ну вот. Проверяйте, приходит ли нужный id в post запросе.
И где, всё таки, находится этот метод?
Вместо self подставьте модель.

[Владислав]

slo_nik, метод находится как раз таки в модели, унаследованной от Active Record, в которой я и хочу произвести поиск.

И ещё - у меня есть абсолютно идентичный код, но с другой таблицей, всё идентично, даже методы getVacancyById и getResumeById идентичны. Но вот с таблицей resume всё работает

Answer 3

[Дмитрий Ким]

// находим вакансию
$vacancy = $model->getVacancyById($id);

// здесь вы должны удостовериться, что вакансия по заданному ID найдена
// а вы сразу пытаетесь получить атрибут user_id, при том что $vacancy в тот момент может быть NULL

// на всякий случай, проверяем, является ли
// пользователь автором вакансии
if($currentUser->getId() != $vacancy->user_id ) {
    return $this->goHome();
}

PS: можно сделать так:

public function getVacancyById($id)
{
  $vacancy =  $this->find()->where(['id' => $id])->one();
  if (!$vacancy){
    throw new NotFoundHttpException('Вакансия не найдена');
  }
  return $vacancy;
}

Comments

[Владислав]

{"name":"Not Found","message":"Вакансия не найдена","code":0,"status":404,"type":"yii\\web\\NotFoundHttpException"}

Да, приходит null

[Дмитрий Ким]

Владислав, вопрос решен?