New option in PhoneNumberValidator to enforce exact number of digits

core/src/main/resources/i18n/displayStrings.properties

@@ -4610,6 +4610,7 @@ validation.phone.missingCountryCode=Need two letter country code to validate pho
 validation.phone.invalidCharacters=Phone number {0} contains invalid characters
 validation.phone.insufficientDigits=There are not enough digits in {0} to be a valid phone number
 validation.phone.tooManyDigits=There are too many digits in {0} to be a valid phone number
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=Country dialing code for number {0} is invalid for country {1}.  \
   The correct dialing code is {2}.
 validation.invalidAddressList=Must be comma separated list of valid addresses

core/src/main/resources/i18n/displayStrings_cs.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=K ověření telefonního čísla je potřeb
 validation.phone.invalidCharacters=Telefonní číslo {0} obsahuje neplatné znaky
 validation.phone.insufficientDigits=V čísle {0} není dostatek číslic, aby mohlo být platné telefonní číslo
 validation.phone.tooManyDigits=V čísle {0} je příliš mnoho číslic, než aby mohlo být platné telefonní číslo
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=Telefonní předvolba země pro číslo {0} je pro zemi {1} neplatná. Správné předčíslí je {2}.
 validation.invalidAddressList=Seznam platných adres musí být oddělený čárkami
 validation.capitual.invalidFormat=Musíte zadat CAP kód ve formátu: CAP-XXXXXX (6 alfanumerických znaků)

core/src/main/resources/i18n/displayStrings_de.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=Es wird eine zweistellige Ländervorwahl ben
 validation.phone.invalidCharacters=Telefonnummer {0} enthält ungültige Zeichen
 validation.phone.insufficientDigits=Das ist keine gültige Telefonnummer. Sie habe nicht genügend Stellen angegeben. 
 validation.phone.tooManyDigits=Es sind zu viele Ziffern in {0} um eine gültige Telefonnummer zu sein.
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=Die Ländervorwahl in der Nummer {0} ist für das Land {1} ungültig. Die richtige Vorwahl ist {2}.
 validation.invalidAddressList=Muss eine kommagetrennte Liste der gültigen Adressen sein
 validation.capitual.invalidFormat=Sollte ein gültiger CAP Code im Format: CAP-XXXXXX (6 alphanumerische Charakter sein)

core/src/main/resources/i18n/displayStrings_es.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=Se necesitan dos letras de código de país
 validation.phone.invalidCharacters=Número de teléfono {0} tiene caracteres inválidos
 validation.phone.insufficientDigits=No hay suficientes dígitos en {0} para ser un número válido de teléfono
 validation.phone.tooManyDigits=Hay demasiados dígitos en {0} para ser un número de teléfono válido.
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=El código de país para el número {0} es inválido para el país {1}. El código de país correcto es {2}.
 validation.invalidAddressList=La lista de direcciones válidas debe ser separada por coma
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_fa.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=Need two letter country code to validate pho
 validation.phone.invalidCharacters=Phone number {0} contains invalid characters
 validation.phone.insufficientDigits=There are not enough digits in {0} to be a valid phone number
 validation.phone.tooManyDigits=There are too many digits in {0} to be a valid phone number
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=Country dialing code for number {0} is invalid for country {1}.  The correct dialing code is {2}.
 validation.invalidAddressList=Must be comma separated list of valid addresses
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_fr.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=Un code pays à deux lettres est nécessaire
 validation.phone.invalidCharacters=Le numéro de téléphone {0} contient des caractères invalides.
 validation.phone.insufficientDigits=Il n'y a pas assez de chiffres dans {0} pour être un numéro de téléphone valide
 validation.phone.tooManyDigits=Il y a trop de chiffres dans {0} pour être un numéro de téléphone valide
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=L'indicatif de pays du numéro {0} est invalide pour le pays {1}. Le bon indicatif est {2}.
 validation.invalidAddressList=Doit être une liste d’adresses valide séparée par des virgules
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_it.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=È necessario un codice paese di due lettere
 validation.phone.invalidCharacters=Il numero di telefono {0} contiene caratteri non validi
 validation.phone.insufficientDigits=There are not enough digits in {0} to be a valid phone number
 validation.phone.tooManyDigits=There are too many digits in {0} to be a valid phone number
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=Country dialing code for number {0} is invalid for country {1}.  The correct dialing code is {2}.
 validation.invalidAddressList=Deve essere un elenco separato da virgole di indirizzi validi
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_ja.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=電話番号を検証するのに2文字国
 validation.phone.invalidCharacters=電話番号 {0} には無効な文字が含まれている
 validation.phone.insufficientDigits={0} には桁数が不十分で有効電話番号になりません
 validation.phone.tooManyDigits={0} には桁数が多過ぎて有効電話番号になりません
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=電話番号 {0} の国番号は国の {1} にとって間違っています。正しい国番号は {2} です。
 validation.invalidAddressList=有効アドレスのコンマ区切りリストでなければなりません
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_pl.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=Potrzeba dwuliterowego kodu krajowego aby zw
 validation.phone.invalidCharacters=Numer telefonu {0} zawiera nieprawidłowe znaki
 validation.phone.insufficientDigits=Nie ma wystarczająco dużo cyfr w {0} aby to był właściwy numer telefonu
 validation.phone.tooManyDigits=Jest zbyt dużo cyfr w {0} aby to był właściwy numer telefonu
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=Numer kierunkowy dla {0} jest nieprawidłowy dla kraju {1}. Prawidłowy numer kierunkowy to {2}.
 validation.invalidAddressList=Musi być przecinek oddzielony listą poprawnych adresów
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_pt-br.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=Precisa do código do país com duas letras
 validation.phone.invalidCharacters=O número de telefone {0} contém caracteres inválidos.
 validation.phone.insufficientDigits=Não há dígitos suficientes em {0} para ser um número de telefone válido.
 validation.phone.tooManyDigits=Há muitos dígitos em {0} para ser um número de telefone válido.
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=O código de discagem do país no número {0} é inválido para o país {1}. O código de discagem correto é {2}.
 validation.invalidAddressList=Precisa ser uma lista delimitada por vírgulas de endereços válidos
 validation.capitual.invalidFormat=Deve ser um código CAP válido no formato: CAP-XXXXXX (6 caracteres alfanuméricos)

