Arthem\GraphQLMapper\GraphQLManager

Complexity

Total Complexity

6

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	query()	0	11	3
A	getSchema()	0	8	2

<?php

namespace Arthem\GraphQLMapper;

use Arthem\GraphQLMapper\Exception\QueryException;
use Arthem\GraphQLMapper\Schema\SchemaFactory;
use GraphQL\Executor\ExecutionResult;
use GraphQL\GraphQL;
use GraphQL\Schema;

class GraphQLManager
{
    /**
     * @var SchemaFactory
     */
    private $schemaFactory;

    /**
     * @var Schema
     */
    protected $schema;

    /**
     * @param SchemaFactory $schemaFactory
     */
    public function __construct(SchemaFactory $schemaFactory)
    {
        $this->schemaFactory = $schemaFactory;
    }

    /**
     * @param string      $requestString
     * @param mixed       $rootValue
     * @param array|null  $variableValues
     * @param string|null $operationName
     * @return ExecutionResult
     */
    public function query($requestString, $rootValue = null, $variableValues = null, $operationName = null)
    {
        $schema = $this->getSchema();
        $result = GraphQL::execute($schema, $requestString, $rootValue, $variableValues, $operationName);

It seems like $operationName defined by parameter $operationName on line 38 can also be of type string; however, GraphQL\GraphQL::execute() does only seem to accept array|null, maybe add an additional type check?

        if (is_array($result) && isset($result['errors'])) {
            throw new QueryException($result['errors']);
        }

        return $result;
    }

    /**
     * @return Schema
     */
    private function getSchema()
    {
        if (null === $this->schema) {
            $this->schema = $this->schemaFactory->createSchema();
        }

        return $this->schema;
    }
}