Merge pull request #559 from shanavas786/ar-sa-mobilephone

Saudi Arabia (ar-SA) Mobile Number Validation
validatorjs · Aug 2, 2016 · d0a02cb · d0a02cb
2 parents e310a87 + f8bf977
commit d0a02cb
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -86,7 +86,7 @@ Passing anything other than a string is an error.
 - **isLowercase(str)** - check if the string is lowercase.
 - **isMACAddress(str)** - check if the string is a MAC address.
 - **isMD5(str)** - check if the string is a MD5 hash.
-- **isMobilePhone(str, locale)** - check if the string is a mobile phone number, (locale is one of `['ar-DZ', 'ar-SY', 'cs-CZ', 'de-DE', 'da-DK', 'el-GR', 'en-AU', 'en-GB', 'en-HK', 'en-IN', 'en-NZ', 'en-US', 'en-CA', 'en-ZA', 'en-ZM', 'es-ES', 'fi-FI', 'fr-FR', 'hu-HU', 'ms-MY', 'nb-NO', 'nn-NO', 'pl-PL', 'pt-PT', 'ru-RU', 'tr-TR', 'vi-VN', 'zh-CN', 'zh-TW']`).
+- **isMobilePhone(str, locale)** - check if the string is a mobile phone number, (locale is one of `['ar-DZ', 'ar-SA', 'ar-SY', 'cs-CZ', 'de-DE', 'da-DK', 'el-GR', 'en-AU', 'en-GB', 'en-HK', 'en-IN', 'en-NZ', 'en-US', 'en-CA', 'en-ZA', 'en-ZM', 'es-ES', 'fi-FI', 'fr-FR', 'hu-HU', 'ms-MY', 'nb-NO', 'nn-NO', 'pl-PL', 'pt-PT', 'ru-RU', 'tr-TR', 'vi-VN', 'zh-CN', 'zh-TW']`).
 - **isMongoId(str)** - check if the string is a valid hex-encoded representation of a [MongoDB ObjectId][mongoid].
 - **isMultibyte(str)** - check if the string contains one or more multibyte chars.
 - **isNull(str)** - check if the string is null (has a length of zero).

diff --git a/lib/isMobilePhone.js b/lib/isMobilePhone.js
@@ -15,6 +15,7 @@ function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { de
 var phones = {
   'ar-DZ': /^(\+?213|0)(5|6|7)\d{8}$/,
   'ar-SY': /^(!?(\+?963)|0)?9\d{8}$/,
+  'ar-SA': /^(!?(\+?966)|0)?5\d{8}$/,
   'en-US': /^(\+?1)?[2-9]\d{2}[2-9](?!11)\d{6}$/,
   'cs-CZ': /^(\+?420)? ?[1-9][0-9]{2} ?[0-9]{3} ?[0-9]{3}$/,
   'de-DE': /^(\+?49[ \.\-])?([\(]{1}[0-9]{1,6}[\)])?([0-9 \.\-\/]{3,20})((x|ext|extension)[ ]?[0-9]{1,4})?$/,

diff --git a/src/lib/isMobilePhone.js b/src/lib/isMobilePhone.js
@@ -4,6 +4,7 @@ import assertString from './util/assertString';
 const phones = {
   'ar-DZ': /^(\+?213|0)(5|6|7)\d{8}$/,
   'ar-SY': /^(!?(\+?963)|0)?9\d{8}$/,
+  'ar-SA': /^(!?(\+?966)|0)?5\d{8}$/,
   'en-US': /^(\+?1)?[2-9]\d{2}[2-9](?!11)\d{6}$/,
   'cs-CZ': /^(\+?420)? ?[1-9][0-9]{2} ?[0-9]{3} ?[0-9]{3}$/,
   'de-DE': /^(\+?49[ \.\-])?([\(]{1}[0-9]{1,6}[\)])?([0-9 \.\-\/]{3,20})((x|ext|extension)[ ]?[0-9]{1,4})?$/,

diff --git a/test/validators.js b/test/validators.js
@@ -1838,6 +1838,25 @@ describe('Validators', function () {
       args: ['ar-SY'],
     });
 
+    test({
+      validator: 'isMobilePhone',
+      valid: [
+        '0556578654',
+        '+966556578654',
+        '966556578654',
+        '596578654',
+        '572655597',
+      ],
+      invalid: [
+        '12345',
+        '',
+        '+9665626626262',
+        '+96633221097',
+        '0114152198',
+      ],
+      args: ['ar-SA'],
+    });
+
     test({
       validator: 'isMobilePhone',
       valid: [

diff --git a/validator.js b/validator.js
@@ -869,6 +869,7 @@
       var phones = {
         'ar-DZ': /^(\+?213|0)(5|6|7)\d{8}$/,
         'ar-SY': /^(!?(\+?963)|0)?9\d{8}$/,
+        'ar-SA': /^(!?(\+?966)|0)?5\d{8}$/,
         'en-US': /^(\+?1)?[2-9]\d{2}[2-9](?!11)\d{6}$/,
         'cs-CZ': /^(\+?420)? ?[1-9][0-9]{2} ?[0-9]{3} ?[0-9]{3}$/,
         'de-DE': /^(\+?49[ \.\-])?([\(]{1}[0-9]{1,6}[\)])?([0-9 \.\-\/]{3,20})((x|ext|extension)[ ]?[0-9]{1,4})?$/,

diff --git a/validator.min.js b/validator.min.js