Scriptotek\Alma\Model\PaginatedListGenerator

Complexity

Total Complexity

7

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	rewind()	0	6	2
A	valid()	0	8	2
A	current()	0	4	1
A	next()	0	4	1
A	key()	0	4	1

<?php

namespace Scriptotek\Alma\Model;

trait PaginatedListGenerator
{
    /* @var integer */
    protected $position = 0;

    /**
     * Rewind the Iterator to the first element.
     *
     * @link http://php.net/manual/en/iterator.rewind.php
     *
     * @return void
     */
    public function rewind()
    {
        if ($this->position > 0) {
            throw new \Exception('Cannot rewind a generator that was already run');
        }
    }

    /**
     * Checks if current position is valid.
     *
     * @link http://php.net/manual/en/iterator.valid.php
     *
     * @return bool
     */
    public function valid()
    {
        if (!isset($this->resources[0])) {

The property resources does not exist. Did you maybe forget to declare it?

            $this->fetchBatch();

It seems like fetchBatch() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return isset($this->resources[0]);
    }

    /**
     * Return the current element.
     *
     * @link http://php.net/manual/en/iterator.current.php
     *
     * @return mixed
     */
    public function current()
    {
        return array_shift($this->resources);
    }

    /**
     * Move forward to next element.
     *
     * @link http://php.net/manual/en/iterator.next.php
     *
     * @return void
     */
    public function next()
    {
        $this->position++;
    }

    /**
     * Return the key of the current element.
     *
     * @link http://php.net/manual/en/iterator.key.php
     *
     * @return int|null Scalar on success, or null on failure.
     */
    public function key()
    {
        return $this->position;
    }
}