Thruster\Component\XMLIterator\ChildIterator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	rewind()	0	5	1
A	valid()	0	6	2

<?php

namespace Thruster\Component\XMLIterator;

/**
 * Class ChildIterator
 *
 * @package Thruster\Component\XMLIterator
 * @author  Aurimas Niekis <aurimas@niekis.lt>
 */
class ChildIterator extends XMLIterator
{
    /**
     * @var int
     */
    private $stopDepth;

    public function __construct(XMLReader $reader)
    {
        parent::__construct($reader);

        $this->stopDepth = $reader->depth;
    }

    public function rewind()
    {
        parent::next();

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

        parent::rewind();
    }

    public function valid()
    {
        $parent = parent::valid();

        return $parent && ($this->reader->depth > $this->stopDepth);
    }
}