Laravel 5.3 Validate Multiple Request Show Errors

Question

Hi I have a controller that inserts multiple values into 2 tables from 1 form, I can validate them but only one at a time. Only If I click on the button to insert it shows the erros of the second validation.

public function store(CreateRequest $request, RegisterRequest $request2)
{
$input = Input::all();
$validation = Validator::make($input, $request->rules());
$validation2 = Validator::make($input, $request2->rules());
if ($validation->passes() and $validation2->passes())
{
  Users::create($input);
  Register::create($input2);
  return Redirect::route('users.main')
  ->withSuccess('Created');
 }

 return Redirect::route('users.create')
 ->withInput()
 ->withErrors();  
}

My question is how do I show the multiple erros, since I have 2 different request and rules. — kaaps
– kaaps, Commented Dec 19, 2016 at 12:57
If I hit the button create it show the first validation erros if I pass the first validation the second error message show's and I want it to show at the same time — kaaps
– kaaps, Commented Dec 19, 2016 at 13:03

aimme · Accepted Answer · 2016-12-19 13:26:14Z

1

you are doing it totally wrong. a request is a single request. though laravel supports request inside request in this scenario what i see is that you are trying to register a user into the system. do it like this.

create one request. say just RegistrationRequest and put all the form fields and the rules in it. use that request only to store the user like this.

public function store(RegisterRequest $request)
{
    $data = $request->all();
    $validation = Validator::make($data, $request->rules());

    if ($validation->passes()) {
       $user = User::create([
          'username' => $request->username,
          'name' => $request->name
       ]);
       $register = Register::create([
         'blabla' => $request->name,
         'username' => $user->username
       ]);
       return Redirect::route('users.main')->withSuccess('Created');
    }
    return Redirect::route('users.create')->withInput()->withErrors();  
}

this is just a demonstration as per the actual problem you are facing is that you are trying a wrong approach, there are another ways to do this.and also validation errors are thrown and redirected to the create view like automatically. so you don't have to do it on your own

edited Dec 19, 2016 at 13:26

answered Dec 19, 2016 at 13:19

aimme

6,9087 gold badges54 silver badges71 bronze badges

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

2 Comments

kaaps Over a year ago

Yep I thought of this aproach and maybe it's the best one. I was going with 2 Request because I thought I could reuse when I want to insert at one table, so I would have the validation of table 1 on the request 1 and the validation of the table 2 at the 2 request.

everytimeicob Over a year ago

I agree, with the Laravel Validator facade, you have some greatful validation like required_if to make your validations depending on more that one table :)

Amit Gupta · Accepted Answer · 2016-12-19 14:35:14Z

0

You can merge the messages of validators and send them back as:

$errors = $validation->errors()->merge($validation2->errors())

return Redirect::route('users.create')->withInput()->withErrors($errors);

answered Dec 19, 2016 at 14:35

Amit Gupta

17.7k5 gold badges46 silver badges54 bronze badges

Collectives™ on Stack Overflow

Laravel 5.3 Validate Multiple Request Show Errors

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related