Digia\GraphQL\Validation\Rule\UniqueDirectivesPerLocationRule::enterNode()

Complexity

Conditions	4
Paths	2

Size

Total Lines	23
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function enterNode(NodeInterface $node): VisitorResult
    {
        if ($node instanceof DirectivesAwareInterface) {
            $directives      = $node->getDirectives();
            $knownDirectives = [];

            foreach ($directives as $directive) {
                $directiveName = $directive->getNameValue();

                if (isset($knownDirectives[$directiveName])) {
                    $this->context->reportError(
                        new ValidationException(
                            duplicateDirectiveMessage($directiveName),

It seems like $directiveName can also be of type null; however, parameter $directiveName of Digia\GraphQL\Validation\duplicateDirectiveMessage() does only seem to accept string, maybe add an additional type check?

                            [$knownDirectives[$directiveName], $directive]
                        )
                    );
                } else {
                    $knownDirectives[$directiveName] = $directive;
                }
            }
        }

        return new VisitorResult($node);
    }