core/src/main/resources/i18n/displayStrings_pt.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=É preciso o código do país de duas letras
 validation.phone.invalidCharacters=O número de telfone {0} contém carácteres inválidos
 validation.phone.insufficientDigits=There are not enough digits in {0} to be a valid phone number
 validation.phone.tooManyDigits=There are too many digits in {0} to be a valid phone number
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=Country dialing code for number {0} is invalid for country {1}.  The correct dialing code is {2}.
 validation.invalidAddressList=Deve ser um lista de endereços válidos separados por vírgulas 
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_ru.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=Need two letter country code to validate pho
 validation.phone.invalidCharacters=Phone number {0} contains invalid characters
 validation.phone.insufficientDigits=There are not enough digits in {0} to be a valid phone number
 validation.phone.tooManyDigits=There are too many digits in {0} to be a valid phone number
+validation.phone.incorrectLength=Поле должно содержать {0} символов
 validation.phone.invalidDialingCode=Country dialing code for number {0} is invalid for country {1}.  The correct dialing code is {2}.
 validation.invalidAddressList=Must be comma separated list of valid addresses
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_th.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=Need two letter country code to validate pho
 validation.phone.invalidCharacters=Phone number {0} contains invalid characters
 validation.phone.insufficientDigits=There are not enough digits in {0} to be a valid phone number
 validation.phone.tooManyDigits=There are too many digits in {0} to be a valid phone number
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=Country dialing code for number {0} is invalid for country {1}.  The correct dialing code is {2}.
 validation.invalidAddressList=Must be comma separated list of valid addresses
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_vi.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=Need two letter country code to validate pho
 validation.phone.invalidCharacters=Phone number {0} contains invalid characters
 validation.phone.insufficientDigits=There are not enough digits in {0} to be a valid phone number
 validation.phone.tooManyDigits=There are too many digits in {0} to be a valid phone number
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=Country dialing code for number {0} is invalid for country {1}.  The correct dialing code is {2}.
 validation.invalidAddressList=Must be comma separated list of valid addresses
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_zh-hans.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=需要两个字母的国家或地区代码
 validation.phone.invalidCharacters=电话号码 {0} 包含无效字符
 validation.phone.insufficientDigits={0} 中没有足够的数字作为有效的电话号码
 validation.phone.tooManyDigits={0} 中的数字太多，不是有效的电话号码
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=数字 {0} 中的国际拨号代码对于 {1} 无效。正确的拨号号码是 {2} 。
 validation.invalidAddressList=使用逗号分隔有效地址列表
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

core/src/main/resources/i18n/displayStrings_zh-hant.properties

@@ -3519,6 +3519,7 @@ validation.phone.missingCountryCode=需要兩個字母的國家或地區代碼
 validation.phone.invalidCharacters=電話號碼 {0} 包含無效字符
 validation.phone.insufficientDigits={0} 中沒有足夠的數字作為有效的電話號碼
 validation.phone.tooManyDigits={0} 中的數字太多，不是有效的電話號碼
+validation.phone.incorrectLength=The field must contain {0} numbers
 validation.phone.invalidDialingCode=數字 {0} 中的國際撥號代碼對於 {1} 無效。正確的撥號號碼是 {2} 。
 validation.invalidAddressList=使用逗號分隔有效地址列表
 validation.capitual.invalidFormat=Must be a valid CAP code of format: CAP-XXXXXX (6 alphanumeric characters)

desktop/src/main/java/bisq/desktop/util/validation/PhoneNumberValidator.java

@@ -106,7 +106,11 @@ public ValidationResult validate(String input) {
                 normalizedPhoneNumber = "+" + getCallingCode() + pureNumber;
             }
 
