Не правильно работают связи?

Question

[javanub]

Пытаюсь вывести связанные таблицы, новости и страны

$news = NewsModel::with('country')->paginate(1);
return view('welcome', ['news' => $news]);

Получаю нормально связанный json


В моделе новости, на всякий пожарный, вот так:

public function country()
{
     return $this->belongsTo(CountryModel::class, 'country', 'id');
}

Но когда делаю, так, то не работает: Trying to get property of non-object

@foreach($news as $item)
    {{ $item->country->country }}
@endforeach

Comments

[Ramm]

попробуйте {{ $item->country()->country }}

[javanub]

Ramm: Undefined property: Illuminate\Database\Eloquent\Relations\BelongsTo::$country

[Антон Шаманов]

может твой шаблонизатор преобразовывает объект в массив? проверь типы переменных

[javanub]

Антон Шаманов: blade по дефолту, {{ gettype($news) }} выдает что это object, ну собственно на картинке видно, что это объект, а происходит дичь какая-то :)

[holfza]

javanub: dd($item->country) чего показывает?

[javanub]

holfza: показывает 1, 1 это ячейка country в таблице news. Вот скрин с бд images.vfl.ru/ii/1481019647/6464c36a/15230472.png

[holfza]

javanub: так назовите связь по другому

[holfza]

javanub: Магический метод сработает только при чтении данных из недоступных свойств. А свойство country у вас есть в модели news. Поэтому надо связь обозвать по другому.

[javanub]

holfza: хм, переименовал метод в моделе новости на get_country и выдала вот такую интересную ошибку: Call to undefined relationship [country] on model [App\NewsModel].

[holfza]

javanub: значит не везде поменяли.. $news = NewsModel::with('country')->paginate(1); - тут тоже.. и во вьюхе.. а вообще лучше поменять в таблице, на country_id

[javanub]

holfza: Блин, я 3 дня е*ался, а надо было переименовать метод, чтобы он отличался от ячейки, теперь все работает. Спасибо огромное.

[holfza]

javanub: А еще почитайте, как правильно именовать методы/свойства/ячейки в бд) Странно вообще выглядит: $item->country->country .. Куда логичней $item->country->name или $item->country->title

[javanub]

holfza: я самоучка, не знаю гайдов программистов. Спасибо, еще раз большое. Сейчас так и сделаю, обязательно переименую.

[holfza]

javanub: Напишите вашу ошибку ниже, и отметьте решением, может людям пришедшим с поиска поможет)

[javanub]

holfza: вы напишите, я сделаю решением.

Answer 1

[javanub]

holfza дал верный совет переименовать метод в модели отличный от названия ячейки в БД.

Answer 2

[Andrzej Wielski]

Если у одного элемента $item нет country, то разумеется упадет с ошибкой.
Делайте проверку до вывода:
{{ $item->country ? $item->country->country : ''}}

Comments

[javanub]

дело в том, что есть, там в country только 2 айдишника (1. Россия, 2.Беларусь), а в новостях я сделал seed на 1 айдишник. Тоесть все новости с 1 айдишником идут.

[Andrzej Wielski]

javanub: Перед обращением к объекту $item->country в массиве проверьте, у всех ли элементов есть этот самый $item->country :)
Возможно у одного нет, и все крашится в итоге с ошибкой.

[javanub]

Andrzej Wielski: короче ответ нашелся выше, но это была жесть :)