@@ -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 | { |
@@ -28,7 +28,7 @@ |
||
28 | 28 | } |
29 | 29 | |
30 | 30 | /** |
31 | - * @return Field[]|Query[] |
|
31 | + * @return Field[] |
|
32 | 32 | */ |
33 | 33 | public function getFields() |
34 | 34 | { |
@@ -45,6 +45,9 @@ |
||
45 | 45 | { |
46 | 46 | } |
47 | 47 | |
48 | + /** |
|
49 | + * @param string $name |
|
50 | + */ |
|
48 | 51 | public function addQuery($name, AbstractObjectType $query) |
49 | 52 | { |
50 | 53 | $this->getQueryType()->getConfig()->addField($name, $query); |
@@ -353,6 +353,9 @@ discard block |
||
353 | 353 | throw new \Exception(sprintf('Property "%s" not found in resolve result', $astField->getName())); |
354 | 354 | } |
355 | 355 | |
356 | + /** |
|
357 | + * @param string $path |
|
358 | + */ |
|
356 | 359 | protected function getPropertyValue($data, $path) |
357 | 360 | { |
358 | 361 | if (is_object($data)) { |
@@ -398,8 +401,8 @@ discard block |
||
398 | 401 | } |
399 | 402 | |
400 | 403 | /** |
401 | - * @param $field Field |
|
402 | - * @param $query Query |
|
404 | + * @param Field $field Field |
|
405 | + * @param Query $query Query |
|
403 | 406 | * |
404 | 407 | * @return array |
405 | 408 | */ |
@@ -418,7 +421,7 @@ discard block |
||
418 | 421 | } |
419 | 422 | |
420 | 423 | /** |
421 | - * @param $query Query |
|
424 | + * @param Query $query Query |
|
422 | 425 | * @param $queryType ObjectType|TypeInterface|Field |
423 | 426 | * @param $resolvedValue mixed |
424 | 427 | * @param $value array |
@@ -81,7 +81,9 @@ discard block |
||
81 | 81 | |
82 | 82 | public function processRequest($payload, $variables = []) |
83 | 83 | { |
84 | - if ($this->hasErrors()) return $this; |
|
84 | + if ($this->hasErrors()) { |
|
85 | + return $this; |
|
86 | + } |
|
85 | 87 | |
86 | 88 | $this->data = []; |
87 | 89 | |
@@ -157,7 +159,9 @@ discard block |
||
157 | 159 | */ |
158 | 160 | protected function executeMutation(Mutation $mutation, $currentLevelSchema) |
159 | 161 | { |
160 | - if (!$currentLevelSchema) throw new ConfigurationException('There is no mutation ' . $mutation->getName()); |
|
162 | + if (!$currentLevelSchema) { |
|
163 | + throw new ConfigurationException('There is no mutation ' . $mutation->getName()); |
|
164 | + } |
|
161 | 165 | |
162 | 166 | if (!$this->resolveValidator->checkFieldExist($currentLevelSchema, $mutation)) { |
163 | 167 | return null; |
@@ -55,6 +55,9 @@ |
||
55 | 55 | return $this; |
56 | 56 | } |
57 | 57 | |
58 | + /** |
|
59 | + * @param AbstractObjectType $value |
|
60 | + */ |
|
58 | 61 | public function isValidValue($value) |
59 | 62 | { |
60 | 63 | if ($value instanceof AbstractObjectType) { |
@@ -59,7 +59,9 @@ |
||
59 | 59 | { |
60 | 60 | if ($value instanceof AbstractObjectType) { |
61 | 61 | foreach($value->getInterfaces() as $interface) { |
62 | - if ($interface instanceof $this) return true; |
|
62 | + if ($interface instanceof $this) { |
|
63 | + return true; |
|
64 | + } |
|
63 | 65 | } |
64 | 66 | } |
65 | 67 | return false; |
@@ -51,7 +51,8 @@ |
||
51 | 51 | /** |
52 | 52 | * @param Field $intField |
53 | 53 | * @param Field $objField |
54 | - * @return bool |
|
54 | + * @param \Youshido\GraphQL\Type\Object\AbstractInterfaceType $interface |
|
55 | + * @return boolean|null |
|
55 | 56 | * @throws ConfigurationException |
56 | 57 | */ |
57 | 58 | protected function assertFieldsIdentical($intField, $objField, $interface) |
@@ -39,7 +39,9 @@ |
||
39 | 39 | |
40 | 40 | protected function assertInterfaceImplementationCorrect(AbstractObjectType $type) |
41 | 41 | { |
42 | - if (!$type->getInterfaces()) return true; |
|
42 | + if (!$type->getInterfaces()) { |
|
43 | + return true; |
|
44 | + } |
|
43 | 45 | |
44 | 46 | foreach ($type->getInterfaces() as $interface) { |
45 | 47 | foreach ($interface->getConfig()->getFields() as $intField) { |
@@ -27,7 +27,9 @@ |
||
27 | 27 | if (!$type) { |
28 | 28 | return; |
29 | 29 | } |
30 | - if (is_object($type) && array_key_exists($type->getName(), $this->types)) return; |
|
30 | + if (is_object($type) && array_key_exists($type->getName(), $this->types)) { |
|
31 | + return; |
|
32 | + } |
|
31 | 33 | |
32 | 34 | switch ($type->getKind()) { |
33 | 35 | case TypeMap::KIND_INTERFACE: |