Completed
Push — master ( 4d2ac5...1a75da )
by Portey
03:12
created
src/Validator/ConfigValidator/ConfigValidator.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,9 @@
 block discarded – undo
36 36
 
37 37
     public function validate($data, $rules = [], $extraFieldsAllowed = null)
38 38
     {
39
-        if ($extraFieldsAllowed !== null) $this->setExtraFieldsAllowed($extraFieldsAllowed);
39
+        if ($extraFieldsAllowed !== null) {
40
+            $this->setExtraFieldsAllowed($extraFieldsAllowed);
41
+        }
40 42
 
41 43
         $processedFields = [];
42 44
         foreach ($rules as $fieldName => $fieldRules) {
Please login to merge, or discard this patch.
src/Validator/ConfigValidator/Rules/TypeValidationRule.php 1 patch
Braces   +21 added lines, -7 removed lines patch added patch discarded remove patch
@@ -79,7 +79,9 @@  discard block
 block discarded – undo
79 79
 
80 80
     private function isArrayOfValues($data)
81 81
     {
82
-        if (!is_array($data)) return false;
82
+        if (!is_array($data)) {
83
+            return false;
84
+        }
83 85
 
84 86
         foreach ($data as $item) {
85 87
             if (!array_key_exists('value', $item)) {
@@ -92,10 +94,14 @@  discard block
 block discarded – undo
92 94
 
93 95
     private function isArrayOfFields($data)
94 96
     {
95
-        if (!is_array($data)) return false;
97
+        if (!is_array($data)) {
98
+            return false;
99
+        }
96 100
 
97 101
         foreach ($data as $name => $item) {
98
-            if (!$this->isField($item, $name)) return false;
102
+            if (!$this->isField($item, $name)) {
103
+                return false;
104
+            }
99 105
         }
100 106
 
101 107
         return true;
@@ -109,7 +115,9 @@  discard block
 block discarded – undo
109 115
 
110 116
         try {
111 117
             /** @todo need to change it to optimize performance */
112
-            if (empty($data['name'])) $data['name'] = $name;
118
+            if (empty($data['name'])) {
119
+                $data['name'] = $name;
120
+            }
113 121
 
114 122
             $config = new FieldConfig($data);
115 123
 
@@ -123,10 +131,14 @@  discard block
 block discarded – undo
123 131
 
124 132
     private function isArrayOfInputs($data)
125 133
     {
126
-        if (!is_array($data)) return false;
134
+        if (!is_array($data)) {
135
+            return false;
136
+        }
127 137
 
128 138
         foreach ($data as $name => $item) {
129
-            if (!$this->isInputField($item, $name)) return false;
139
+            if (!$this->isInputField($item, $name)) {
140
+                return false;
141
+            }
130 142
         }
131 143
 
132 144
         return true;
@@ -139,7 +151,9 @@  discard block
 block discarded – undo
139 151
         }
140 152
         try {
141 153
             /** @todo need to change it to optimize performance */
142
-            if (empty($data['name'])) $data['name'] = $name;
154
+            if (empty($data['name'])) {
155
+                $data['name'] = $name;
156
+            }
143 157
 
144 158
             $config = new InputFieldConfig($data);
145 159
 
Please login to merge, or discard this patch.