Merge pull request #627 from arpadszasz/master

Add support for Romanian mobile phone numbers
validatorjs · Feb 24, 2017 · 5b62a91 · 5b62a91
2 parents 0024404 + 8afd190
commit 5b62a91
Show file tree

Hide file tree

Showing 6 changed files with 35 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -88,7 +88,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-SA', 'ar-SY', 'cs-CZ', 'de-DE', 'da-DK', 'el-GR', 'en-AU', 'en-GB', 'en-HK', 'en-IN',  'en-NG', 'en-NZ', 'en-US', 'en-CA', 'en-ZA', 'en-ZM', 'es-ES', 'fi-FI', 'fr-FR', 'he-IL', 'hu-HU', 'it-IT', 'ja-JP', 'ms-MY', 'nb-NO', 'nn-NO', 'pl-PL', 'pt-PT', 'ru-RU', 'sr-RS', '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-NG', 'en-NZ', 'en-US', 'en-CA', 'en-ZA', 'en-ZM', 'es-ES', 'fi-FI', 'fr-FR', 'he-IL', 'hu-HU', 'it-IT', 'ja-JP', 'ms-MY', 'nb-NO', 'nn-NO', 'pl-PL', 'pt-PT', 'ro-RO', 'ru-RU', 'sr-RS', '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.
 - **isNumeric(str)** - check if the string contains only numbers.

diff --git a/lib/isMobilePhone.js b/lib/isMobilePhone.js
@@ -43,6 +43,7 @@ var phones = {
   'pl-PL': /^(\+?48)? ?[5-8]\d ?\d{3} ?\d{2} ?\d{2}$/,
   'pt-BR': /^(\+?55|0)\-?[1-9]{2}\-?[2-9]{1}\d{3,4}\-?\d{4}$/,
   'pt-PT': /^(\+?351)?9[1236]\d{7}$/,
+  'ro-RO': /^(\+?4?0)\s?7\d{2}(\/|\s|\.|\-)?\d{3}(\s|\.|\-)?\d{3}$/,
   'ru-RU': /^(\+?7|8)?9\d{9}$/,
   'sr-RS': /^(\+3816|06)[- \d]{5,9}$/,
   'tr-TR': /^(\+?90|0)?5\d{9}$/,

diff --git a/src/lib/isMobilePhone.js b/src/lib/isMobilePhone.js
@@ -32,6 +32,7 @@ const phones = {
   'pl-PL': /^(\+?48)? ?[5-8]\d ?\d{3} ?\d{2} ?\d{2}$/,
   'pt-BR': /^(\+?55|0)\-?[1-9]{2}\-?[2-9]{1}\d{3,4}\-?\d{4}$/,
   'pt-PT': /^(\+?351)?9[1236]\d{7}$/,
+  'ro-RO': /^(\+?4?0)\s?7\d{2}(\/|\s|\.|\-)?\d{3}(\s|\.|\-)?\d{3}$/,
   'en-PK': /^((\+92)|(0092))-{0,1}\d{3}-{0,1}\d{7}$|^\d{11}$|^\d{4}-\d{7}$/,
   'ru-RU': /^(\+?7|8)?9\d{9}$/,
   'sr-RS': /^(\+3816|06)[- \d]{5,9}$/,

diff --git a/test/validators.js b/test/validators.js
@@ -2887,6 +2887,36 @@ describe('Validators', function () {
       ],
       args: ['nl-BE'],
     });
+
+    test({
+      validator: 'isMobilePhone',
+      valid: [
+        '+40740123456',
+        '+40 740123456',
+        '+40740 123 456',
+        '+40740.123.456',
+        '+40740-123-456',
+        '40740123456',
+        '40 740123456',
+        '40740 123 456',
+        '40740.123.456',
+        '40740-123-456',
+        '0740123456',
+        '0740/123456',
+        '0740 123 456',
+        '0740.123.456',
+        '0740-123-456',
+      ],
+      invalid: [
+        '',
+        'Vml2YW11cyBmZXJtZtesting123',
+        '123456',
+        '740123456',
+        '+40640123456',
+        '+40210123456',
+      ],
+      args: ['ro-RO'],
+    });
   });
 
   it('should validate currency', function () {

diff --git a/validator.js b/validator.js
@@ -1111,6 +1111,7 @@
         'pl-PL': /^(\+?48)? ?[5-8]\d ?\d{3} ?\d{2} ?\d{2}$/,
         'pt-BR': /^(\+?55|0)\-?[1-9]{2}\-?[2-9]{1}\d{3,4}\-?\d{4}$/,
         'pt-PT': /^(\+?351)?9[1236]\d{7}$/,
+        'ro-RO': /^(\+?4?0)\s?7\d{2}(\/|\s|\.|\-)?\d{3}(\s|\.|\-)?\d{3}$/,
         'ru-RU': /^(\+?7|8)?9\d{9}$/,
         'sr-RS': /^(\+3816|06)[- \d]{5,9}$/,
         'tr-TR': /^(\+?90|0)?5\d{9}$/,

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