SilverStripe\Core\Manifest\ClassManifestVisitor

Complexity

Total Complexity

14

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	resetState()	0	6	1
A	beforeTraverse()	0	4	1
D	enterNode()	0	36	9
A	getClasses()	0	4	1
A	getTraits()	0	4	1
A	getInterfaces()	0	4	1

<?php

namespace SilverStripe\Core\Manifest;

use PhpParser\Node;
use PhpParser\NodeTraverser;
use PhpParser\NodeVisitorAbstract;

class ClassManifestVisitor extends NodeVisitorAbstract
{

    private $classes = [];

    private $traits = [];

    private $interfaces = [];

    public function resetState()
    {
        $this->classes = [];
        $this->traits = [];
        $this->interfaces = [];
    }

    public function beforeTraverse(array $nodes)
    {
        $this->resetState();
    }

    public function enterNode(Node $node)
    {
        if ($node instanceof Node\Stmt\Class_) {
            $extends = '';
            $interfaces = [];

            if ($node->extends) {
                $extends = array((string)$node->extends);
            }

            if ($node->implements) {

The expression $node->implements of type PhpParser\Node\Name[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                foreach ($node->implements as $interface) {
                    $interfaces[] = (string)$interface;
                }
            }

            $this->classes[(string)$node->namespacedName] = [

The property namespacedName does not seem to exist in PhpParser\Node\Stmt\Class_.

                'extends' => $extends,
                'interfaces' => $interfaces,
            ];
        } elseif ($node instanceof Node\Stmt\Trait_) {
            $this->traits[(string)$node->namespacedName] = array();

The property namespacedName does not seem to exist in PhpParser\Node\Stmt\Trait_.

        } elseif ($node instanceof Node\Stmt\Interface_) {
            $extends = array();
            foreach ($node->extends as $ancestor) {

The expression $node->extends of type null|object<PhpParser\Node\Name> is not guaranteed to be traversable. How about adding an additional type check?

                $extends[] = (string)$ancestor;
            }
            $this->interfaces[(string)$node->namespacedName] = [

The property namespacedName does not seem to exist in PhpParser\Node\Stmt\Interface_.

                'extends' => $extends,
            ];
        }
        if (!$node instanceof Node\Stmt\Namespace_) {
            //break out of traversal as we only need highlevel information here!
            return NodeTraverser::DONT_TRAVERSE_CHILDREN;
        }
    }

    public function getClasses()
    {
        return $this->classes;
    }

    public function getTraits()
    {
        return $this->traits;
    }

    public function getInterfaces()
    {
        return $this->interfaces;
    }
}