Laravel null object hasMany relationship

Question

SOLVED i should not have called out my attributes in the models. For some reason this was keeping it from working.

I have the following structure of two models, Listing and Image. Table Images has FK listing_id that references id on table Listing.

My Eloquent Model for Listing:

namespace App\Models;

use Illuminate\Database\Eloquent\Model as Model;

class Listing extends Model {

    protected $table = 'listings';
    protected $connection = 'mysql';

    public $id;
    public $name;
    public $value;
    public $itemDescr;
    public $user_id;
    public $category_id;

    protected $fillable = [
        'id',
        'name',
        'value',
        'itemDescr',
        'user_id',
        'category_id'
    ];

    /**
     * Return images
     *
     * @return \Illuminate\Database\Eloquent\Relations\HasMany
     */
    public function images()
    {
        return $this->hasMany('App\Models\Image');
    }

}

My Eloquent Model for Image:

namespace App\Models;
use Illuminate\Database\Eloquent\Model as Model;

class Image extends Model
{
    protected $table = 'images';
    protected $connection = 'mysql';

    public $id;
    public $imageType;
    public $title;
    public $filename;
    public $path;
    public $author_id;
    public $listing_id;
    public $user_id;
    public $size;
    public $width;
    public $height;


    protected $fillable = [
        'imageType',
        'title',
        'filename',
        'path',
        'author_id',
        'listing_id',
        'user_id',
        'size',
        'width',
        'height'
    ];

    /**
     *
     * @return \Illuminate\Database\Eloquent\Relations\BelongsTo
     */
    public function listings()
    {
        return $this->belongsTo('App\Models\Listing', 'listing_id');
    }

}

My controller:

/**
 * Return the full list of listings
 *
 * @param Request $request
 *
 * @return string
 */

    public function itemList(Request $request)
    {
        $listings = Listing::with('images')->get();

        return $listings;

    }

I cannot seem to return my listing WITH the image objects -- they are always null. Here is a sample response:

{ id: 6, name: "rare listing item", value: "100", itemDescr: "this is a descr", user_id: 1, category_id: 6, created_at: "2016-05-30 13:47:33", updated_at: "2016-05-30 13:47:33", images: [ ] }

The database shows two images with listing_id of 6 for this particular item. I cannot figure out what could be wrong, I've tried nearly all suggestions. Any ideas?

It's a silly question, but is your Model at App/Models? Your code seems to be ok! It should work... — Laerte
– Laerte, Commented May 31, 2016 at 17:54
@Laerte yes. I dumped the sql its running, and I think my listing.id is going in as null to the statement, which is puzzling to me. — caro
– caro, Commented May 31, 2016 at 18:00
I think your relationships are backwards. You say there are two images with listing id of 6. That means listings can have many images. — user1669496
– user1669496, Commented May 31, 2016 at 18:07
@user3158900 this is true, but on Listing I have the method saying hasMany with Images? Is that incorrect? — caro
– caro, Commented May 31, 2016 at 18:09
Try to remove those definition of attributes on your model, just curious why you do that? If you intended to make it visible then use $visible property. — Rifki
– Rifki, Commented May 31, 2016 at 18:20

Rifki · Accepted Answer · 2016-05-31 18:36:40Z

1

In Laravel table columns and their values is held in $attributes property, Laravel uses __get() magic method to get the $attributes data. By defining public $listing_id as property on your model it just like telling Laravel: "Hey this is the listing_id and not the one that held $attributes property so please return this instead".

When you define the relationship:

return $this->belongsTo('App\Models\Listing', 'listing_id', 'id');

The public $listing_id property will be returned which is the value is null. Remove those public property definition and it should be works.

answered May 31, 2016 at 18:36

Rifki

3,5601 gold badge21 silver badges24 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Laravel null object hasMany relationship

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related