Youshido\GraphQL\Type\NonNullType

Complexity

Total Complexity

11

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

65.22%

Importance

Changes	4
Bugs	1	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	checkBuild()	0	3	1
A	getName()	0	4	1
A	getKind()	0	4	1
A	resolve()	0	4	1
A	isValidValue()	0	4	1
A	isCompositeType()	0	4	1
A	getNamedType()	0	4	1
A	getNullableType()	0	4	1
A	getTypeOf()	0	4	1

<?php
/*
 * This file is a part of GraphQL project.
 *
 * @author Alexandr Viniychuk <a@viniychuk.com>
 * created: 3:40 PM 4/29/16
 */

namespace Youshido\GraphQL\Type;


class NonNullType extends AbstractType implements CompositeTypeInterface
{
    private $_typeOf;

    /**
     * NonNullType constructor.
     * @param AbstractType|string $fieldType
     */
    public function __construct($fieldType)
    {
        if (TypeMap::isScalarType($fieldType)) {
            $fieldType = TypeMap::getScalarTypeObject($fieldType);

It seems like $fieldType defined by \Youshido\GraphQL\Type\TypeMap::getScalarTypeObject($fieldType) on line 23 can also be of type object<Youshido\GraphQL\Type\AbstractType>; however, Youshido\GraphQL\Type\TypeMap::getScalarTypeObject() does only seem to accept string, maybe add an additional type check?

        }
        $this->_typeOf = $fieldType;
    }

    public function checkBuild()
    {
    }

    public function getName()
    {
        return null;
    }

    public function getKind()
    {
        return TypeMap::KIND_NON_NULL;
    }

    public function resolve($value)
    {
        return $value;
    }

    public function isValidValue($value)
    {
        return $value !== null;
    }

    public function isCompositeType()
    {
        return true;
    }

    public function getNamedType()
    {
        return $this->getTypeOf();
    }

    public function getNullableType()
    {
        return $this->getTypeOf();
    }

    public function getTypeOf()
    {
        return $this->_typeOf;

The expression $this->_typeOf; of type Youshido\GraphQL\Type\AbstractType|string adds the type string to the return on line 69 which is incompatible with the return type declared by the interface Youshido\GraphQL\Type\CompositeTypeInterface::getTypeOf of type Youshido\GraphQL\Type\AbstractType.

    }


}