Laravel: Why are validation messages not show?

Question

I use Laravel 5.4 on hosting. My validation method is working, but it doesn't show messages. I do all the needed operations for displaying errors but it doesn't work.

@if($errors->has('recipient'))
<div class="form-group">
<label class="col-lg-2 control-label"></label>  
<div class="col-lg-10">                     
    <div class="alert alert-danger">
        <ul>
            @foreach ($errors->get('recipient') as $error)
                <li>{{ $error }}</li>
            @endforeach
        </ul>
    </div>
</div>
</div>
@endif

I use {{ csrf_field() }} tokens. Validation rules work if wrong user types are used, but messages are not displayed.

My send method:

public function send(Request $request)
{
   $this->validator($request->all())->validate();
   $this->create($request->all());
   return redirect($this->redirectToAfterSendMessage);
}

My validator code:

protected function validator(array $data)
{        
    return Validator::make($data, [

        "$this->recipient" => [
            'sometimes',
            'required',
            'email',
            'exists:users,email',
            Rule::notIn([auth()->user()->email])
        ],

        "$this->subject" => 'sometimes|required|min:10',

        "$this->message" => 'sometimes|required|min:50',

    ], $this->validator_messages());
}

My routes:

Route::get('/compose', 'InboxController@compose')     ->name('compose');
Route::post('/compose', 'InboxController@send');

Result of dump($errors):

ViewErrorBag {#244 ▼
  #bags: []
}

Answer 1

EDITED

To fix your current problem, change $errors->get('recipient') to $errors->all().

It should look like this:

@foreach ($errors->all() as $error)
   <li>{{ $error }}</li>
@endforeach

--

This isn't a direct solution for your question. However an even better approach (imo) than yours are custom requests. You can quickly generate custom requests by using the following command:

php artisan make:request CustomRequest

After that you can change type from Request to CustomRequest

public function send(CustomRequest $request)
{
   $this->create($request->all());
   return redirect($this->redirectToAfterSendMessage);
}

And in CustomRequest.php your extract your validation logic like so:

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;

class CustomRequest extends FormRequest
{
    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize()
    {
        return true;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {
        return [
            'name' => 'required',
            'email' => 'required',
            'message' => 'required',

        ];
    }

    public function messages()
    {
        return [
            'name.required' => 'Name is required.',
            'email.required' => 'Email is required.',
            'message.required' => 'Message is required.',
        ];
    }
}

Hope this helps.

Answer 2

You need to change:

"$this->recipient"
"$this->subject" 
"$this->message"

To:

'recipient'
'subject'
'message'