@@ -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) { |
@@ -61,7 +61,7 @@ |
||
| 61 | 61 | } |
| 62 | 62 | |
| 63 | 63 | /** |
| 64 | - * @return Field[]|Query[] |
|
| 64 | + * @return Query[] |
|
| 65 | 65 | */ |
| 66 | 66 | public function getFields() |
| 67 | 67 | { |
@@ -61,7 +61,7 @@ |
||
| 61 | 61 | } |
| 62 | 62 | |
| 63 | 63 | /** |
| 64 | - * @return Field[]|Query[] |
|
| 64 | + * @return Query[] |
|
| 65 | 65 | */ |
| 66 | 66 | public function getFields() |
| 67 | 67 | { |
@@ -189,7 +189,7 @@ discard block |
||
| 189 | 189 | /** |
| 190 | 190 | * @param $objectType InputObjectType|ObjectType |
| 191 | 191 | * @param $query Mutation|Query |
| 192 | - * @return null |
|
| 192 | + * @return boolean |
|
| 193 | 193 | */ |
| 194 | 194 | private function checkFieldExist($objectType, $query) |
| 195 | 195 | { |
@@ -226,7 +226,7 @@ discard block |
||
| 226 | 226 | } |
| 227 | 227 | |
| 228 | 228 | /** |
| 229 | - * @param $field Field |
|
| 229 | + * @param Field $field Field |
|
| 230 | 230 | * @param $contextValue mixed |
| 231 | 231 | * @param $query Query |
| 232 | 232 | * |
@@ -79,7 +79,9 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 | |