Drupal\graphql\GraphQL\Execution\Reducer

Complexity

Total Complexity

29

Size/Duplication

Total Lines	157
Duplicated Lines	6.37 %

Coupling/Cohesion

Components	1
Dependencies	12

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	reduceRequest()	0	8	2
B	reduceOperation()	0	25	6
B	getType()	0	11	7
C	walkQuery()	10	50	13

<?php

namespace Drupal\graphql\GraphQL\Execution;

use Drupal\graphql\GraphQL\Execution\Visitor\VisitorInterface;
use Drupal\graphql\GraphQL\Utility\TypeCollector;
use Youshido\GraphQL\Exception\ResolveException;
use Youshido\GraphQL\Execution\Context\ExecutionContext;
use Youshido\GraphQL\Field\FieldInterface;
use Youshido\GraphQL\Parser\Ast\AbstractAst;
use Youshido\GraphQL\Parser\Ast\Field;
use Youshido\GraphQL\Parser\Ast\Fragment;
use Youshido\GraphQL\Parser\Ast\FragmentReference;
use Youshido\GraphQL\Parser\Ast\Interfaces\FragmentInterface;
use Youshido\GraphQL\Parser\Ast\Mutation;
use Youshido\GraphQL\Parser\Ast\Query;
use Youshido\GraphQL\Parser\Ast\TypedFragmentReference;
use Youshido\GraphQL\Type\AbstractType;
use Youshido\GraphQL\Type\InterfaceType\AbstractInterfaceType;
use Youshido\GraphQL\Type\Object\AbstractObjectType;

class Reducer {

  /**
   * @var \Youshido\GraphQL\Execution\Request
   */
  protected $request;

  /**
   * @var \Youshido\GraphQL\Schema\AbstractSchema
   */
  protected $schema;

  /**
   * @var \Youshido\GraphQL\Type\TypeInterface[]
   */
  protected $types;

  /**
   * @var \Youshido\GraphQL\Execution\Context\ExecutionContext
   */
  protected $context;

  /**
   * QueryReducer constructor.
   *
   */
  public function __construct(ExecutionContext $executionContext) {
    $this->context = $executionContext;
    $this->schema = $executionContext->getSchema();
    $this->request = $executionContext->getRequest();
    $this->types = TypeCollector::collectTypes($this->schema);
  }

  /**
   * @param \Drupal\graphql\GraphQL\Execution\Visitor\VisitorInterface $visitor
   *
   * @return mixed|\Youshido\GraphQL\Parser\Ast\Query
   */
  public function reduceRequest(VisitorInterface $visitor) {
    $operations = $this->request->getAllOperations();

    return array_reduce($operations, function ($carry, $current) use ($visitor) {
      $type = $current instanceof Mutation ? $this->schema->getMutationType() : $this->schema->getQueryType();
      return $visitor->reduce($carry, $this->reduceOperation($current, $type, $visitor), $this->context);

The call to VisitorInterface::reduce() has too many arguments starting with $this->context.

    }, $visitor->initial($this->context));

The call to VisitorInterface::initial() has too many arguments starting with $this->context.

  }

  /**
   * @param \Youshido\GraphQL\Parser\Ast\AbstractAst $query
   * @param \Youshido\GraphQL\Type\AbstractType $type
   * @param \Drupal\graphql\GraphQL\Execution\Visitor\VisitorInterface $visitor
   *
   * @return mixed|void
   */
  protected function reduceOperation(AbstractAst $query, AbstractType $type, VisitorInterface $visitor) {
    $carry = $visitor->initial();
    if (!($type instanceof AbstractObjectType) || !$type->hasField($query->getName())) {

It seems like you code against a specific sub-type and not the parent class Youshido\GraphQL\Parser\Ast\AbstractAst as the method getName() does only exist in the following sub-classes of Youshido\GraphQL\Parser\Ast\AbstractAst: Youshido\GraphQL\Parser\Ast\Argument, Youshido\GraphQL\Parser\Ast\ArgumentValue\Variable, Youshido\GraphQL\Parser\Ast\ArgumentValue\VariableReference, Youshido\GraphQL\Parser\Ast\Directive, Youshido\GraphQL\Parser\Ast\Field, Youshido\GraphQL\Parser\Ast\Fragment, Youshido\GraphQL\Parser\Ast\FragmentReference, Youshido\GraphQL\Parser\Ast\Mutation, Youshido\GraphQL\Parser\Ast\Query. Maybe you want to instanceof check for one of these explicitly?

      return $carry;
    }

    if (($name = $query->getName()) && $type->hasField($name)) {

It seems like you code against a specific sub-type and not the parent class Youshido\GraphQL\Parser\Ast\AbstractAst as the method getName() does only exist in the following sub-classes of Youshido\GraphQL\Parser\Ast\AbstractAst: Youshido\GraphQL\Parser\Ast\Argument, Youshido\GraphQL\Parser\Ast\ArgumentValue\Variable, Youshido\GraphQL\Parser\Ast\ArgumentValue\VariableReference, Youshido\GraphQL\Parser\Ast\Directive, Youshido\GraphQL\Parser\Ast\Field, Youshido\GraphQL\Parser\Ast\Fragment, Youshido\GraphQL\Parser\Ast\FragmentReference, Youshido\GraphQL\Parser\Ast\Mutation, Youshido\GraphQL\Parser\Ast\Query. Maybe you want to instanceof check for one of these explicitly?

      $operation = $type->getField($query->getName());

It seems like you code against a specific sub-type and not the parent class Youshido\GraphQL\Parser\Ast\AbstractAst as the method getName() does only exist in the following sub-classes of Youshido\GraphQL\Parser\Ast\AbstractAst: Youshido\GraphQL\Parser\Ast\Argument, Youshido\GraphQL\Parser\Ast\ArgumentValue\Variable, Youshido\GraphQL\Parser\Ast\ArgumentValue\VariableReference, Youshido\GraphQL\Parser\Ast\Directive, Youshido\GraphQL\Parser\Ast\Field, Youshido\GraphQL\Parser\Ast\Fragment, Youshido\GraphQL\Parser\Ast\FragmentReference, Youshido\GraphQL\Parser\Ast\Mutation, Youshido\GraphQL\Parser\Ast\Query. Maybe you want to instanceof check for one of these explicitly?

      $walker = $this->walkQuery($query, $operation, $visitor);

      while ($walker->valid()) {
        /** @var \Youshido\GraphQL\Parser\Ast\Field $field */
        /** @var \Youshido\GraphQL\Field\Field $ast */
        list($field, $ast, $child) = $walker->current();

        $args = $field->getKeyValueArguments();
        $result = $visitor->visit($args, $ast, $child, $this->context);

The call to VisitorInterface::visit() has too many arguments starting with $this->context.

        $carry = $visitor->reduce($carry, $result, $this->context);

The call to VisitorInterface::reduce() has too many arguments starting with $this->context.

        $walker->send($result);
      }
    }

    return $carry;
  }

