You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "sebbASF (via GitHub)" <gi...@apache.org> on 2023/01/26 23:44:00 UTC
[GitHub] [commons-validator] sebbASF commented on a diff in pull request #116: Adding validation support for Mayotte (YT) IBANs
sebbASF commented on code in PR #116:
URL: https://github.com/apache/commons-validator/pull/116#discussion_r1088446205
##########
src/main/java/org/apache/commons/validator/routines/IBANValidator.java:
##########
@@ -141,6 +141,7 @@ public RegexValidator getRegexValidator() {
new Validator("BG", 22, "BG\\d{2}[A-Z]{4}\\d{6}[A-Z0-9]{8}" ), // Bulgaria
new Validator("BH", 22, "BH\\d{2}[A-Z]{4}[A-Z0-9]{14}" ), // Bahrain
new Validator("BI", 27, "BI\\d{25}" ), // Burundi
+ new Validator("BL", 27, "BL\\d{12}[A-Z0-9]{11}\\d{2}" ), // Saint Barhelemy
Review Comment:
The comment ought to indicate that these are included in another country code
##########
src/main/java/org/apache/commons/validator/routines/IBANValidator.java:
##########
@@ -206,13 +215,16 @@ public RegexValidator getRegexValidator() {
new Validator("SM", 27, "SM\\d{2}[A-Z]{1}\\d{10}[A-Z0-9]{12}" ), // San Marino
new Validator("ST", 25, "ST\\d{23}" ), // Sao Tome and Principe
new Validator("SV", 28, "SV\\d{2}[A-Z]{4}\\d{20}" ), // El Salvador
+ new Validator("TF", 27, "TF\\d{12}[A-Z0-9]{11}\\d{2}" ), // French Southern Territories
new Validator("TL", 23, "TL\\d{21}" ), // Timor-Leste
new Validator("TN", 24, "TN\\d{22}" ), // Tunisia
new Validator("TR", 26, "TR\\d{8}[A-Z0-9]{16}" ), // Turkey
new Validator("UA", 29, "UA\\d{8}[A-Z0-9]{19}" ), // Ukraine
new Validator("VA", 22, "VA\\d{20}" ), // Vatican City State
new Validator("VG", 24, "VG\\d{2}[A-Z]{4}\\d{16}" ), // Virgin Islands
+ new Validator("WF", 27, "WF\\d{12}[A-Z0-9]{11}\\d{2}" ), // Wallis and Futuna Islands
new Validator("XK", 20, "XK\\d{18}" ), // Kosovo
+ new Validator("YT", 27, "YT\\d{12}[A-Z0-9]{11}\\d{2}" ) // Mayotte
};
/** The singleton instance which uses the default formats */
Review Comment:
Also, it's not only France that has these additional includes; e.g. GB has some and there are others
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org