@@ -14,8 +14,9 @@ discard block |
||
14 | 14 | protected $validators = []; |
15 | 15 | |
16 | 16 | public function __construct($name) { |
17 | - if (!$this->isValidName($name)) |
|
18 | - throw new InvalidFieldNameException(); |
|
17 | + if (!$this->isValidName($name)) { |
|
18 | + throw new InvalidFieldNameException(); |
|
19 | + } |
|
19 | 20 | $this->name = $name; |
20 | 21 | } |
21 | 22 | |
@@ -46,11 +47,13 @@ discard block |
||
46 | 47 | } |
47 | 48 | |
48 | 49 | public function validate($value): Validation\Result { |
49 | - if ($this->isRequiredOrNotEmpty($value)) |
|
50 | - $this->runValidators($value); |
|
50 | + if ($this->isRequiredOrNotEmpty($value)) { |
|
51 | + $this->runValidators($value); |
|
52 | + } |
|
51 | 53 | |
52 | - if ($this->isRequiredAndEmpty($value) && count($this->messages) === 0) |
|
53 | - $this->addMessage('Required'); |
|
54 | + if ($this->isRequiredAndEmpty($value) && count($this->messages) === 0) { |
|
55 | + $this->addMessage('Required'); |
|
56 | + } |
|
54 | 57 | |
55 | 58 | return $this->validationResult(); |
56 | 59 | } |
@@ -89,8 +92,9 @@ discard block |
||
89 | 92 | |
90 | 93 | protected function runValidators($value) { |
91 | 94 | foreach ($this->validators as $validator) { |
92 | - if (!$validator->validate($value)) |
|
93 | - $this->addMessage($validator->getDescription()); |
|
95 | + if (!$validator->validate($value)) { |
|
96 | + $this->addMessage($validator->getDescription()); |
|
97 | + } |
|
94 | 98 | } |
95 | 99 | } |
96 | 100 |
@@ -42,8 +42,9 @@ |
||
42 | 42 | $particleResult = $this->validator->validate([ |
43 | 43 | $this->name => $value |
44 | 44 | ]); |
45 | - if ($particleResult->isValid()) |
|
46 | - return new Validation\ValidResult(); |
|
45 | + if ($particleResult->isValid()) { |
|
46 | + return new Validation\ValidResult(); |
|
47 | + } |
|
47 | 48 | |
48 | 49 | return $this->invalidResult($particleResult); |
49 | 50 | } |
@@ -10,7 +10,7 @@ discard block |
||
10 | 10 | * @see ../docs/config/README.md |
11 | 11 | */ |
12 | 12 | abstract class Config extends Registry { |
13 | - private $locked = false; |
|
13 | + private $locked = false; |
|
14 | 14 | |
15 | 15 | /** |
16 | 16 | * Config constructor. |
@@ -51,7 +51,7 @@ discard block |
||
51 | 51 | |
52 | 52 | public function toArray() { |
53 | 53 | return array_map( |
54 | - function ($value) { |
|
54 | + function($value) { |
|
55 | 55 | // $value may be another Config object, if so then we toArray() it also. |
56 | 56 | return $this->hasToArray($value) ? $value->toArray() : $value; |
57 | 57 | }, |
@@ -26,7 +26,9 @@ discard block |
||
26 | 26 | } |
27 | 27 | |
28 | 28 | public function import(array $data) { |
29 | - foreach ($data as $key=>$value) $this->offsetSet($key, $value); |
|
29 | + foreach ($data as $key=>$value) { |
|
30 | + $this->offsetSet($key, $value); |
|
31 | + } |
|
30 | 32 | } |
31 | 33 | |
32 | 34 | // override this to have a base config |
@@ -82,7 +84,9 @@ discard block |
||
82 | 84 | } |
83 | 85 | |
84 | 86 | protected function settableValue($value) { |
85 | - if (is_array($value)) return $this->subConfig($value); |
|
87 | + if (is_array($value)) { |
|
88 | + return $this->subConfig($value); |
|
89 | + } |
|
86 | 90 | return $value; |
87 | 91 | } |
88 | 92 | |
@@ -116,7 +120,8 @@ discard block |
||
116 | 120 | * @throws CanNotMutateException |
117 | 121 | */ |
118 | 122 | private function blockIfLocked() { |
119 | - if ($this->isLocked()) |
|
120 | - throw new CanNotMutateException(); |
|
123 | + if ($this->isLocked()) { |
|
124 | + throw new CanNotMutateException(); |
|
125 | + } |
|
121 | 126 | } |
122 | 127 | } |
123 | 128 | \ No newline at end of file |