@@ -23,7 +23,7 @@ discard block |
||
23 | 23 | /** |
24 | 24 | * @var array $rules |
25 | 25 | */ |
26 | - private $rules = []; |
|
26 | + private $rules = [ ]; |
|
27 | 27 | |
28 | 28 | public function __construct(Rule... $rules) |
29 | 29 | { |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | */ |
38 | 38 | public function validate($data): Result |
39 | 39 | { |
40 | - $errors = []; |
|
40 | + $errors = [ ]; |
|
41 | 41 | foreach ($this->rules as $rule) { |
42 | 42 | |
43 | 43 | $result = $rule->validate($data); |
@@ -49,7 +49,7 @@ discard block |
||
49 | 49 | * @var Error $error |
50 | 50 | */ |
51 | 51 | foreach ($result->getErrors() as $error) { |
52 | - $errors[] = $error; |
|
52 | + $errors[ ] = $error; |
|
53 | 53 | } |
54 | 54 | } |
55 | 55 |
@@ -55,9 +55,9 @@ |
||
55 | 55 | |
56 | 56 | } |
57 | 57 | |
58 | - if (empty($errors)) |
|
59 | - return new Success(); |
|
60 | - else { |
|
58 | + if (empty($errors)) { |
|
59 | + return new Success(); |
|
60 | + } else { |
|
61 | 61 | return new Failure(...$errors); // Use the splat operator |
62 | 62 | |
63 | 63 | } |
@@ -90,7 +90,7 @@ |
||
90 | 90 | $containsLower = preg_match('/[a-z]/', $text); |
91 | 91 | |
92 | 92 | if ($this->strictCheck) |
93 | - return !preg_match('/[\W]/', $text) && $containsLower; |
|
93 | + return !preg_match('/[\W]/', $text) && $containsLower; |
|
94 | 94 | else |
95 | 95 | return $containsLower; |
96 | 96 | } |
@@ -44,9 +44,10 @@ discard block |
||
44 | 44 | */ |
45 | 45 | public function __construct(int $caseType, bool $allowSpecialChars = true) |
46 | 46 | { |
47 | - if ($caseType > 4 || $caseType < 0) |
|
48 | - throw new \InvalidArgumentException('Case Type ' . $caseType . ' is invalid. |
|
47 | + if ($caseType > 4 || $caseType < 0) { |
|
48 | + throw new \InvalidArgumentException('Case Type ' . $caseType . ' is invalid. |
|
49 | 49 | Check the class constants available to be used as caseTypes'); |
50 | + } |
|
50 | 51 | |
51 | 52 | $this->caseType = $caseType; |
52 | 53 | $this->strictCheck = !$allowSpecialChars; |
@@ -59,10 +60,11 @@ discard block |
||
59 | 60 | private function isMixed(string $text) : bool |
60 | 61 | { |
61 | 62 | if ($this->strictCheck) { |
62 | - if (preg_match("/^[a-zA-Z]+$/", $text)) |
|
63 | - return preg_match('/[a-z]/', $text) && preg_match('/[A-Z]/', $text); |
|
64 | - else |
|
65 | - return false; |
|
63 | + if (preg_match("/^[a-zA-Z]+$/", $text)) { |
|
64 | + return preg_match('/[a-z]/', $text) && preg_match('/[A-Z]/', $text); |
|
65 | + } else { |
|
66 | + return false; |
|
67 | + } |
|
66 | 68 | } |
67 | 69 | |
68 | 70 | return preg_match('/[a-z]/', $text) && preg_match('/[A-Z]/', $text); |
@@ -70,39 +72,43 @@ discard block |
||
70 | 72 | |
71 | 73 | private function isAllUpper(string $text) : bool |
72 | 74 | { |
73 | - if ($this->strictCheck) |
|
74 | - return ctype_upper($text); |
|
75 | - else |
|
76 | - return !preg_match('/[a-z]/', $text) && preg_match('/[A-Z]/', $text); |
|
75 | + if ($this->strictCheck) { |
|
76 | + return ctype_upper($text); |
|
77 | + } else { |
|
78 | + return !preg_match('/[a-z]/', $text) && preg_match('/[A-Z]/', $text); |
|
79 | + } |
|
77 | 80 | |
78 | 81 | } |
79 | 82 | |
80 | 83 | private function isAllLower(string $text) : bool |
81 | 84 | { |
82 | - if ($this->strictCheck) |
|
83 | - return ctype_lower($text); |
|
84 | - else |
|
85 | - return preg_match('/[a-z]/', $text) && !preg_match('/[A-Z]/', $text); |
|
85 | + if ($this->strictCheck) { |
|
86 | + return ctype_lower($text); |
|
87 | + } else { |
|
88 | + return preg_match('/[a-z]/', $text) && !preg_match('/[A-Z]/', $text); |
|
89 | + } |
|
86 | 90 | } |
87 | 91 | |
88 | 92 | private function isSomeLower(string $text) : bool |
89 | 93 | { |
90 | 94 | $containsLower = preg_match('/[a-z]/', $text); |
91 | 95 | |
92 | - if ($this->strictCheck) |
|
93 | - return !preg_match('/[\W]/', $text) && $containsLower; |
|
94 | - else |
|
95 | - return $containsLower; |
|
96 | + if ($this->strictCheck) { |
|
97 | + return !preg_match('/[\W]/', $text) && $containsLower; |
|
98 | + } else { |
|
99 | + return $containsLower; |
|
100 | + } |
|
96 | 101 | } |
97 | 102 | |
98 | 103 | private function isSomeUpper(string $text) : bool |
99 | 104 | { |
100 | 105 | $containsUpper = preg_match('/[A-Z]/', $text); |
101 | 106 | |
102 | - if ($this->strictCheck) |
|
103 | - return !preg_match('/[\W]/', $text) && $containsUpper; |
|
104 | - else |
|
105 | - return $containsUpper; |
|
107 | + if ($this->strictCheck) { |
|
108 | + return !preg_match('/[\W]/', $text) && $containsUpper; |
|
109 | + } else { |
|
110 | + return $containsUpper; |
|
111 | + } |
|
106 | 112 | } |
107 | 113 | |
108 | 114 | public function validate($data): Result |
@@ -127,10 +133,11 @@ discard block |
||
127 | 133 | break; |
128 | 134 | } |
129 | 135 | |
130 | - if ($isValid) |
|
131 | - return new Success(); |
|
132 | - else |
|
133 | - return new Failure(new RuleError(RuleErrorCode::CASING_MISMATCH, |
|
136 | + if ($isValid) { |
|
137 | + return new Success(); |
|
138 | + } else { |
|
139 | + return new Failure(new RuleError(RuleErrorCode::CASING_MISMATCH, |
|
134 | 140 | "The given string doesn't match the required case")); |
141 | + } |
|
135 | 142 | } |
136 | 143 | } |
137 | 144 | \ No newline at end of file |
@@ -19,13 +19,13 @@ |
||
19 | 19 | * Handy for username validations |
20 | 20 | * @var array $allowedSpecialChars |
21 | 21 | */ |
22 | - protected $allowedSpecialChars = []; |
|
22 | + protected $allowedSpecialChars = [ ]; |
|
23 | 23 | |
24 | 24 | /** |
25 | 25 | * AlphaNumeric constructor. |
26 | 26 | * @param array $allowedSpecialChars |
27 | 27 | */ |
28 | - public function __construct($allowedSpecialChars = []) |
|
28 | + public function __construct($allowedSpecialChars = [ ]) |
|
29 | 29 | { |
30 | 30 | $this->allowedSpecialChars = $allowedSpecialChars; |
31 | 31 | } |
@@ -23,9 +23,10 @@ |
||
23 | 23 | { |
24 | 24 | public function validate($data): Result |
25 | 25 | { |
26 | - if (ctype_alpha(str_replace($this->allowedSpecialChars, '', $data))) |
|
27 | - return new Success(); |
|
28 | - else |
|
29 | - return new Failure(new RuleError(RuleErrorCode::NOT_ALPHA, 'String is not Alphabetic.')); |
|
26 | + if (ctype_alpha(str_replace($this->allowedSpecialChars, '', $data))) { |
|
27 | + return new Success(); |
|
28 | + } else { |
|
29 | + return new Failure(new RuleError(RuleErrorCode::NOT_ALPHA, 'String is not Alphabetic.')); |
|
30 | + } |
|
30 | 31 | } |
31 | 32 | } |
32 | 33 | \ No newline at end of file |
@@ -26,17 +26,17 @@ |
||
26 | 26 | */ |
27 | 27 | private $username; |
28 | 28 | |
29 | - private function getOption(int $key, $arr = [], $default) { |
|
30 | - return isset($arr[$key]) ? $arr[$key]: $default; |
|
29 | + private function getOption(int $key, $arr = [ ], $default) { |
|
30 | + return isset($arr[ $key ]) ? $arr[ $key ] : $default; |
|
31 | 31 | } |
32 | - public function __construct(string $username, $options = [], Validator $validator = null) |
|
32 | + public function __construct(string $username, $options = [ ], Validator $validator = null) |
|
33 | 33 | { |
34 | 34 | if ($validator == null) { |
35 | 35 | // use the default validator |
36 | 36 | $validator = new UsernameValidator( |
37 | 37 | $this->getOption(self::OPT_MIN_LEN, $options, 4), |
38 | 38 | $this->getOption(self::OPT_MAX_LEN, $options, 12), |
39 | - $this->getOption(self::OPT_ALLOWED_SPECIAL_CHARS, $options, ['-', '_'])); |
|
39 | + $this->getOption(self::OPT_ALLOWED_SPECIAL_CHARS, $options, [ '-', '_' ])); |
|
40 | 40 | } |
41 | 41 | |
42 | 42 | $result = $validator->validate($username, $options); |
@@ -30,9 +30,9 @@ |
||
30 | 30 | /** |
31 | 31 | * @var array $allowedChars |
32 | 32 | */ |
33 | - private $allowedSpecialChars = []; |
|
33 | + private $allowedSpecialChars = [ ]; |
|
34 | 34 | |
35 | - public function __construct(int $minLength = 4, int $maxLength = 12, $allowedSpecialChars = []) |
|
35 | + public function __construct(int $minLength = 4, int $maxLength = 12, $allowedSpecialChars = [ ]) |
|
36 | 36 | { |
37 | 37 | $this->minLength = $minLength; |
38 | 38 | $this->maxLength = $maxLength; |