Ошибка Trying to get property 'name' of non-object ->with() как решить?

Question

[Герман Мартин]

есть запрос вида:

$query = Event::find()->where(['i_user' => $_SESSION['user']['id']])
                    ->with('category')
                    ->with('type')
            ->orderBy($rsort2)

в модели:

public function getCategory() {
        return $this->hasOne(Category::className(), ['id' => 'i_cat']);
    }
    public function getType() {
        return $this->hasOne(Type::className(), ['id' => 'type']);
    }

при выводе возникает описанная в теме ошибка:

<?=$ev['category']->name?></td>
    <?=$ev['type']->name?></span>

причем вывод имени категории отрабатывает без ошибок, а вывод имени типа не видит, хотя функции выборки аналогичны. Была идея просто работать с массивами:

$query = Event::find()->where(['i_user' => $_SESSION['user']['id']])
                    ->with('category')
                    ->with('type')
            ->orderBy($rsort2)
            ->asArray()
            ->all();

но в коде присутствует еще пагинация

Есть идеи?

Answer 1

[VelAnna]

Попробуйте получать данные так:

<td><?= $ev->category->name ?></td>
<span><?= $ev->type->name ?></span>

Comments

[Герман Мартин]

выходит аналогичная ошибка. Если сделать
<span><?= $ev->type ?></span>
выводится значение первого столбца таблицы, а именно id

[VelAnna]

Герман Мартин, а откуда у вас переменная $ev? В приведенном коде этого нет.
Я так поняла, что вы пытаетесь вывести данные в gridView?

[VelAnna]

Герман Мартин, я не вижу ваш код, попробуйте перебрать через foreach

foreach($ev as $item) {
  echo $item->type->name."<br />";
}

[Герман Мартин]

$query = Event::find()->where(['i_user' => $_SESSION['user']['id']])
                    ->with('category')
                    ->with('types')
            ->orderBy($rsort2)
            //->asArray()
            //->all();
        ;
        //echo Debug::d($query,'query'); die;
        $q_counts = 10;
        $pages = new Pagination(['totalCount' => $query->count(),'pageSize' => $q_counts,
                                'pageSizeParam' => false, 'forcePageParam' => false]);
        $events = $query->offset($pages->offset)

   // далее 
   foreach ($events as $ek => $ev): 
   ...
   <td class="item_type">
     <?=$ev->types['color']?>
     <?=$ev->types->name?>
  </td>
   ...
   endforeach;

причем можно разными вариантами получить значение, например: $ev->['types']['color']

решил проблему путем изменения ->with('type') на ->with('types')

Спасибо за подсказки

Answer 2

[vksee]

Преамбула:
Класс Event наследник ActiveRecord.
При обращении к свойству, если в вашем классе в рамках PHP не объявлено доступное свойство type - то вызывается магический метод __get('type') (у родительского класса) для определения существования в рамках Yii атрибута или геттера getType() для получения значения этого атрибута: yii\db\BaseActiveRecord::__get()
- Вначале проверяется наличие атрибута с таким именем, если имеется - возвращается значение атрибута.
- Затем проверяется наличие уже полученных значений по связанным данным. Это, как правило, другой объект наследник ActiveRecord, в вашем случае описано для class Type.
- Если пока ни чего не найдено, проверяется наличие геттера и его вызов, к примеру getType().
- Если геттер вернул не объект запроса данных, то возвращается это значение из геттера.
Методы определения связей (hasOne, hasMany и т.п.) - возвращают объект типа ActiveQueryInterface (как правило это ActiveQuery).
- Если результат геттера возвращает объект типа ActiveQueryInterface, то этот запрос выполняется, и затем возвращается уже результат запроса.
Таким образом мы получаем результат запроса и получаем связанную запись или записи типа ActiveRecord.
Если требуется получить результат запроса связанных данных (объект типа ActiveRecord), то вызывается $ev->type, но если сам запрос для дальнейшей манипуляции - то $ev->getType().

Ответ:
В вашем случае ошибка в том, что до описанной в getType() связи дело не доходит. Потому что есть атрибут type.
Ваш код $ev->type - возвращает "айдишник". А вы к нему обращаетесь затем как к объекту ...->name.

1. Требуется по разному назвать связь и атрибут для хранения связи.
Удобнее сделать так - атрибут назвать type_id (в базе и в правилах):

public function getType() {
    return $this->hasOne(Type::className(), ['id' => 'type_id']);
}

И тогда:
- $ev->type_id - айдишник связанных данных.
- $ev->type - благодаря магии __get() и описанию связи через getType() - вернётся связанный объект типа class Type (наследник ActiveRecord).
- $ev->getType() - запрос ActiveQuery для получения связанных данных.

2. Или переименуйте getType() на что-нибудь типа getEventType(), и тогда сработает $ev->eventType->name.

Comments

[Герман Мартин]

Спасибо, Ваш ответ полный и подробный, решил проблему изменением ->with('type') на ->with('types')
а этот код:

public function getType() {
    return $this->hasOne(Type::className(), ['id' => 'type']); //  вместо более правильного  ['id' => 'type_id']
}

остался из за ошибок при проектировании.

Спасибо!