@@ -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 |