GroupUseReplacer::leaveNode() - Code Metrics - spatie/7to5 - Measure and Improve Code Quality continuously with Scrutinizer

GroupUseReplacer::leaveNode() A
last analyzed 2018-10-24 11:34 UTC

↳ Parent: GroupUseReplacer

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	14
rs	9.7998
c	0
b	0
f	0
cc	2
nc	2
nop	1

<?php

namespace Spatie\Php7to5\NodeVisitors;

use PhpParser\Node;
use PhpParser\NodeVisitorAbstract;

class GroupUseReplacer extends NodeVisitorAbstract
{
    /**
     * {@inheritdoc}
     */
    public function leaveNode(Node $node)
    {
        if (!$node instanceof Node\Stmt\GroupUse) {
            return;
        }

        $nodePrefixParts = $node->prefix->parts;

        $seperateUseStatements = array_map(function ($useNode) use ($nodePrefixParts) {
            return $this->createUseNode($nodePrefixParts, $useNode);
        }, $node->uses);

        return $seperateUseStatements;
    }

    protected function createUseNode(array $nodePrefixParts, Node $useNode)
    {
        $fullClassName = array_merge($nodePrefixParts, [$useNode->name]);
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

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

        $nameNode = new Node\Name($fullClassName);

        $alias = ($useNode->alias == $useNode->name) ? null : $useNode->alias;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

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

        $useNode = new Node\Stmt\Use_([new Node\Stmt\UseUse($nameNode, $alias)], $useNode->type);
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

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

        return $useNode;
    }
}


GroupUseReplacer::leaveNode() A
last analyzed 2018-10-24 11:34 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1			<?php
2
3			namespace Spatie\Php7to5\NodeVisitors;
4
5			use PhpParser\Node;
6			use PhpParser\NodeVisitorAbstract;
7
8			class GroupUseReplacer extends NodeVisitorAbstract
9			{
10			/**
11			* {@inheritdoc}
12			*/
13			public function leaveNode(Node $node)
14			{
15			if (!$node instanceof Node\Stmt\GroupUse) {
16			return;
17			}
18
19			$nodePrefixParts = $node->prefix->parts;
20
21			$seperateUseStatements = array_map(function ($useNode) use ($nodePrefixParts) {
22			return $this->createUseNode($nodePrefixParts, $useNode);
23			}, $node->uses);
24
25			return $seperateUseStatements;
26			}
27
28			protected function createUseNode(array $nodePrefixParts, Node $useNode)
29			{
30			$fullClassName = array_merge($nodePrefixParts, [$useNode->name]);
			0 ignored issues – show Bug introduced 2016-03-26 22:42 UTC by Report Bug Copy Issue Report Accessing `name` on the interface `PhpParser\Node` 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...
31
32			$nameNode = new Node\Name($fullClassName);
33
34			$alias = ($useNode->alias == $useNode->name) ? null : $useNode->alias;
			0 ignored issues – show Bug introduced 2016-03-26 22:42 UTC by Report Bug Copy Issue Report Accessing `alias` on the interface `PhpParser\Node` 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... Bug introduced 2016-03-26 22:42 UTC by Report Bug Copy Issue Report Accessing `name` on the interface `PhpParser\Node` 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...
35
36			$useNode = new Node\Stmt\Use_([new Node\Stmt\UseUse($nameNode, $alias)], $useNode->type);
			0 ignored issues – show Bug introduced 2016-03-26 22:42 UTC by Report Bug Copy Issue Report Accessing `type` on the interface `PhpParser\Node` 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...
37
38			return $useNode;
39			}
40			}
41

spatie / 7to5

GroupUseReplacer::leaveNode() A last analyzed 2018-10-24 11:34 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

GroupUseReplacer::leaveNode() A
last analyzed 2018-10-24 11:34 UTC