GraphQL\Doctrine\Utils

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getOperatorTypeName()	0	3	1
A	getTypeName()	0	5	1

<?php

declare(strict_types=1);

namespace GraphQL\Doctrine;

use GraphQL\Type\Definition\EnumType;
use GraphQL\Type\Definition\LeafType;
use GraphQL\Type\Definition\ScalarType;

/**
 * A few utils
 */
abstract class Utils
{
    /**
     * Get the GraphQL type name for an output type from the PHP class
     *
     * @param string $className
     *
     * @return string
     */
    public static function getTypeName(string $className): string
    {
        $parts = explode('\\', $className);

        return end($parts);
    }

    /**
     * Get the GraphQL type name for a Filter type from the PHP class
     *
     * @param string $className
     * @param EnumType|ScalarType $type
     *
     * @return string
     */
    public static function getOperatorTypeName(string $className, LeafType $type): string
    {
        return preg_replace('~Type$~', '', self::getTypeName($className)) . ucfirst($type->name);

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

    }
}