Skip to content
This repository was archived by the owner on Apr 19, 2022. It is now read-only.

Support for stopping validation on first error per field #250

Open
olaoluwa-98 opened this issue Aug 19, 2019 · 2 comments
Open

Support for stopping validation on first error per field #250

olaoluwa-98 opened this issue Aug 19, 2019 · 2 comments

Comments

@olaoluwa-98
Copy link

For instance, if you have the following rules:

    email: 'required|email|unique:users,email',
    phone_no: 'required|unique:users,phone_no',
    password: 'required|min:8'

You may want the validation for email to stop on first error but continue to validate other fields such as phone_no and password which also will stop on first error of any rule item. I believe this feature will allow you to save time spent validating each item of a single rule.

validateAll continues to validate all fields including all the rules under each field. What I am proposing is a validate method that stops validation for each field on first error but continues to validate the other fields.

I am willing to work on it but will need assistance.

@olaoluwa-98 olaoluwa-98 changed the title Support for stopping validation on first error for per field Support for stopping validation on first error per field Aug 19, 2019
@thetutlage
Copy link
Member

Hello @olaoluwa-98

Yeah, will be nice to have such a feature. Couple of questions?

  1. What should be the name of property? Since validateAll is reserved, what should we call it?
  2. Can you start working on the PR and just ping me, if you get stuck somewhere?

@olaoluwa-98
Copy link
Author

olaoluwa-98 commented Aug 23, 2019

Hello @thetutlage, sorry for the late response.

I'm not sure what name can go with it. But I suggest that we accept an option on the validateAll or validate that will allow this feature. e.g In validate(data, rules, messages, options), the options object will accept the option for.
I'll start working on the PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants