fix(isMobilePhone): fix 'ro-RO' matching invalid numbers

* fixed the bug that would cause it to match invalid numbers * added tests for invalid numbers fixes validatorjs#2055
pano9000 · Jan 28, 2023 · cd237dc · cd237dc
1 parent f97e8d4
commit cd237dc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/lib/isMobilePhone.js b/src/lib/isMobilePhone.js
@@ -131,7 +131,7 @@ const phones = {
   'pt-BR': /^((\+?55\ ?[1-9]{2}\ ?)|(\+?55\ ?\([1-9]{2}\)\ ?)|(0[1-9]{2}\ ?)|(\([1-9]{2}\)\ ?)|([1-9]{2}\ ?))((\d{4}\-?\d{4})|(9[1-9]{1}\d{3}\-?\d{4}))$/,
   'pt-PT': /^(\+?351)?9[1236]\d{7}$/,
   'pt-AO': /^(\+244)\d{9}$/,
-  'ro-RO': /^(\+?4?0)\s?7\d{2}(\/|\s|\.|\-)?\d{3}(\s|\.|\-)?\d{3}$/,
+  'ro-RO': /^(\+?40|0)\s?7\d{2}(\/|\s|\.|-)?\d{3}(\s|\.|-)?\d{3}$/,
   'ru-RU': /^(\+?7|8)?9\d{9}$/,
   'si-LK': /^(?:0|94|\+94)?(7(0|1|2|4|5|6|7|8)( |-)?)\d{7}$/,
   'sl-SI': /^(\+386\s?|0)(\d{1}\s?\d{3}\s?\d{2}\s?\d{2}|\d{2}\s?\d{3}\s?\d{3})$/,

diff --git a/test/validators.test.js b/test/validators.test.js
@@ -8856,6 +8856,8 @@ describe('Validators', () => {
           '740123456',
           '+40640123456',
           '+40210123456',
+          '+0765351689',
+          '+0711419752',
         ],
       },
       {