CODEWITHSUNDEEP
laraveleager-loadinglaravel-3
Meddie
Meddie

Reputation: 557

Laravel eager loading result

I'm having trouble with eager loading in laravel

I have these models:

class Page extends Eloquent {

    public function translations() {
        return $this->has_many('Pagestl', 'page_id');
    }
}

and 

class Pagestl extends Eloquent {

    public static $table = 'pages_tl';

    public function page() {
        return $this->belongs_to('Page', 'id');
    }
}

I want a specific page with its translation data of a specific language. I retrieve the data like this:

$page_data = Page::with(array('translations' => function($query) {
    $query->where('lang_id', '=', 'nl')->first();
}))->find($id);

De result is ok-ish. I get all the page data and the translation of 1 language, dutch (nl). But in order to get a field from the language data I have to this:

$page_data->translations[0]->attributes['content_or_whatever'];

..which i find ugly. I feel i should only have to do something like:

$page_data->translations->content;

..but that gives me an error (Trying to get property of non-object).

Am I missing something or is this just the way it is?

Upvotes: 0

Views: 426

Answers (1)

kJamesy
kJamesy

Reputation: 6233

I don't find it necessary to do eager-loading here.

Both ways of doing it will result in 2 queries: select the Page with the given id and then find its first translation.

So, I would do: 

$page = Page::find($id);

and then 

$page_data = Pagestl::where_page_id_and_lang_id($page->id, 'nl')->first();

Then, assuming the content field exists in pages_tl table, we'll get it by doing:

echo $page_data->content;

Finally, your code looks alright; you don't have to have the second parameter when defining the relationships unless they are different from what Laravel expects.

I'll let you find where the problem is with your code (I couldn't figure out from the information given - could be something to do with strings.php)

Upvotes: 1

Related Questions