Digia\GraphQL\Validation\Rule\UniqueFragmentNamesRule::enterFragmentDefinition()

Complexity

Conditions	2
Paths	2

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function enterFragmentDefinition(FragmentDefinitionNode $node): VisitorResult
    {
        $fragmentName = $node->getNameValue();

        if (isset($this->knownFragmentNames[$fragmentName])) {
            $this->context->reportError(
                new ValidationException(
                    duplicateFragmentMessage($fragmentName),

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

                    [$this->knownFragmentNames[$fragmentName], $node->getName()]
                )
            );
        } else {
            $this->knownFragmentNames[$fragmentName] = $node->getName();
        }

        return new VisitorResult(null);
    }