digiaonline /
graphql-php
| Total Complexity | 42 |
| Total Lines | 394 |
| Duplicated Lines | 0 % |
| Changes | 0 | ||
Complex classes like DefinitionBuilder often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use DefinitionBuilder, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 56 | class DefinitionBuilder implements DefinitionBuilderInterface |
||
| 57 | { |
||
| 58 | use CacheAwareTrait; |
||
| 59 | |||
| 60 | private const CACHE_PREFIX = 'GraphQL_DefinitionBuilder_'; |
||
| 61 | |||
| 62 | /** |
||
| 63 | * @var array |
||
| 64 | */ |
||
| 65 | protected $typeDefinitionsMap; |
||
| 66 | |||
| 67 | /** |
||
| 68 | * @var ResolverRegistryInterface |
||
| 69 | */ |
||
| 70 | protected $resolverRegistry; |
||
| 71 | |||
| 72 | /** |
||
| 73 | * @var callable |
||
| 74 | */ |
||
| 75 | protected $resolveTypeFunction; |
||
| 76 | |||
| 77 | /** |
||
| 78 | * DefinitionBuilder constructor. |
||
| 79 | * @param array $typeDefinitionsMap |
||
| 80 | * @param ResolverRegistryInterface $resolverRegistry |
||
| 81 | * @param callable|null $resolveTypeFunction |
||
| 82 | * @param CacheInterface $cache |
||
| 83 | * @throws InvalidArgumentException |
||
| 84 | */ |
||
| 85 | public function __construct( |
||
| 86 | array $typeDefinitionsMap, |
||
| 87 | ResolverRegistryInterface $resolverRegistry, |
||
| 88 | ?callable $resolveTypeFunction = null, |
||
| 89 | CacheInterface $cache |
||
| 90 | ) { |
||
| 91 | $this->typeDefinitionsMap = $typeDefinitionsMap; |
||
| 92 | $this->resolverRegistry = $resolverRegistry; |
||
| 93 | $this->cache = $cache; |
||
| 94 | $this->resolveTypeFunction = $resolveTypeFunction ?? [$this, 'defaultTypeResolver']; |
||
|
|
|||
| 95 | |||
| 96 | $builtInTypes = keyMap( |
||
| 97 | \array_merge(specifiedScalarTypes(), introspectionTypes()), |
||
| 98 | function (NamedTypeInterface $type) { |
||
| 99 | return $type->getName(); |
||
| 100 | } |
||
| 101 | ); |
||
| 102 | |||
| 103 | foreach ($builtInTypes as $name => $type) { |
||
| 104 | $this->setInCache($name, $type); |
||
| 105 | } |
||
| 106 | } |
||
| 107 | |||
| 108 | /** |
||
| 109 | * @inheritdoc |
||
| 110 | */ |
||
| 111 | public function buildTypes(array $nodes): array |
||
| 116 | } |
||
| 117 | |||
| 118 | /** |
||
| 119 | * @param NamedTypeNode|TypeDefinitionNodeInterface $node |
||
| 120 | * @inheritdoc |
||
| 121 | */ |
||
| 122 | public function buildType(NodeInterface $node): TypeInterface |
||
| 123 | { |
||
| 124 | $typeName = $node->getNameValue(); |
||
| 125 | |||
| 126 | if (!$this->isInCache($typeName)) { |
||
| 127 | if ($node instanceof NamedTypeNode) { |
||
| 128 | $definition = $this->getTypeDefinition($typeName); |
||
| 129 | |||
| 130 | $type = null !== $definition ? $this->buildNamedType($definition) : $this->resolveType($node); |
||
| 131 | |||
| 132 | $this->setInCache($typeName, $type); |
||
| 133 | } else { |
||
| 134 | $this->setInCache($typeName, $this->buildNamedType($node)); |
||
| 135 | } |
||
| 136 | } |
||
| 137 | |||
| 138 | return $this->getFromCache($typeName); |
||
| 139 | } |
||
| 140 | |||
| 141 | /** |
||
| 142 | * @inheritdoc |
||
| 143 | */ |
||
| 144 | public function buildDirective(DirectiveDefinitionNode $node): DirectiveInterface |
||
| 145 | { |
||
| 146 | return GraphQLDirective([ |
||
| 147 | 'name' => $node->getNameValue(), |
||
| 148 | 'description' => $node->getDescriptionValue(), |
||
| 149 | 'locations' => \array_map(function (NameNode $node) { |
||
| 150 | return $node->getValue(); |
||
| 151 | }, $node->getLocations()), |
||
| 152 | 'args' => $node->hasArguments() ? $this->buildArguments($node->getArguments()) : [], |
||
| 153 | 'astNode' => $node, |
||
| 154 | ]); |
||
| 155 | } |
||
| 156 | |||
| 157 | /** |
||
| 158 | * @inheritdoc |
||
| 159 | */ |
||
| 160 | public function buildField($node, ?callable $resolve = null): array |
||
| 161 | { |
||
| 162 | return [ |
||
| 163 | 'type' => $this->buildWrappedType($node->getType()), |
||
| 164 | 'description' => $node->getDescriptionValue(), |
||
| 165 | 'args' => $node->hasArguments() ? $this->buildArguments($node->getArguments()) : [], |
||
| 166 | 'deprecationReason' => $this->getDeprecationReason($node), |
||
| 167 | 'resolve' => $resolve, |
||
| 168 | 'astNode' => $node, |
||
| 169 | ]; |
||
| 170 | } |
||
| 171 | |||
| 172 | /** |
||
| 173 | * @param TypeNodeInterface $typeNode |
||
| 174 | * @return TypeInterface |
||
| 175 | * @throws InvariantException |
||
| 176 | * @throws InvalidTypeException |
||
| 177 | */ |
||
| 178 | protected function buildWrappedType(TypeNodeInterface $typeNode): TypeInterface |
||
| 179 | { |
||
| 180 | $typeDefinition = $this->buildType($this->getNamedTypeNode($typeNode)); |
||
| 181 | return buildWrappedType($typeDefinition, $typeNode); |
||
| 182 | } |
||
| 183 | |||
| 184 | /** |
||
| 185 | * @param array $nodes |
||
| 186 | * @return array |
||
| 187 | * @throws CoercingException |
||
| 188 | */ |
||
| 189 | protected function buildArguments(array $nodes): array |
||
| 190 | { |
||
| 191 | return keyValueMap( |
||
| 192 | $nodes, |
||
| 193 | function (InputValueDefinitionNode $value) { |
||
| 194 | return $value->getNameValue(); |
||
| 195 | }, |
||
| 196 | function (InputValueDefinitionNode $value): array { |
||
| 197 | $type = $this->buildWrappedType($value->getType()); |
||
| 198 | $defaultValue = $value->getDefaultValue(); |
||
| 199 | return [ |
||
| 200 | 'type' => $type, |
||
| 201 | 'description' => $value->getDescriptionValue(), |
||
| 202 | 'defaultValue' => null !== $defaultValue |
||
| 203 | ? valueFromAST($defaultValue, $type) |
||
| 204 | : null, |
||
| 205 | 'astNode' => $value, |
||
| 206 | ]; |
||
| 207 | }); |
||
| 208 | } |
||
| 209 | |||
| 210 | /** |
||
| 211 | * @param TypeDefinitionNodeInterface $node |
||
| 212 | * @return NamedTypeInterface |
||
| 213 | * @throws LanguageException |
||
| 214 | */ |
||
| 215 | protected function buildNamedType(TypeDefinitionNodeInterface $node): NamedTypeInterface |
||
| 237 | } |
||
| 238 | |||
| 239 | /** |
||
| 240 | * @param ObjectTypeDefinitionNode $node |
||
| 241 | * @return ObjectType |
||
| 242 | */ |
||
| 243 | protected function buildObjectType(ObjectTypeDefinitionNode $node): ObjectType |
||
| 244 | { |
||
| 245 | return GraphQLObjectType([ |
||
| 246 | 'name' => $node->getNameValue(), |
||
| 247 | 'description' => $node->getDescriptionValue(), |
||
| 248 | 'fields' => function () use ($node) { |
||
| 249 | return $this->buildFields($node); |
||
| 250 | }, |
||
| 251 | 'interfaces' => function () use ($node) { |
||
| 252 | return $node->hasInterfaces() ? \array_map(function (NodeInterface $interface) { |
||
| 253 | return $this->buildType($interface); |
||
| 254 | }, $node->getInterfaces()) : []; |
||
| 255 | }, |
||
| 256 | 'astNode' => $node, |
||
| 257 | ]); |
||
| 258 | } |
||
| 259 | |||
| 260 | /** |
||
| 261 | * @param ObjectTypeDefinitionNode|InterfaceTypeDefinitionNode|InputObjectTypeDefinitionNode $node |
||
| 262 | * @return array |
||
| 263 | */ |
||
| 264 | protected function buildFields($node): array |
||
| 265 | { |
||
| 266 | return $node->hasFields() ? keyValueMap( |
||
| 267 | $node->getFields(), |
||
| 268 | function ($value) { |
||
| 269 | /** @var FieldDefinitionNode|InputValueDefinitionNode $value */ |
||
| 270 | return $value->getNameValue(); |
||
| 271 | }, |
||
| 272 | function ($value) use ($node) { |
||
| 273 | /** @var FieldDefinitionNode|InputValueDefinitionNode $value */ |
||
| 274 | return $this->buildField( |
||
| 275 | $value, |
||
| 276 | $this->resolverRegistry->lookup($node->getNameValue(), $value->getNameValue()) |
||
| 277 | ); |
||
| 278 | } |
||
| 279 | ) : []; |
||
| 280 | } |
||
| 281 | |||
| 282 | /** |
||
| 283 | * @param InterfaceTypeDefinitionNode $node |
||
| 284 | * @return InterfaceType |
||
| 285 | */ |
||
| 286 | protected function buildInterfaceType(InterfaceTypeDefinitionNode $node): InterfaceType |
||
| 287 | { |
||
| 288 | return GraphQLInterfaceType([ |
||
| 289 | 'name' => $node->getNameValue(), |
||
| 290 | 'description' => $node->getDescriptionValue(), |
||
| 291 | 'fields' => function () use ($node): array { |
||
| 292 | return $this->buildFields($node); |
||
| 293 | }, |
||
| 294 | 'astNode' => $node, |
||
| 295 | ]); |
||
| 296 | } |
||
| 297 | |||
| 298 | /** |
||
| 299 | * @param EnumTypeDefinitionNode $node |
||
| 300 | * @return EnumType |
||
| 301 | */ |
||
| 302 | protected function buildEnumType(EnumTypeDefinitionNode $node): EnumType |
||
| 303 | { |
||
| 304 | return GraphQLEnumType([ |
||
| 305 | 'name' => $node->getNameValue(), |
||
| 306 | 'description' => $node->getDescriptionValue(), |
||
| 307 | 'values' => $node->hasValues() ? keyValueMap( |
||
| 308 | $node->getValues(), |
||
| 309 | function (EnumValueDefinitionNode $value): string { |
||
| 310 | return $value->getNameValue(); |
||
| 311 | }, |
||
| 312 | function (EnumValueDefinitionNode $value): array { |
||
| 313 | return [ |
||
| 314 | 'description' => $value->getDescriptionValue(), |
||
| 315 | 'deprecationReason' => $this->getDeprecationReason($value), |
||
| 316 | 'astNode' => $value, |
||
| 317 | ]; |
||
| 318 | } |
||
| 319 | ) : [], |
||
| 320 | 'astNode' => $node, |
||
| 321 | ]); |
||
| 322 | } |
||
| 323 | |||
| 324 | /** |
||
| 325 | * @param UnionTypeDefinitionNode $node |
||
| 326 | * @return UnionType |
||
| 327 | */ |
||
| 328 | protected function buildUnionType(UnionTypeDefinitionNode $node): UnionType |
||
| 329 | { |
||
| 330 | return GraphQLUnionType([ |
||
| 331 | 'name' => $node->getNameValue(), |
||
| 332 | 'description' => $node->getDescriptionValue(), |
||
| 333 | 'types' => $node->hasTypes() ? \array_map(function (TypeNodeInterface $type) { |
||
| 334 | return $this->buildType($type); |
||
| 335 | }, $node->getTypes()) : [], |
||
| 336 | 'astNode' => $node, |
||
| 337 | ]); |
||
| 338 | } |
||
| 339 | |||
| 340 | /** |
||
| 341 | * @param ScalarTypeDefinitionNode $node |
||
| 342 | * @return ScalarType |
||
| 343 | */ |
||
| 344 | protected function buildScalarType(ScalarTypeDefinitionNode $node): ScalarType |
||
| 345 | { |
||
| 346 | return GraphQLScalarType([ |
||
| 347 | 'name' => $node->getNameValue(), |
||
| 348 | 'description' => $node->getDescriptionValue(), |
||
| 349 | 'serialize' => function ($value) { |
||
| 350 | return $value; |
||
| 351 | }, |
||
| 352 | 'astNode' => $node, |
||
| 353 | ]); |
||
| 354 | } |
||
| 355 | |||
| 356 | /** |
||
| 357 | * @param InputObjectTypeDefinitionNode $node |
||
| 358 | * @return InputObjectType |
||
| 359 | */ |
||
| 360 | protected function buildInputObjectType(InputObjectTypeDefinitionNode $node): InputObjectType |
||
| 361 | { |
||
| 362 | return GraphQLInputObjectType([ |
||
| 363 | 'name' => $node->getNameValue(), |
||
| 364 | 'description' => $node->getDescriptionValue(), |
||
| 365 | 'fields' => $node->hasFields() ? function () use ($node) { |
||
| 366 | return keyValueMap( |
||
| 367 | $node->getFields(), |
||
| 368 | function (InputValueDefinitionNode $value): string { |
||
| 369 | return $value->getNameValue(); |
||
| 370 | }, |
||
| 371 | function (InputValueDefinitionNode $value): array { |
||
| 372 | $type = $this->buildWrappedType($value->getType()); |
||
| 373 | $defaultValue = $value->getDefaultValue(); |
||
| 374 | return [ |
||
| 375 | 'type' => $type, |
||
| 376 | 'description' => $value->getDescriptionValue(), |
||
| 377 | 'defaultValue' => null !== $defaultValue |
||
| 378 | ? valueFromAST($defaultValue, $type) |
||
| 379 | : null, |
||
| 380 | 'astNode' => $value, |
||
| 381 | ]; |
||
| 382 | } |
||
| 383 | ); |
||
| 384 | } : [], |
||
| 385 | 'astNode' => $node, |
||
| 386 | ]); |
||
| 387 | } |
||
| 388 | |||
| 389 | /** |
||
| 390 | * @inheritdoc |
||
| 391 | */ |
||
| 392 | protected function resolveType(NamedTypeNode $node): ?NamedTypeInterface |
||
| 393 | { |
||
| 394 | return \call_user_func($this->resolveTypeFunction, $node); |
||
| 395 | } |
||
| 396 | |||
| 397 | /** |
||
| 398 | * @param NamedTypeNode $node |
||
| 399 | * @return NamedTypeInterface|null |
||
| 400 | * @throws \Psr\SimpleCache\InvalidArgumentException |
||
| 401 | */ |
||
| 402 | public function defaultTypeResolver(NamedTypeNode $node): ?NamedTypeInterface |
||
| 405 | } |
||
| 406 | |||
| 407 | /** |
||
| 408 | * @param string $typeName |
||
| 409 | * @return TypeDefinitionNodeInterface|null |
||
| 410 | */ |
||
| 411 | protected function getTypeDefinition(string $typeName): ?TypeDefinitionNodeInterface |
||
| 412 | { |
||
| 413 | return $this->typeDefinitionsMap[$typeName] ?? null; |
||
| 414 | } |
||
| 415 | |||
| 416 | /** |
||
| 417 | * @param TypeNodeInterface $typeNode |
||
| 418 | * @return NamedTypeNode |
||
| 419 | */ |
||
| 420 | protected function getNamedTypeNode(TypeNodeInterface $typeNode): NamedTypeNode |
||
| 421 | { |
||
| 422 | $namedType = $typeNode; |
||
| 423 | |||
| 424 | while ($namedType instanceof ListTypeNode || $namedType instanceof NonNullTypeNode) { |
||
| 425 | $namedType = $namedType->getType(); |
||
| 426 | } |
||
| 427 | |||
| 428 | return $namedType; |
||
| 429 | } |
||
| 430 | |||
| 431 | /** |
||
| 432 | * @param NodeInterface|EnumValueDefinitionNode|FieldDefinitionNode $node |
||
| 433 | * @return null|string |
||
| 434 | * @throws InvariantException |
||
| 435 | * @throws ExecutionException |
||
| 436 | * @throws InvalidTypeException |
||
| 437 | */ |
||
| 438 | protected function getDeprecationReason(NodeInterface $node): ?string |
||
| 439 | { |
||
| 440 | $deprecated = coerceDirectiveValues(GraphQLDeprecatedDirective(), $node); |
||
| 441 | return $deprecated['reason'] ?? null; |
||
| 442 | } |
||
| 443 | |||
| 444 | /** |
||
| 445 | * @return string |
||
| 446 | */ |
||
| 447 | protected function getCachePrefix(): string |
||
| 450 | } |
||
| 451 | } |
||
| 452 | |||
| 453 | /** |
||
| 454 | * @param TypeInterface $innerType |
||
| 455 | * @param NamedTypeInterface|TypeNodeInterface $inputTypeNode |
||
| 456 | * @return TypeInterface |
||
| 457 | * @throws InvariantException |
||
| 458 | * @throws InvalidTypeException |
||
| 459 | */ |
||
| 460 | function buildWrappedType(TypeInterface $innerType, TypeNodeInterface $inputTypeNode): TypeInterface |
||
| 461 | { |
||
| 462 | if ($inputTypeNode instanceof ListTypeNode) { |
||
| 463 | return GraphQLList(buildWrappedType($innerType, $inputTypeNode->getType())); |
||
| 464 | } |
||
| 465 | |||
| 466 | if ($inputTypeNode instanceof NonNullTypeNode) { |
||
| 467 | $wrappedType = buildWrappedType($innerType, $inputTypeNode->getType()); |
||
| 473 |
Loading history...
Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly.
For example, imagine you have a variable
$accountIdthat can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to theidproperty of an instance of theAccountclass. This class holds a proper account, so the id value must no longer be false.Either this assignment is in error or a type check should be added for that assignment.