Spatie\Php7to5\NodeVisitors\GroupUseReplacer

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	leaveNode()	0	14	2
A	createUseNode()	0	12	2

<?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]);

Accessing name on the interface PhpParser\Node suggest that you code against a concrete implementation. How about adding an instanceof check?

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

        $alias = ($useNode->alias == $useNode->name) ? null : $useNode->alias;

Accessing alias on the interface PhpParser\Node suggest that you code against a concrete implementation. How about adding an instanceof check?

Accessing name on the interface PhpParser\Node suggest that you code against a concrete implementation. How about adding an instanceof check?

        $useNode = new Node\Stmt\Use_([new Node\Stmt\UseUse($nameNode, $alias)], $useNode->type);

Accessing type on the interface PhpParser\Node suggest that you code against a concrete implementation. How about adding an instanceof check?

        return $useNode;
    }
}