Germania\Salesmen\ActiveSalesmanFilterIterator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

87.5%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	2
A	accept()	0	12	2

<?php
namespace Germania\Salesmen;

class ActiveSalesmanFilterIterator extends \FilterIterator
{

    /**
     * @var boolean
     */
    public $active_flag = true;

    /**
     * @param \Traversable $collection      Collection of SalesmanIdProviderInterface
     * @param bool $active_flag
     */
    public function __construct( \Traversable $collection, $active_flag = true )

The parameter $active_flag is not used and could be removed.

    {
        // Take care of Traversable's two faces,
        // since both IteratorAggregate and Iterator implement Traversable
        parent::__construct( $collection instanceOf \IteratorAggregate ? $collection->getIterator() : $collection );
    }

    public function accept()
    {
        $item = $this->getInnerIterator()->current();

        // Disclose items not implementing SalesmanIdProviderInterface
        if (!$item instanceOf SalesmanIdProviderInterface) {
            return false;
        }

        return $item->isActive() == $this->active_flag;

    }

}