Scriptotek\Alma\Model\PaginatedList

Complexity

Total Complexity

7

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	fetchData()	0	4	1
A	rewind()	0	4	1
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 PaginatedList
{
    /* @var integer */
    protected $position = 0;

    /**
     * Fetch all the data.
     */
    protected function fetchData()
    {
        return iterator_to_array($this);
    }

    /**
     * Rewind the Iterator to the first element.
     *
     * @link http://php.net/manual/en/iterator.rewind.php
     * @return void
     */
    public function rewind()
    {
        $this->position = 0;
    }

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

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[$this->position]);
    }

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

    /**
     * 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 integer|null Scalar on success, or null on failure.
     */
    public function key()
    {
        return $this->position;
    }
}