Passed
Push — master ( a9840c...5ab0c2 )
by Todd
03:34
created
src/Config.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
         },
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -19,8 +19,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
Please login to merge, or discard this patch.
src/Config/StrictConfig.php 1 patch
Braces   +9 added lines, -6 removed lines patch added patch discarded remove patch
@@ -17,22 +17,25 @@
 block discarded – undo
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
   }
Please login to merge, or discard this patch.
src/Config/Field/Adapter/Particle.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -24,12 +24,12 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -41,8 +41,9 @@
 block discarded – undo
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
   }
Please login to merge, or discard this patch.
src/Config/Field/Validation/Validator/Regex.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -14,8 +14,9 @@
 block discarded – undo
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
   }
Please login to merge, or discard this patch.
src/Config/Field/Validation/Validator/IsInstanceOf.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -14,8 +14,9 @@
 block discarded – undo
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}";
Please login to merge, or discard this patch.
src/Config/Field/Field.php 1 patch
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -16,8 +16,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.
src/Pair.php 1 patch
Braces   +9 added lines, -6 removed lines patch added patch discarded remove patch
@@ -20,14 +20,17 @@
 block discarded – undo
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
   }
Please login to merge, or discard this patch.