FieldsNode::toConfig() - Code Metrics - Inspection of "Add support of inheritance for relay-mutation-inpu..." - overblog/GraphQLBundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#423)

unknown

created 2018-11-23 10:17 UTC

FieldsNode::toConfig() B

↳ Parent: FieldsNode

Complexity

Conditions	6
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	29
ccs	14
cts	14
cp	1
rs	8.8337
c	0
b	0
f	0
cc	6
nc	2
nop	2
crap	6

<?php

namespace Overblog\GraphQLBundle\Config\Parser\GraphQL\ASTConverter;

use GraphQL\Language\AST\Node;
use GraphQL\Language\AST\NodeKind;
use GraphQL\Language\AST\ValueNode;

class FieldsNode implements NodeInterface
{
    public static function toConfig(Node $node, $property = 'fields')
    {
        $config = [];
        if (!empty($node->$property)) {
            foreach ($node->$property as $definition) {
                $fieldConfig = [
                    'type' => TypeNode::toConfig($definition),
                    'description' => DescriptionNode::toConfig($definition),
                ];

                if (!empty($definition->arguments)) {
                    $fieldConfig['args'] = self::toConfig($definition, 'arguments');
                }

                if (!empty($definition->defaultValue)) {
                    $fieldConfig['defaultValue'] = self::astValueNodeToConfig($definition->defaultValue);
                }

                $directiveConfig = DirectiveNode::toConfig($definition);
                if (isset($directiveConfig['deprecationReason'])) {
                    $fieldConfig['deprecationReason'] = $directiveConfig['deprecationReason'];
                }

                $config[$definition->name->value] = $fieldConfig;
            }
        }

        return $config;
    }

    private static function astValueNodeToConfig(ValueNode $valueNode)
    {
        $config = null;
        switch ($valueNode->kind) {
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
            case NodeKind::INT:
                $config = \intval($valueNode->value, 10);
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
                break;

            case NodeKind::FLOAT:
                $config = \floatval($valueNode->value);
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
                break;

            case NodeKind::STRING:
            case NodeKind::BOOLEAN:
            case NodeKind::ENUM:
                $config = $valueNode->value;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
                break;

            case NodeKind::LST:
                $config = [];
                foreach ($valueNode->values as $node) {
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
                    $config[] = self::astValueNodeToConfig($node);
                }
                break;

            case NodeKind::NULL:
                $config = null;
                break;
        }

        return $config;
    }
}


1		<?php
2
3		namespace Overblog\GraphQLBundle\Config\Parser\GraphQL\ASTConverter;
4
5		use GraphQL\Language\AST\Node;
6		use GraphQL\Language\AST\NodeKind;
7		use GraphQL\Language\AST\ValueNode;
8
9		class FieldsNode implements NodeInterface
10		{
11		public static function toConfig(Node $node, $property = 'fields')
12	2	{
13		$config = [];
14	2	if (!empty($node->$property)) {
15	2	foreach ($node->$property as $definition) {
16	2	$fieldConfig = [
17	2	'type' => TypeNode::toConfig($definition),
18		'description' => DescriptionNode::toConfig($definition),
19	2	];
20	2
21		if (!empty($definition->arguments)) {
22		$fieldConfig['args'] = self::toConfig($definition, 'arguments');
23	2	}
24	1
25		if (!empty($definition->defaultValue)) {
26		$fieldConfig['defaultValue'] = self::astValueNodeToConfig($definition->defaultValue);
27	2	}
28	2
29	1	$directiveConfig = DirectiveNode::toConfig($definition);
30		if (isset($directiveConfig['deprecationReason'])) {
31		$fieldConfig['deprecationReason'] = $directiveConfig['deprecationReason'];
32	2	}
33
34		$config[$definition->name->value] = $fieldConfig;
35		}
36	2	}
37
38		return $config;
39		}
40
41		private static function astValueNodeToConfig(ValueNode $valueNode)
42		{
43		$config = null;
44		switch ($valueNode->kind) {
		0 ignored issues – show Bug introduced 2017-11-30 08:02 UTC by Report Bug Copy Issue Report Accessing `kind` on the interface `GraphQL\Language\AST\ValueNode` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
45		case NodeKind::INT:
46		$config = \intval($valueNode->value, 10);
		0 ignored issues – show Bug introduced 2018-10-09 14:57 UTC by Report Bug Copy Issue Report Accessing `value` on the interface `GraphQL\Language\AST\ValueNode` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
47		break;
48
49		case NodeKind::FLOAT:
50		$config = \floatval($valueNode->value);
		0 ignored issues – show Bug introduced 2018-10-09 14:57 UTC by Report Bug Copy Issue Report Accessing `value` on the interface `GraphQL\Language\AST\ValueNode` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
51		break;
52
53		case NodeKind::STRING:
54		case NodeKind::BOOLEAN:
55		case NodeKind::ENUM:
56		$config = $valueNode->value;
		0 ignored issues – show Bug introduced 2017-11-30 08:02 UTC by Report Bug Copy Issue Report Accessing `value` on the interface `GraphQL\Language\AST\ValueNode` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
57		break;
58
59		case NodeKind::LST:
60		$config = [];
61		foreach ($valueNode->values as $node) {
		0 ignored issues – show Bug introduced 2017-11-30 08:02 UTC by Report Bug Copy Issue Report Accessing `values` on the interface `GraphQL\Language\AST\ValueNode` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
62		$config[] = self::astValueNodeToConfig($node);
63		}
64		break;
65
66		case NodeKind::NULL:
67		$config = null;
68		break;
69		}
70
71		return $config;
72		}
73		}
74

overblog / GraphQLBundle

Scrutinizer GitHub App not installed

Pull Request — master (#423)

FieldsNode::toConfig() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like