Skip to content

isMobilePhone - fixed validation for am-AM #2140

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.

Sign up for GitHub

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
profnandaa merged 9 commits into from
Apr 25, 2024
Merged

isMobilePhone - fixed validation for am-AM #2140

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
78c73cb
isMobilePhone - fixed validation for am-AM
AlexKrupko Jan 3, 2023
98b9fdf
Update tests for isMobilePhone
AlexKrupko Jan 3, 2023
9b2aa48
Updated phone validation pattern for am-AM locale
AlexKrupko Jan 3, 2023
043eba1
Updated tests for am-AM phone validation pattern
AlexKrupko Jan 3, 2023
680b016
removed validation for fixed telephony services
AlexKrupko Jan 3, 2023
374dbb0
Fixed test for phone validation
AlexKrupko Jan 3, 2023
4bf31c5
Moved removed phone numbers to valid
AlexKrupko Jan 20, 2023
ce55c93
Merge branch 'master' into patch-1
AlexKrupko Aug 17, 2023
158489b
fixed mobile phone pattern for AM
AlexKrupko Aug 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/lib/isMobilePhone.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import assertString from './util/assertString';

/* eslint-disable max-len */
const phones = {
'am-AM': /^(\+?374|0)((10|[9|7][0-9])\d{6}$|[2-4]\d{7}$)/,
'am-AM': /^(\+?374|0)(1[0125][2-9]|22[2346]|23[1-7]|24[2-69]|25[2-7]|26[1-9]|28[13-7]|3[12]2|(33|4[134]|55|77|88|9[13-689])\d)\d{5}$/,
Copy link
Member

@profnandaa profnandaa Jan 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: wondering if this regex could be any simpler?

Copy link
Contributor Author

@AlexKrupko AlexKrupko Jan 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think, I did all my best to combine and simplify similar parts.

Copy link
Member

@WikiRik WikiRik Jan 30, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're still validating fixed phone numbers in this RegExp. See the section for 'Non-geographic number for mobile services' on page 12 for the only mobile phone numbers; https://www.itu.int/dms_pub/itu-t/oth/02/02/T020200000A0007PDFE.pdf

Suggested change
'am-AM': /^(\+?374|0)(1[0125][2-9]|22[2346]|23[1-7]|24[2-69]|25[2-7]|26[1-9]|28[13-7]|3[12]2|(33|4[134]|55|77|88|9[13-689])\d)\d{5}$/,
'am-AM': /^(\+?374|0)(33|4[134]|55|77|88|9[13-689])\d{6}$/,

Copy link
Contributor Author

@AlexKrupko AlexKrupko Aug 17, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@WikiRik,
pattern for fixed phone numbers has been removed, only mobile numbers are covered now there.
Thank you for suggestion

'ar-AE': /^((\+?971)|0)?5[024568]\d{7}$/,
'ar-BH': /^(\+?973)?(3|6)\d{7}$/,
'ar-DZ': /^(\+?213|0)(5|6|7)\d{8}$/,
Expand Down
12 changes: 9 additions & 3 deletions test/validators.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6458,14 +6458,20 @@ describe('Validators', () => {
'022698763',
'37491987654',
'+37494567890',
'+37455123456',
'+37477123456',
'+37488123456',
],
invalid: [
'12345',
'+37411498855',
Copy link
Member

@profnandaa profnandaa Jan 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is sort of minor "breaking change", can you move the removed test-cases to the valid side then?

Copy link
Contributor Author

@AlexKrupko AlexKrupko Jan 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

'+37411498123',
'+37403498855',
'+37416498123',
'05614988556',
'',
'37456789000',
'37486789000',
'+37431312345',
'+37430312345',
'+37460123456',
],
},
{
Expand Down