+            //  If the 'requiredLength' was set, there's still one more check to apply on the normalizedNumber itself
             result = validateRequiredLength(normalizedPhoneNumber, normalizedCallingCode);
+            if (!result.isValid) {
+                normalizedPhoneNumber = null;
+            }
         }
         return result;
     }

desktop/src/test/java/bisq/desktop/util/validation/PhoneNumberValidatorTest.java

@@ -9,6 +9,7 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 public class PhoneNumberValidatorTest {
@@ -296,4 +297,78 @@ public void testUSAreaCodeMatchesCallingCode() {
         // TODO validator incorrectly treats input as if it were +1 (202) 53-0000
         /// assertEquals("+1202530000", validator.getNormalizedPhoneNumber());
     }
+
+    @Test
+    public void testPhoneNumberLength() {
+        // Construct the validator to be used in these test cases
+        validator = new PhoneNumberValidator("US");  // No requiredLength passed in these tests
+
+        // Most important, does the validator work for correct phone numbers?
+        validationResult = validator.validate("+1 512 888 0150");
+        assertTrue(validationResult.isValid, "+1 512 888 0150 should be a valid number in US");
+        assertNotNull(validator.getNormalizedPhoneNumber());
+        assertEquals(validator.getNormalizedPhoneNumber(), "+15128880150");
+
+        // If no country code provided by user, normalized number should have it added
+        validationResult = validator.validate("5128880150");
+        assertTrue(validationResult.isValid, "5128880150 should be a valid number in US");
+        assertNotNull(validator.getNormalizedPhoneNumber());
+        assertEquals(validator.getNormalizedPhoneNumber(), "+15128880150");
+
+        // If no phone number too short, there's a message for that
+        validationResult = validator.validate("+15121");
+        assertFalse(validationResult.isValid, "+15121 should be too short");
+        assertNull(validator.getNormalizedPhoneNumber());
+        assertEquals(Res.get("validation.phone.insufficientDigits", "+15121"), validationResult.errorMessage);
+
+        // If no phone number too long, there's a message for that
+        validationResult = validator.validate("51288801505128880150");
+        assertFalse(validationResult.isValid, "51288801505128880150 should be too long");
+        assertNull(validator.getNormalizedPhoneNumber());
+        assertEquals(Res.get("validation.phone.tooManyDigits", "51288801505128880150"), validationResult.errorMessage);
+
+        // If phone number is not exactly the requiredLength, it still may be okay if length in the range 4-12
+        validationResult = validator.validate("123456");
+        assertTrue(validationResult.isValid, "123456 should be a considered a valid number");
+        assertNotNull(validator.getNormalizedPhoneNumber());
+        assertEquals(validator.getNormalizedPhoneNumber(), "+123456");
+    }
+
+    @Test
+    public void testPhoneNumberRequiredLength() {
+        // Construct the validator to be used in these test cases
+        final int REQUIRED_LENGTH = 10;
+        validator = new PhoneNumberValidator("US", REQUIRED_LENGTH);  // requiredLength passed for these tests
+
+        // Most important, does the validator work for correct phone numbers?
+        validationResult = validator.validate("+1 512 888 0150");
+        assertTrue(validationResult.isValid, "+1 512 888 0150 should be a valid number in US");
+        assertNotNull(validator.getNormalizedPhoneNumber());
+        assertEquals(validator.getNormalizedPhoneNumber(), "+15128880150");
+
+        // If no country code provided by user, normalized number should have it added
+        validationResult = validator.validate("5128880150");
+        assertTrue(validationResult.isValid, "5128880150 should be a valid number in US");
+        assertNotNull(validator.getNormalizedPhoneNumber());
+        assertEquals(validator.getNormalizedPhoneNumber(), "+15128880150");
+
+        // If no phone number too short, there's a message for that
+        validationResult = validator.validate("+15121");
+        assertFalse(validationResult.isValid, "+15121 should be too short");
+        assertNull(validator.getNormalizedPhoneNumber());
+        assertEquals(Res.get("validation.phone.insufficientDigits", "+15121"), validationResult.errorMessage);
+
+        // If no phone number too long, there's a message for that
+        validationResult = validator.validate("51288801505128880150");
+        assertFalse(validationResult.isValid, "51288801505128880150 should be too long");
+        assertNull(validator.getNormalizedPhoneNumber());
+        assertEquals(Res.get("validation.phone.tooManyDigits", "51288801505128880150"), validationResult.errorMessage);
+
+        // If phone number not exactly the requiredLength, there's a message for that too
+        validationResult = validator.validate("+1 888 123 456");
+        assertFalse(validationResult.isValid, "+1 888 123 456 should not be a valid number in US");
+        assertNull(validator.getNormalizedPhoneNumber());
+        assertEquals(Res.get("validation.phone.incorrectLength", validator.getRequiredLength()), validationResult.errorMessage);
+    }
+
 }