@@ -36,7 +36,9 @@ |
||
| 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) { |
@@ -89,7 +89,9 @@ discard block |
||
| 89 | 89 | |
| 90 | 90 | private function isArrayOfValues($data) |
| 91 | 91 | { |
| 92 | - if (!is_array($data)) return false; |
|
| 92 | + if (!is_array($data)) { |
|
| 93 | + return false; |
|
| 94 | + } |
|
| 93 | 95 | |
| 94 | 96 | foreach ($data as $item) { |
| 95 | 97 | if (!array_key_exists('value', $item)) { |
@@ -102,7 +104,9 @@ discard block |
||
| 102 | 104 | |
| 103 | 105 | private static function isArrayOfInterfaces($data) |
| 104 | 106 | { |
| 105 | - if (!is_array($data)) return false; |
|
| 107 | + if (!is_array($data)) { |
|
| 108 | + return false; |
|
| 109 | + } |
|
| 106 | 110 | |
| 107 | 111 | foreach ($data as $item) { |
| 108 | 112 | if (!TypeMap::isInterface($item)) { |
@@ -115,10 +119,14 @@ discard block |
||
| 115 | 119 | |
| 116 | 120 | private function isArrayOfFields($data) |
| 117 | 121 | { |
| 118 | - if (!is_array($data)) return false; |
|
| 122 | + if (!is_array($data)) { |
|
| 123 | + return false; |
|
| 124 | + } |
|
| 119 | 125 | |
| 120 | 126 | foreach ($data as $name => $item) { |
| 121 | - if (!$this->isField($item, $name)) return false; |
|
| 127 | + if (!$this->isField($item, $name)) { |
|
| 128 | + return false; |
|
| 129 | + } |
|
| 122 | 130 | } |
| 123 | 131 | |
| 124 | 132 | return true; |
@@ -132,7 +140,9 @@ discard block |
||
| 132 | 140 | |
| 133 | 141 | try { |
| 134 | 142 | /** @todo need to change it to optimize performance */ |
| 135 | - if (empty($data['name'])) $data['name'] = $name; |
|
| 143 | + if (empty($data['name'])) { |
|
| 144 | + $data['name'] = $name; |
|
| 145 | + } |
|
| 136 | 146 | |
| 137 | 147 | $config = new FieldConfig($data); |
| 138 | 148 | |
@@ -146,10 +156,14 @@ discard block |
||
| 146 | 156 | |
| 147 | 157 | private function isArrayOfInputs($data) |
| 148 | 158 | { |
| 149 | - if (!is_array($data)) return false; |
|
| 159 | + if (!is_array($data)) { |
|
| 160 | + return false; |
|
| 161 | + } |
|
| 150 | 162 | |
| 151 | 163 | foreach ($data as $name => $item) { |
| 152 | - if (!$this->isInputField($item, $name)) return false; |
|
| 164 | + if (!$this->isInputField($item, $name)) { |
|
| 165 | + return false; |
|
| 166 | + } |
|
| 153 | 167 | } |
| 154 | 168 | |
| 155 | 169 | return true; |
@@ -162,7 +176,9 @@ discard block |
||
| 162 | 176 | } |
| 163 | 177 | try { |
| 164 | 178 | /** @todo need to change it to optimize performance */ |
| 165 | - if (empty($data['name'])) $data['name'] = $name; |
|
| 179 | + if (empty($data['name'])) { |
|
| 180 | + $data['name'] = $name; |
|
| 181 | + } |
|
| 166 | 182 | |
| 167 | 183 | $config = new InputFieldConfig($data); |
| 168 | 184 | |