@@ -1,9 +1,9 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Date: 03.12.15 |
|
4 | - * |
|
5 | - * @author Portey Vasil <[email protected]> |
|
6 | - */ |
|
3 | + * Date: 03.12.15 |
|
4 | + * |
|
5 | + * @author Portey Vasil <[email protected]> |
|
6 | + */ |
|
7 | 7 | |
8 | 8 | namespace Youshido\GraphQL\Definition; |
9 | 9 |
@@ -1,9 +1,9 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Date: 04.12.15 |
|
4 | - * |
|
5 | - * @author Portey Vasil <[email protected]> |
|
6 | - */ |
|
3 | + * Date: 04.12.15 |
|
4 | + * |
|
5 | + * @author Portey Vasil <[email protected]> |
|
6 | + */ |
|
7 | 7 | |
8 | 8 | namespace Youshido\GraphQL\Definition\Traits; |
9 | 9 |
@@ -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 | { |
@@ -1,9 +1,9 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Date: 03.12.15 |
|
4 | - * |
|
5 | - * @author Portey Vasil <[email protected]> |
|
6 | - */ |
|
3 | + * Date: 03.12.15 |
|
4 | + * |
|
5 | + * @author Portey Vasil <[email protected]> |
|
6 | + */ |
|
7 | 7 | |
8 | 8 | namespace Youshido\GraphQL\Definition; |
9 | 9 |
@@ -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 | * |
@@ -156,7 +156,7 @@ discard block |
||
156 | 156 | $preResolvedValue = $this->getPreResolvedValue($contextValue, $query); |
157 | 157 | |
158 | 158 | if ($field->getConfig()->getType()->getKind() == TypeMap::KIND_LIST) { |
159 | - if(!is_array($preResolvedValue)){ |
|
159 | + if (!is_array($preResolvedValue)) { |
|
160 | 160 | $value = null; |
161 | 161 | $this->resolveValidator->addError(new ResolveException('Not valid resolve value for list type')); |
162 | 162 | } |
@@ -169,7 +169,7 @@ discard block |
||
169 | 169 | |
170 | 170 | if ($type->getKind() == TypeMap::KIND_ENUM) { |
171 | 171 | /** @var $type AbstractEnumType */ |
172 | - if(!$type->isValidValue($resolvedValueItem)) { |
|
172 | + if (!$type->isValidValue($resolvedValueItem)) { |
|
173 | 173 | $this->resolveValidator->addError(new ResolveException('Not valid value for enum type')); |
174 | 174 | |
175 | 175 | $listValue = null; |
@@ -186,7 +186,7 @@ discard block |
||
186 | 186 | $value = $listValue; |
187 | 187 | } else { |
188 | 188 | if ($field->getType()->getKind() == TypeMap::KIND_ENUM) { |
189 | - if(!$field->getType()->isValidValue($preResolvedValue)) { |
|
189 | + if (!$field->getType()->isValidValue($preResolvedValue)) { |
|
190 | 190 | $this->resolveValidator->addError(new ResolveException('Not valid value for enum type')); |
191 | 191 | $value = null; |
192 | 192 | } else { |
@@ -217,7 +217,7 @@ discard block |
||
217 | 217 | $value[] = []; |
218 | 218 | $index = count($value) - 1; |
219 | 219 | |
220 | - if($field->getConfig()->getType()->getConfig()->getItem()->getKind() == TypeMap::KIND_INTERFACE) { |
|
220 | + if ($field->getConfig()->getType()->getConfig()->getItem()->getKind() == TypeMap::KIND_INTERFACE) { |
|
221 | 221 | $resolvedValueItem = $field->getConfig()->getType()->getConfig()->getItemConfig()->resolveType($resolvedValueItem); |
222 | 222 | $type = $field->getConfig()->getType()->getConfig()->getItem(); |
223 | 223 | } else { |
@@ -287,7 +287,7 @@ discard block |
||
287 | 287 | { |
288 | 288 | $resolvedValue = $field->getConfig()->resolve($contextValue, $this->parseArgumentsValues($field, $query)); |
289 | 289 | |
290 | - if($field->getType()->getKind() == TypeMap::KIND_INTERFACE){ |
|
290 | + if ($field->getType()->getKind() == TypeMap::KIND_INTERFACE) { |
|
291 | 291 | $resolvedValue = $field->getType()->resolveType($resolvedValue); |
292 | 292 | } |
293 | 293 | |
@@ -306,7 +306,7 @@ discard block |
||
306 | 306 | return []; |
307 | 307 | } |
308 | 308 | |
309 | - $args = []; |
|
309 | + $args = []; |
|
310 | 310 | foreach ($query->getArguments() as $argument) { |
311 | 311 | $args[$argument->getName()] = $field->getConfig()->getArgument($argument->getName())->getType()->parseValue($argument->getValue()->getValue()); |
312 | 312 | } |
@@ -339,7 +339,7 @@ discard block |
||
339 | 339 | foreach ($fragment->getFields() as $fragmentField) { |
340 | 340 | $value = $this->collectValue($value, $this->executeQuery($fragmentField, $queryType, $resolvedValue)); |
341 | 341 | } |
342 | - } else if($field->getName() == self::TYPE_NAME_QUERY) { |
|
342 | + } else if ($field->getName() == self::TYPE_NAME_QUERY) { |
|
343 | 343 | $value = $this->collectValue($value, [$field->getAlias() ?: $field->getName() => $queryType->getName()]); |
344 | 344 | } else { |
345 | 345 | $value = $this->collectValue($value, $this->executeQuery($field, $queryType, $resolvedValue)); |
@@ -40,7 +40,7 @@ |
||
40 | 40 | */ |
41 | 41 | public function addFields($fieldsArray) |
42 | 42 | { |
43 | - foreach($fieldsArray as $fieldName => $fieldConfig) { |
|
43 | + foreach ($fieldsArray as $fieldName => $fieldConfig) { |
|
44 | 44 | if (is_object($fieldConfig)) { |
45 | 45 | $this->addField($fieldName, $fieldConfig); |
46 | 46 | } else { |
@@ -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 |
@@ -28,7 +28,7 @@ |
||
28 | 28 | { |
29 | 29 | parent::build($config); |
30 | 30 | |
31 | - $config->getField('friends')->set('resolve', function($droid){ |
|
31 | + $config->getField('friends')->set('resolve', function($droid) { |
|
32 | 32 | return StarWarsData::getFriends($droid); |
33 | 33 | }); |
34 | 34 |
@@ -1,9 +1,9 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Date: 07.12.15 |
|
4 | - * |
|
5 | - * @author Portey Vasil <[email protected]> |
|
6 | - */ |
|
3 | + * Date: 07.12.15 |
|
4 | + * |
|
5 | + * @author Portey Vasil <[email protected]> |
|
6 | + */ |
|
7 | 7 | |
8 | 8 | namespace Youshido\Tests\Type\Config; |
9 | 9 |
@@ -31,7 +31,7 @@ |
||
31 | 31 | ->addField('friends', new ListType([ |
32 | 32 | 'item' => new CharacterInterface() |
33 | 33 | ]), [ |
34 | - 'resolve' => function ($droid) { |
|
34 | + 'resolve' => function($droid) { |
|
35 | 35 | return StarWarsData::getFriends($droid); |
36 | 36 | }, |
37 | 37 | ]) |