How to add a custom validation rule in Laravel?

Question

I would like to add custom validations rules on my Laravel Controller:

The container exists in the database
The logged user is owner of the resource

So I wrote this:

public function update($id, Request $request) {
    $validator = Validator::make($request->all(), [
        'name' => 'required|unique:stock.containers|max:255'
    ]);

    $container = Container::find($id);

    if(!$container)
    {
        $validator->errors()->add('id', 'Not a valid resource');
    }

    if($container->owner_id != user_id())
    {
        $validator->errors()->add('owner_id', 'Not owner of this resource');
    }

    if ($validator->fails()) {
        return response()->json($validator->errors(), 422);  //i'm not getting any
    }
}

Unfortunately the $validator->errors() or even $validator->addMessageBag() does not work. I noticed $validator->fails() clears the error messages and adding an error will not make the validation to fail.

What is the proper way to achieve this?

Well, then how do you apply this rule. The rule does not know the id of the resource and the rule does not apply to any value in the request... — nowox
– nowox, Commented Jun 6, 2019 at 12:35
You are using a wrong approach I believe. Because you can use the exists on the validator to make sure that the resource exists, but then you need to use a policy which will authorize the user for using the resource. — nakov
– nakov, Commented Jun 6, 2019 at 12:40
@nakov, Yes I can use the exists, but I don't have the $id of my resource, so I cannot check if it exists :( — nowox
– nowox, Commented Jun 6, 2019 at 12:47

nowox · Accepted Answer · 2019-06-06 12:40:02Z

1

Then you can use after:

public function update($id, Request $request) {
    $validator = Validator::make($request->all(), [
        'name' => 'required|unique:stock.containers|max:255'
    ]);

    $validator->after(function($validator) use($id) {
        $container = Container::find($id);

        if(!$container)
        {
            $validator->errors()->add('id', 'Not a valid resource');
        }

        if($container->owner_id != user_id())
        {
            $validator->errors()->add('owner_id', 'Not owner of this resource');
        }
    });

    if ($validator->fails()) {
        return response()->json($validator->errors(), 422);  //i'm not getting any
    }
}

answered Jun 6, 2019 at 12:40

nowox

29.7k49 gold badges179 silver badges361 bronze badges

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

1 Comment

Kamlesh Over a year ago

Your suggested solution is not working. laravel does not come in $validator->after(function($validator) use($id) { function. I have added a echo 'test';exit; before $container = Container::find($id); line but nothing is working. Any suggestion will be welcomed.

Collectives™ on Stack Overflow

How to add a custom validation rule in Laravel?

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related