Issue in validating isMobilePhone() with 'en-US' options.

[Madhavi96]

The regular expression used for this validation is /^((\+1|1)?( |-)?)?(\([2-9][0-9]{2}\)|[2-9][0-9]{2})( |-)?([2-9][0-9]{2}( |-)?[0-9]{4})$/.

In common US phone validation web sites, the following condition is checked for US phone number validations.

1. The area code (first 3 digits) cannot be the same digit

But the above mentioned regular expression does not satisfy the first condition.
For example, if we validate the number 2225550135 using the isMobilePhone() method, it results in a true. But the first 3 digits of the phone number(area code) are similar, thus the web site https://www.textmagic.com/free-tools/phone-validator gives a false.

Due to this reason, this validation could not be used for the project I was working on recently.
Is this an issue in the library? If so, I'm willing to raise a PR.

[rubiin]

@Madhavi96 can you list some websites where i might be able to find the validations? I may then be able to reproduce the issue and thus may work on a fix for this

[Madhavi96]

I think the issue is In US, the area code (first 3 digits) cannot be the same digit and the existing regular expression for US mobile number validation does not validate that condition. (Even when the three digits are the same, currently, the number is validated as a valid US mobile number)

To fix this, I suggest the following regular expression for 'en-US' locale.

^((\+1|1)?( |-)?)?(\(([2-9])(?:\d(?!\5)\d|(?!\5)\d\d)\)|([2-9])(?:\d(?!\6)\d|(?!\6)\d\d))( |-)?([2-9][0-9]{2}( |-)?[0-9]{4})$

If this is okay, I can fix it through a PR.

For reference, I will list some websites that check this condition.

https://www.phonevalidator.com/results.aspx
https://www.textmagic.com/free-tools/phone-validator

[rubiin]

@profnandaa @ezkemboi what do you guys think

[profnandaa]

I'll recheck this; but please do raise the PR so that I'm not blocking...