MC-39885: Improve customer api

Viktor Sevch · Viktor Sevch · commit 3bade65291d0 · 2021-02-26T08:54:12.000+02:00
diff --git a/app/code/Magento/Customer/Model/Validator/Name.php b/app/code/Magento/Customer/Model/Validator/Name.php
@@ -15,6 +15,8 @@
  */
 class Name extends AbstractValidator
 {
+    const PATTERN_NAME = '/(?:[\p{L}\p{M}\,\-\_\.\'\s\d]){1,255}+/u';
+
     /**
      * Validate name fields.
      *
@@ -47,8 +49,7 @@ public function isValid($customer)
     private function isValidName($nameValue)
     {
         if ($nameValue != null) {
-            $pattern = '/(?:[\p{L}\p{M}\,\-\_\.\'\"\s\d]){1,255}+/u';
-            if (preg_match($pattern, $nameValue, $matches)) {
+            if (preg_match(self::PATTERN_NAME, $nameValue, $matches)) {
                 return $matches[0] == $nameValue;
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,8 @@`
`15`	`15`	`*/`
`16`	`16`	`class Name extends AbstractValidator`
`17`	`17`	`{`
	`18`	`+ const PATTERN_NAME = '/(?:[\p{L}\p{M}\,\-\_\.\'\s\d]){1,255}+/u';`
	`19`	`+`
`18`	`20`	`/**`
`19`	`21`	`* Validate name fields.`
`20`	`22`	`*`
`@@ -47,8 +49,7 @@ public function isValid($customer)`
`47`	`49`	`private function isValidName($nameValue)`
`48`	`50`	`{`
`49`	`51`	`if ($nameValue != null) {`
`50`		`- $pattern = '/(?:[\p{L}\p{M}\,\-\_\.\'\"\s\d]){1,255}+/u';`
`51`		`- if (preg_match($pattern, $nameValue, $matches)) {`
	`52`	`+ if (preg_match(self::PATTERN_NAME, $nameValue, $matches)) {`
`52`	`53`	`return $matches[0] == $nameValue;`
`53`	`54`	`}`
`54`	`55`	`}`