Hey Tavicu!

Thanks for your help, it's really appreciated :).

I simply forgot to add the return as you mentioned, as well as inserting the user_id and category_id.

I will test if it works as intended, then I will return here.

I am a little bit concerned about

public function listings() 
{       
    return $this->hasMany('Listing', 'user_id');
}

if it does not confuse, because the method name is singular, even though it returns hasMany results

The function name must be plural so listings it's ok ($user->listings)

The hasMany just call a Model, Listing, which needs to be singular :) ($listing->user)

But yes, in Listing model functions needs to be singular since is belongsTo (listing istead of listings and user instead of users)

But this is just aesthetic.

delmadord said:

I am a little bit concerned about

public function listings() 
{       
   return $this->hasMany('Listing', 'user_id');
}

if it does not confuse, because the method name is singular, even though it returns hasMany results

I managed to fix it with your help!

So to any other people with similar issues, always double-check your models as well as your controllers.

My issue was that I didn't save the user_id to the database along with the other values when I submitted my form.

So now I am simply doing:

$listing->user_id = Auth::user()->id;

However as a side question, is there a better way to achieve this?

Just to modify the default model function you are using to create a new listing and add by default the current user id.

Reached said:

I managed to fix it with your help!

So to any other people with similar issues, always double-check your models as well as your controllers.

My issue was that I didn't save the user_id to the database along with the other values when I submitted my form.

So now I am simply doing:

$listing->user_id = Auth::user()->id;

However as a side question, is there a better way to achieve this?

Can you be a bit more precise? :)

Tavicu said:

Just to modify the default model function you are using to create a new listing and add by default the current user id.

Reached said:

I managed to fix it with your help!

So to any other people with similar issues, always double-check your models as well as your controllers.

My issue was that I didn't save the user_id to the database along with the other values when I submitted my form.

So now I am simply doing:

$listing->user_id = Auth::user()->id;

However as a side question, is there a better way to achieve this?

As i said you need to overwrite model functions.

For example:

// Default model function
/**
 * Save a new model and return the instance.
 *
 * @param  array  $attributes
 * @return \Illuminate\Database\Eloquent\Model|static
 */
public static function create(array $attributes)
{
	$model = new static($attributes);

	$model->save();

	return $model;
}

// How your function need to be:
/**
 * Save a new model and return the instance.
 *
 * @param  array  $attributes
 * @return \Illuminate\Database\Eloquent\Model|static
 */
public static function create(array $attributes)
{
	$attributes['user_id'] = Auth::user()->id;

	$model = new static($attributes);

	$model->save();

	return $model;
}

Just add this function in Listing model and it will overwrite the default function. Now when you will use create funtion ( Example: $user = User::create(array('name' => 'John')); ) it will add user_id by default.

I'm not at pc right now but i will look tonight and maybe we can add this in save function so it will works with every method you are using.

Reached said:

Can you be a bit more precise? :)