  /**
   * @param $node
   * @param \Youshido\GraphQL\Field\FieldInterface $current
   * @param \Drupal\graphql\GraphQL\Execution\Visitor\VisitorInterface $visitor
   *
   * @return \Generator
   */
  protected function walkQuery(AbstractAst $node, FieldInterface $current, VisitorInterface $visitor) {
    $carry = $visitor->initial();

    if (!($node instanceof Field)) {
      /** @var \Youshido\GraphQL\Parser\Ast\Field $field */
      foreach ($node->getFields() as $field) {

It seems like you code against a specific sub-type and not the parent class Youshido\GraphQL\Parser\Ast\AbstractAst as the method getFields() does only exist in the following sub-classes of Youshido\GraphQL\Parser\Ast\AbstractAst: Youshido\GraphQL\Parser\Ast\Field, Youshido\GraphQL\Parser\Ast\Fragment, Youshido\GraphQL\Parser\Ast\Mutation, Youshido\GraphQL\Parser\Ast\Query, Youshido\GraphQL\Parser\Ast\TypedFragmentReference. Maybe you want to instanceof check for one of these explicitly?

        if ($field instanceof FragmentInterface) {
          if ($field instanceof FragmentReference) {
            $field = $this->request->getFragment($field->getName());
          }

          $walker = $this->walkQuery($field, $current, $visitor);

It seems like $field defined by $this->request->getFragment($field->getName()) on line 117 can also be of type null or object<Youshido\GraphQL\Parser\Ast\Interfaces\FragmentInterface>; however, Drupal\graphql\GraphQL\Execution\Reducer::walkQuery() does only seem to accept object<Youshido\GraphQL\Parser\Ast\AbstractAst>, maybe add an additional type check?

          $next = $walker->current();

          while ($walker->valid()) {

This code seems to be duplicated across your project.

            $received = (yield $next);
            $carry = $visitor->reduce($carry, $received, $this->context);

The call to VisitorInterface::reduce() has too many arguments starting with $this->context.

            $next = $walker->send($received);
          }
        }
        else {
          $type = $this->getType($node, $current);
          $name = $field->getName();

          if ($name !== Processor::TYPE_NAME_QUERY && ($type instanceof AbstractObjectType || $type instanceof AbstractInterfaceType)) {
            if (!$type->hasField($name)) {
              $type = $type->getNamedType()->getName();
              $location = $field->getLocation();

              throw new ResolveException(sprintf('Unknown field %s for type %s.', $name, $type), $location);
            }

            $ast = $type->getField($name);
            $walker = $this->walkQuery($field, $ast, $visitor);
            $next = $walker->current();

            while ($walker->valid()) {

This code seems to be duplicated across your project.

              $received = (yield $next);
              $carry = $visitor->reduce($carry, $received, $this->context);

The call to VisitorInterface::reduce() has too many arguments starting with $this->context.

              $next = $walker->send($received);
            }
          }
        }
      }
    }

    if ($node instanceof Query || $node instanceof Field) {
      yield [$node, $current, $carry];
    }
  }

  /**
   * @param \Youshido\GraphQL\Parser\Ast\AbstractAst $node
   * @param \Youshido\GraphQL\Field\FieldInterface $current
   *
   * @return null|\Youshido\GraphQL\Type\AbstractType|\Youshido\GraphQL\Type\TypeInterface
   */
  protected function getType(AbstractAst $node, FieldInterface $current) {
    if ($node instanceof Fragment && $name = $node->getModel()) {
      return isset($this->types[$name]) ? $this->types[$name] : NULL;
    }

    if ($node instanceof TypedFragmentReference && $name = $node->getTypeName()) {
      return isset($this->types[$name]) ? $this->types[$name] : NULL;
    }

    return $current->getType()->getNamedType();
  }

}