Skip to content

Tiny inconsistency between 'between' and 'digits_between' validation rules messages #2377

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 29, 2013

Conversation

rmobis
Copy link
Contributor

@rmobis rmobis commented Oct 26, 2013

Even though between and digits_between validation rules are pretty much the same, they do have a small difference: between uses - for the range, while digits_between uses and.

between: The :attribute must be between :min - :max.
digits_between: The :attribute must be between :min and :max digits.

I know this is really no big deal, but it does look nonconforming when you look at both errors side by side, on your website. I was thinking maybe we could decide which is better and just change the other.

@franzliedke
Copy link
Contributor

The dash doesn't feel like correct English to me. If there's a "between", there should be an "and".

You can send a pull request for this. =)

taylorotwell added a commit that referenced this pull request Oct 29, 2013
Tiny inconsistency between 'between' and 'digits_between' validation rules messages
@taylorotwell taylorotwell merged commit 65842ae into laravel:master Oct 29, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants