Storeman\Index\RecursiveIndexIterator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	current()	0	4	1
A	hasChildren()	0	4	1
A	getChildren()	0	4	1

<?php

namespace Storeman\Index;

class RecursiveIndexIterator extends \ArrayIterator implements \RecursiveIterator
{
    /**
     * @var IndexNode
     */
    protected $indexNode;

    public function __construct(IndexNode $indexNode)
    {
        $this->indexNode = $indexNode;

        parent::__construct($indexNode->getChildren());
    }

    /**
     * {@inheritdoc}
     */
    public function current()
    {
        return parent::current()->getIndexObject();
    }

    /**
     * {@inheritdoc}
     */
    public function hasChildren()
    {
        return count($this->indexNode) > 0;
    }

    /**
     * {@inheritdoc}
     */
    public function getChildren()
    {
        return new static(parent::current());

It seems like you call parent on a different method (current() instead of getChildren()). Are you sure this is correct? If so, you might want to change this to $this->current().

    }
}