@@ -37,7 +37,9 @@ |
||
37 | 37 | |
38 | 38 | private function nextNumericIndex() { |
39 | 39 | $numericKeys = array_filter(array_keys($this->rawValues()), 'is_numeric'); |
40 | - if (count($numericKeys) === 0) return 0; |
|
40 | + if (count($numericKeys) === 0) { |
|
41 | + return 0; |
|
42 | + } |
|
41 | 43 | return max($numericKeys) + 1; |
42 | 44 | } |
43 | 45 | } |
44 | 46 | \ No newline at end of file |
@@ -15,8 +15,9 @@ |
||
15 | 15 | } |
16 | 16 | |
17 | 17 | public function find($path, $default=null, $delimiter='.') { |
18 | - if ($this->pathStartsWithConfig($path, $delimiter)) |
|
19 | - return $this->evalSubPath($path, $delimiter, $default); |
|
18 | + if ($this->pathStartsWithConfig($path, $delimiter)) { |
|
19 | + return $this->evalSubPath($path, $delimiter, $default); |
|
20 | + } |
|
20 | 21 | |
21 | 22 | return $this->get($this->getFirstToken($path, $delimiter), $default); |
22 | 23 | } |
@@ -58,8 +58,9 @@ |
||
58 | 58 | public function __isset($offset) { return $this->offsetExists($offset); } |
59 | 59 | |
60 | 60 | protected function requireOffset($offset) { |
61 | - if (!$this->offsetExists($offset)) |
|
62 | - throw new \OutOfBoundsException("offset '{$offset}' does not exist"); |
|
61 | + if (!$this->offsetExists($offset)) { |
|
62 | + throw new \OutOfBoundsException("offset '{$offset}' does not exist"); |
|
63 | + } |
|
63 | 64 | |
64 | 65 | return $this->values[$offset]; |
65 | 66 | } |
@@ -17,22 +17,25 @@ |
||
17 | 17 | |
18 | 18 | public function isValid() { |
19 | 19 | foreach ($this->_fields as $field) { |
20 | - if (!$this->isFieldValid($field)) |
|
21 | - return false; |
|
20 | + if (!$this->isFieldValid($field)) { |
|
21 | + return false; |
|
22 | + } |
|
22 | 23 | } |
23 | 24 | return true; |
24 | 25 | } |
25 | 26 | |
26 | 27 | public function validate() { |
27 | - if (!$this->isValid()) |
|
28 | - throw new InvalidConfigStateException($this); |
|
28 | + if (!$this->isValid()) { |
|
29 | + throw new InvalidConfigStateException($this); |
|
30 | + } |
|
29 | 31 | } |
30 | 32 | |
31 | 33 | public function validationMessages() { |
32 | 34 | $failures = []; |
33 | 35 | foreach ($this->_fields as $field) { |
34 | - if (!$this->isFieldValid($field)) |
|
35 | - $failures[$field->getName()] = $this->fieldValidationMessages($field); |
|
36 | + if (!$this->isFieldValid($field)) { |
|
37 | + $failures[$field->getName()] = $this->fieldValidationMessages($field); |
|
38 | + } |
|
36 | 39 | } |
37 | 40 | return $failures; |
38 | 41 | } |
@@ -23,8 +23,9 @@ discard block |
||
23 | 23 | $this->validateNotNull($key); |
24 | 24 | $this->validateNotObjectOrHasToString($key); |
25 | 25 | |
26 | - if (is_numeric($key) || is_string($key)) |
|
27 | - return $key; |
|
26 | + if (is_numeric($key) || is_string($key)) { |
|
27 | + return $key; |
|
28 | + } |
|
28 | 29 | |
29 | 30 | return (string) $key; |
30 | 31 | } |
@@ -34,8 +35,9 @@ discard block |
||
34 | 35 | * @throws InvalidTypeException |
35 | 36 | */ |
36 | 37 | private function validateNotNull($key) { |
37 | - if (is_null($key)) |
|
38 | - throw new InvalidTypeException('Can not use null for a key'); |
|
38 | + if (is_null($key)) { |
|
39 | + throw new InvalidTypeException('Can not use null for a key'); |
|
40 | + } |
|
39 | 41 | } |
40 | 42 | |
41 | 43 | /** |
@@ -43,7 +45,8 @@ discard block |
||
43 | 45 | * @throws InvalidTypeException |
44 | 46 | */ |
45 | 47 | private function validateNotObjectOrHasToString($key) { |
46 | - if (is_object($key) && !method_exists($key, '__toString')) |
|
47 | - throw new InvalidTypeException('Object keys must implement __toString'); |
|
48 | + if (is_object($key) && !method_exists($key, '__toString')) { |
|
49 | + throw new InvalidTypeException('Object keys must implement __toString'); |
|
50 | + } |
|
48 | 51 | } |
49 | 52 | } |
50 | 53 | \ No newline at end of file |
@@ -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 | } |
@@ -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 |
@@ -15,8 +15,9 @@ |
||
15 | 15 | protected $validation; |
16 | 16 | |
17 | 17 | public function __construct($name) { |
18 | - if (!$this->isValidName($name)) |
|
19 | - throw new InvalidFieldNameException(); |
|
18 | + if (!$this->isValidName($name)) { |
|
19 | + throw new InvalidFieldNameException(); |
|
20 | + } |
|
20 | 21 | $this->name = $name; |
21 | 22 | |
22 | 23 | $this->validation = new Validation(); |
@@ -13,15 +13,17 @@ |
||
13 | 13 | protected $validators = []; |
14 | 14 | |
15 | 15 | public function addValidator(Validator ...$validators) { |
16 | - foreach($validators as $v) |
|
17 | - array_push($this->validators, $v); |
|
16 | + foreach($validators as $v) { |
|
17 | + array_push($this->validators, $v); |
|
18 | + } |
|
18 | 19 | } |
19 | 20 | |
20 | 21 | public function validate($value): Result { |
21 | 22 | $messages = []; |
22 | 23 | foreach ($this->validators as $validator) { |
23 | - if (!$validator->validate($value)) |
|
24 | - array_push($messages, $validator->getDescription()); |
|
24 | + if (!$validator->validate($value)) { |
|
25 | + array_push($messages, $validator->getDescription()); |
|
26 | + } |
|
25 | 27 | } |
26 | 28 | return count($messages) |
27 | 29 | ? new Validation\InvalidResult($messages) |