TheCodingMachine\GraphQL\Controllers\TypeRegistry

Complexity

Total Complexity

7

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getType()	0	6	2
A	getObjectType()	0	7	2
A	hasType()	0	3	1
A	registerType()	0	6	2

<?php


namespace TheCodingMachine\GraphQL\Controllers;

use function get_class;
use GraphQL\Type\Definition\NamedType;
use GraphQL\Type\Definition\ObjectType;
use GraphQL\Type\Definition\InterfaceType;
use GraphQL\Type\Definition\Type;

/**
 * A cache used to store already FULLY COMPUTED types.
 */
class TypeRegistry
{
    /**
     * @var array<string,NamedType&Type&(ObjectType|InterfaceType)>
     */
    private $outputTypes = [];

    /**
     * Registers a type.
     * IMPORTANT: the type MUST be fully computed (so ExtendType annotations must have ALREADY been applied to the tag)
     * ONLY THE RecursiveTypeMapper IS ALLOWED TO CALL THIS METHOD.
     *
     * @param NamedType&Type&(ObjectType|InterfaceType) $type
     */
    public function registerType(NamedType $type): void
    {
        if (isset($this->outputTypes[$type->name])) {

Accessing name on the interface GraphQL\Type\Definition\NamedType suggest that you code against a concrete implementation. How about adding an instanceof check?

            throw new GraphQLException('Type "'.$type->name.'" is already registered');
        }
        $this->outputTypes[$type->name] = $type;
    }

    public function hasType(string $typeName): bool
    {
        return isset($this->outputTypes[$typeName]);
    }

    /**
     * @param string $typeName
     * @return NamedType&Type&(ObjectType|InterfaceType)
     */
    public function getType(string $typeName): NamedType
    {
        if (!isset($this->outputTypes[$typeName])) {
            throw new GraphQLException('Could not find type "'.$typeName.'" in registry');
        }
        return $this->outputTypes[$typeName];
    }

    public function getObjectType(string $typeName): ObjectType
    {
        $type = $this->getType($typeName);
        if (!$type instanceof ObjectType) {
            throw new GraphQLException('Expected GraphQL type "'.$typeName.'" to be an ObjectType. Got a '.get_class($type));
        }
        return $type;
    }
}