Mbh\Tree\Visitor\PreOrder

Complexity

Total Complexity

2

Size/Duplication

Total Lines	14
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	visit()	0	12	2

<?php namespace Mbh\Tree\Visitor;

use Mbh\Tree\Interfaces\Node as NodeInterface;
use Mbh\Tree\Visitor;

class PreOrder extends Visitor
{
    public function visit(NodeInterface $node)
    {
        $nodes = [$node];

        foreach ($node->getChildren() as $child) {
            $nodes = array_merge(
                $nodes,
                $child->accept($this)

Are you sure the usage of $child->accept($this) targeting Mbh\Tree\Interfaces\Node::accept() seems to always return null.

$child->accept($this) of type void is incompatible with the type null|array expected by parameter $array2 of array_merge().

            );
        }

        return $nodes;
    }
}