@@ -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 | |
@@ -1,9 +1,9 @@ |
||
| 1 | 1 | <?php |
| 2 | 2 | /** |
| 3 | - * Date: 20.11.15 |
|
| 4 | - * |
|
| 5 | - * @author Portey Vasil <[email protected]> |
|
| 6 | - */ |
|
| 3 | + * Date: 20.11.15 |
|
| 4 | + * |
|
| 5 | + * @author Portey Vasil <[email protected]> |
|
| 6 | + */ |
|
| 7 | 7 | |
| 8 | 8 | namespace Youshido\GraphQL\Parser\Ast; |
| 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 | * |
@@ -154,7 +154,7 @@ discard block |
||
| 154 | 154 | $preResolvedValue = $this->getPreResolvedValue($contextValue, $query); |
| 155 | 155 | |
| 156 | 156 | if ($field->getConfig()->getType()->getKind() == TypeMap::KIND_LIST) { |
| 157 | - if(!is_array($preResolvedValue)){ |
|
| 157 | + if (!is_array($preResolvedValue)) { |
|
| 158 | 158 | $value = null; |
| 159 | 159 | $this->resolveValidator->addError(new ResolveException('Not valid resolve value for list type')); |
| 160 | 160 | } |
@@ -167,7 +167,7 @@ discard block |
||
| 167 | 167 | |
| 168 | 168 | if ($type->getKind() == TypeMap::KIND_ENUM) { |
| 169 | 169 | /** @var $type AbstractEnumType */ |
| 170 | - if(!$type->isValidValue($resolvedValueItem)) { |
|
| 170 | + if (!$type->isValidValue($resolvedValueItem)) { |
|
| 171 | 171 | $this->resolveValidator->addError(new ResolveException('Not valid value for enum type')); |
| 172 | 172 | |
| 173 | 173 | $listValue = null; |
@@ -184,7 +184,7 @@ discard block |
||
| 184 | 184 | $value = $listValue; |
| 185 | 185 | } else { |
| 186 | 186 | if ($field->getType()->getKind() == TypeMap::KIND_ENUM) { |
| 187 | - if(!$field->getType()->isValidValue($preResolvedValue)) { |
|
| 187 | + if (!$field->getType()->isValidValue($preResolvedValue)) { |
|
| 188 | 188 | $this->resolveValidator->addError(new ResolveException('Not valid value for enum type')); |
| 189 | 189 | $value = null; |
| 190 | 190 | } else { |
@@ -215,7 +215,7 @@ discard block |
||
| 215 | 215 | $value[] = []; |
| 216 | 216 | $index = count($value) - 1; |
| 217 | 217 | |
| 218 | - if(in_array($field->getConfig()->getType()->getConfig()->getItem()->getKind(), [TypeMap::KIND_UNION, TypeMap::KIND_INTERFACE]) ) { |
|
| 218 | + if (in_array($field->getConfig()->getType()->getConfig()->getItem()->getKind(), [TypeMap::KIND_UNION, TypeMap::KIND_INTERFACE])) { |
|
| 219 | 219 | $type = $field->getConfig()->getType()->getConfig()->getItemConfig()->resolveType($resolvedValueItem); |
| 220 | 220 | } else { |
| 221 | 221 | $type = $field->getType(); |
@@ -284,7 +284,7 @@ discard block |
||
| 284 | 284 | { |
| 285 | 285 | $resolvedValue = $field->getConfig()->resolve($contextValue, $this->parseArgumentsValues($field, $query)); |
| 286 | 286 | |
| 287 | - if(in_array($field->getType()->getKind(), [TypeMap::KIND_UNION, TypeMap::KIND_INTERFACE])){ |
|
| 287 | + if (in_array($field->getType()->getKind(), [TypeMap::KIND_UNION, TypeMap::KIND_INTERFACE])) { |
|
| 288 | 288 | $resolvedType = $field->getType()->resolveType($resolvedValue); |
| 289 | 289 | $field->setType($resolvedType); |
| 290 | 290 | } |
@@ -304,7 +304,7 @@ discard block |
||
| 304 | 304 | return []; |
| 305 | 305 | } |
| 306 | 306 | |
| 307 | - $args = []; |
|
| 307 | + $args = []; |
|
| 308 | 308 | foreach ($query->getArguments() as $argument) { |
| 309 | 309 | $args[$argument->getName()] = $field->getConfig()->getArgument($argument->getName())->getType()->parseValue($argument->getValue()->getValue()); |
| 310 | 310 | } |
@@ -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 { |
@@ -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 | |
@@ -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 | |
@@ -38,7 +38,7 @@ |
||
| 38 | 38 | { |
| 39 | 39 | $callable = $this->get('resolveType'); |
| 40 | 40 | |
| 41 | - if($callable && is_callable($callable)) { |
|
| 41 | + if ($callable && is_callable($callable)) { |
|
| 42 | 42 | return call_user_func_array($callable, [$object]); |
| 43 | 43 | } |
| 44 | 44 | |