How to include user id in $validated create?

Can you show the code for validateThread()?

@sinnbeck sure, here it is

public function validateThread()
{
    $validated = $this->validate(request(), [
        'title' => 'required',
        'body' => 'required',
        'channel_id' => 'required|exists:channels,id',
    ]);

    resolve(Spam::class)->detect(request('body'));

    return $validated;
}

and now $validated contains this

array:3 [
  "title" => "Autem quis quia nisi alias."
  "body" => "Eligendi nesciunt error cupiditate esse labore soluta. Nam consequatur aut assumenda ullam eligendi aliquam. Tenetur aliquam occaecati beatae eum voluptas."
  "channel_id" => 1
]

public function validateThread()
{
    $validated = $this->validate(request(), [
        'title' => 'required',
        'body' => 'required',
        'channel_id' => 'required|exists:channels,id',
    ]);

    resolve(Spam::class)->detect(request('body'));

    return $validated + ['user_id' => auth()->id()];
}

If there is relation in eloquent between User and Thread supposely named threads() you could do:

public function store(Request $request)
{
    $thread = $request->user()->threads()->create($this->validateThread());
}

public function validateThread()
{
    $validated = $this->validate(request(), [
        'title' => 'required',
        'body' => 'required',
        'channel_id' => 'required|exists:channels,id',
    ]);

    resolve(Spam::class)->detect(request('body'));

    return $validated;
}

Hi @martinzeltin

Well, you could do an array_merge and pass that to your function instead of using the request() internally.

So you'll end up with something like this

public function store(Request $request)
{
    $validated = $this->validateThread(array_merge(
        request(), 
        ['user_id' => auth()->id()]
    ));

    $thread = Thread::create($validated);
}

and then in your function:

public function validateThread($fields)
{
    $validated = $this->validate($fields, [
	'user_id' => 'required',
        'title' => 'required',
        'body' => 'required',
        'channel_id' => 'required|exists:channels,id',
    ]);

    resolve(Spam::class)->detect(request('body'));

    return $validated;
}

Does that make sense?

Although I think I like @sinnbeck solution better, as you don't actually need to include the user_id in the validation, just returning it at the end would be enough.

@marianomoreyra The best solution is by @thewebartisan7

You are right @michaloravec !!

Don’t know why I skipped @thewebartisan7 reply!!

My bad!