@@ -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. |
@@ -19,7 +19,7 @@ discard block |
||
| 19 | 19 | */ |
| 20 | 20 | public function __construct(array $arrayConfig = []) { |
| 21 | 21 | parent::__construct(); |
| 22 | - foreach($arrayConfig as $key => $value) |
|
| 22 | + foreach ($arrayConfig as $key => $value) |
|
| 23 | 23 | $this->offsetSet($key, $value); |
| 24 | 24 | |
| 25 | 25 | $this->onConstruct(); |
@@ -42,7 +42,7 @@ discard block |
||
| 42 | 42 | |
| 43 | 43 | public function toArray() { |
| 44 | 44 | return array_map( |
| 45 | - function ($value) { |
|
| 45 | + function($value) { |
|
| 46 | 46 | // $value may be another Config object, if so then we toArray() it also. |
| 47 | 47 | return $this->hasToArray($value) ? $value->toArray() : $value; |
| 48 | 48 | }, |
@@ -19,8 +19,9 @@ discard block |
||
| 19 | 19 | */ |
| 20 | 20 | public function __construct(array $arrayConfig = []) { |
| 21 | 21 | parent::__construct(); |
| 22 | - foreach($arrayConfig as $key => $value) |
|
| 23 | - $this->offsetSet($key, $value); |
|
| 22 | + foreach($arrayConfig as $key => $value) { |
|
| 23 | + $this->offsetSet($key, $value); |
|
| 24 | + } |
|
| 24 | 25 | |
| 25 | 26 | $this->onConstruct(); |
| 26 | 27 | } |
@@ -101,7 +102,8 @@ discard block |
||
| 101 | 102 | * @throws CanNotMutateException |
| 102 | 103 | */ |
| 103 | 104 | private function blockIfLocked() { |
| 104 | - if ($this->isLocked()) |
|
| 105 | - throw new CanNotMutateException(); |
|
| 105 | + if ($this->isLocked()) { |
|
| 106 | + throw new CanNotMutateException(); |
|
| 107 | + } |
|
| 106 | 108 | } |
| 107 | 109 | } |
| 108 | 110 | \ No newline at end of file |
@@ -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 | } |
@@ -24,12 +24,12 @@ |
||
| 24 | 24 | } |
| 25 | 25 | |
| 26 | 26 | public static function required($name) : Particle { |
| 27 | - $v = new Particle\Validator();; |
|
| 27 | + $v = new Particle\Validator(); ; |
|
| 28 | 28 | return new self($name, $v, $v->required($name)); |
| 29 | 29 | } |
| 30 | 30 | |
| 31 | 31 | public static function optional($name) : Particle { |
| 32 | - $v = new Particle\Validator();; |
|
| 32 | + $v = new Particle\Validator(); ; |
|
| 33 | 33 | return new self($name, $v, $v->optional($name)); |
| 34 | 34 | } |
| 35 | 35 | |
@@ -41,8 +41,9 @@ |
||
| 41 | 41 | $particleResult = $this->validator->validate([ |
| 42 | 42 | $this->name => $value |
| 43 | 43 | ]); |
| 44 | - if ($particleResult->isValid()) |
|
| 45 | - return new Field\Validation\ValidResult(); |
|
| 44 | + if ($particleResult->isValid()) { |
|
| 45 | + return new Field\Validation\ValidResult(); |
|
| 46 | + } |
|
| 46 | 47 | |
| 47 | 48 | return $this->invalidResult($particleResult); |
| 48 | 49 | } |
@@ -14,8 +14,9 @@ |
||
| 14 | 14 | * @throws Exception |
| 15 | 15 | */ |
| 16 | 16 | public function __construct($pattern, $description = null) { |
| 17 | - if (!$this->isValidRegexPattern($pattern)) |
|
| 18 | - throw new Exception("Invalid Regex pattern: {$pattern}"); |
|
| 17 | + if (!$this->isValidRegexPattern($pattern)) { |
|
| 18 | + throw new Exception("Invalid Regex pattern: {$pattern}"); |
|
| 19 | + } |
|
| 19 | 20 | $this->pattern = $pattern; |
| 20 | 21 | $this->message = $description; |
| 21 | 22 | } |
@@ -14,8 +14,9 @@ |
||
| 14 | 14 | * @throws Exception |
| 15 | 15 | */ |
| 16 | 16 | public function __construct($fqcn, $description = null) { |
| 17 | - if (!$this->isValidClassName($fqcn)) |
|
| 18 | - throw new Exception("Invalid Regex pattern: {$fqcn}"); |
|
| 17 | + if (!$this->isValidClassName($fqcn)) { |
|
| 18 | + throw new Exception("Invalid Regex pattern: {$fqcn}"); |
|
| 19 | + } |
|
| 19 | 20 | |
| 20 | 21 | $this->fqcn = $fqcn; |
| 21 | 22 | $this->description = $description ?? "Must be an instance of {$fqcn}"; |
@@ -16,8 +16,9 @@ discard block |
||
| 16 | 16 | protected $validators = []; |
| 17 | 17 | |
| 18 | 18 | public function __construct($name) { |
| 19 | - if (!$this->isValidName($name)) |
|
| 20 | - throw new InvalidFieldNameException(); |
|
| 19 | + if (!$this->isValidName($name)) { |
|
| 20 | + throw new InvalidFieldNameException(); |
|
| 21 | + } |
|
| 21 | 22 | $this->name = $name; |
| 22 | 23 | } |
| 23 | 24 | |
@@ -48,11 +49,13 @@ discard block |
||
| 48 | 49 | } |
| 49 | 50 | |
| 50 | 51 | public function validate($value): ValidationResult { |
| 51 | - if ($this->isRequired() || $this->isNotEmpty($value)) |
|
| 52 | - $this->runValidators($value); |
|
| 52 | + if ($this->isRequired() || $this->isNotEmpty($value)) { |
|
| 53 | + $this->runValidators($value); |
|
| 54 | + } |
|
| 53 | 55 | |
| 54 | - if ($this->isRequiredAndEmpty($value) && count($this->messages) === 0) |
|
| 55 | - $this->addMessage('Required'); |
|
| 56 | + if ($this->isRequiredAndEmpty($value) && count($this->messages) === 0) { |
|
| 57 | + $this->addMessage('Required'); |
|
| 58 | + } |
|
| 56 | 59 | |
| 57 | 60 | return $this->validationResult(); |
| 58 | 61 | } |
@@ -87,8 +90,9 @@ discard block |
||
| 87 | 90 | |
| 88 | 91 | protected function runValidators($value) { |
| 89 | 92 | foreach ($this->validators as $validator) { |
| 90 | - if (!$validator->validate($value)) |
|
| 91 | - $this->addMessage($validator->getDescription()); |
|
| 93 | + if (!$validator->validate($value)) { |
|
| 94 | + $this->addMessage($validator->getDescription()); |
|
| 95 | + } |
|
| 92 | 96 | } |
| 93 | 97 | } |
| 94 | 98 | |
@@ -20,14 +20,17 @@ |
||
| 20 | 20 | } |
| 21 | 21 | |
| 22 | 22 | private function getScalar($key) { |
| 23 | - if (is_null($key)) |
|
| 24 | - throw new InvalidTypeException('Can not use null for a key'); |
|
| 23 | + if (is_null($key)) { |
|
| 24 | + throw new InvalidTypeException('Can not use null for a key'); |
|
| 25 | + } |
|
| 25 | 26 | |
| 26 | - if (is_object($key) && !method_exists($key, '__toString')) |
|
| 27 | - throw new InvalidTypeException('Object keys must implement __toString'); |
|
| 27 | + if (is_object($key) && !method_exists($key, '__toString')) { |
|
| 28 | + throw new InvalidTypeException('Object keys must implement __toString'); |
|
| 29 | + } |
|
| 28 | 30 | |
| 29 | - if (is_numeric($key) || is_string($key)) |
|
| 30 | - return $key; |
|
| 31 | + if (is_numeric($key) || is_string($key)) { |
|
| 32 | + return $key; |
|
| 33 | + } |
|
| 31 | 34 | |
| 32 | 35 | return (string) $key; |
| 33 | 36 | } |