DependentDefinitionBuilder::buildType() - Code Metrics - Inspection of "Apply fixes from StyleCI" - railt/webonyx-adapter - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3b79f3...426227 )

by Kirill

created 2017-12-26 12:05 UTC

DependentDefinitionBuilder::buildType() A

↳ Parent: DependentDefinitionBuilder

Complexity

Conditions	4
Paths	6

Size

Total Lines	16
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	8
nc	6
nop	0
dl	0
loc	16
rs	9.2
c	0
b	0
f	0

<?php
/**
 * This file is part of Railt package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
declare(strict_types=1);

namespace Railt\Adapters\Webonyx\Builders;

use GraphQL\Type\Definition\Type;
use Railt\Reflection\Contracts\Behavior\AllowsTypeIndication;
filter:
    dependency_paths: ["lib/*"]

/**
 * @property AllowsTypeIndication $reflection
 */
abstract class DependentDefinitionBuilder extends TypeBuilder
{
    /**
     * @return Type
     * @throws \InvalidArgumentException
     */
    protected function buildType(): Type
    {
        /** @var Type $result */
        $type = $this->load($this->reflection->getTypeDefinition());

        if ($this->reflection->isListOfNonNulls()) {
            $type = Type::listOf(Type::nonNull($type));
        } elseif ($this->reflection->isList()) {
            $type = Type::listOf($type);
        }

        if ($this->reflection->isNonNull()) {
            $type = Type::nonNull($type);
        }

        return $type;

    }
}


1			<?php
2			/**
3			* This file is part of Railt package.
4			*
5			* For the full copyright and license information, please view the LICENSE
6			* file that was distributed with this source code.
7			*/
8			declare(strict_types=1);
9
10			namespace Railt\Adapters\Webonyx\Builders;
11
12			use GraphQL\Type\Definition\Type;
13			use Railt\Reflection\Contracts\Behavior\AllowsTypeIndication;
			0 ignored issues – show Bug introduced 2017-12-26 12:08 UTC by Report Bug Copy Issue Report The type `Railt\Reflection\Contrac...or\AllowsTypeIndication` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
14
15			/**
16			* @property AllowsTypeIndication $reflection
17			*/
18			abstract class DependentDefinitionBuilder extends TypeBuilder
19			{
20			/**
21			* @return Type
22			* @throws \InvalidArgumentException
23			*/
24			protected function buildType(): Type
25			{
26			/** @var Type $result */
27			$type = $this->load($this->reflection->getTypeDefinition());
28
29			if ($this->reflection->isListOfNonNulls()) {
30			$type = Type::listOf(Type::nonNull($type));
31			} elseif ($this->reflection->isList()) {
32			$type = Type::listOf($type);
33			}
34
35			if ($this->reflection->isNonNull()) {
36			$type = Type::nonNull($type);
37			}
38
39			return $type;
			0 ignored issues – show Bug Best Practice introduced 2017-12-26 12:08 UTC by Report Bug Copy Issue Report The expression `return $type` could return the type `GraphQL\Type\Definition\Directive` which is incompatible with the type-hinted return `GraphQL\Type\Definition\Type`. Consider adding an additional type-check to rule them out. Loading history...
40			}
41			}
42

railt / webonyx-adapter

Push — master ( 3b79f3...426227 )

DependentDefinitionBuilder::buildType() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like