Phile\Repository\PageCollection

Complexity

Total Complexity

10

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

63.33%

Importance

Changes	1
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	toArray()	0	5	1
A	__construct()	0	4	1
A	getIterator()	0	5	1
A	offsetExists()	0	5	1
A	offsetGet()	0	5	1
A	offsetSet()	0	5	1
A	offsetUnset()	0	5	1
A	count()	0	5	1
A	load()	0	6	2

<?php

namespace Phile\Repository;

/**
 * Page collection which delays searching for and loading pages until necessary.
 *
 * @author  PhileCMS
 * @link    https://philecms.com
 * @license http://opensource.org/licenses/MIT
 * @package Phile\Repository
 */
class PageCollection implements \ArrayAccess, \IteratorAggregate, \Countable
{
    /**
     * @var callback A function to be used for loading the pages.
     */
    private $loader;

    /**
     * @var array of \Phile\Model\Page
     */
    private $pages;

    /**
     * Constructor.
     *
     * @param callable $loader pages loader
     */
    public function __construct(callable $loader)
    {
        $this->loader = $loader;
    }

    /**
     * Perform page loading.
     *
     * @return void
     */
    private function load()
    {
        if ($this->pages === null) {
            $this->pages = call_user_func($this->loader);

It seems like call_user_func($this->loader) of type * is incompatible with the declared type array of property $pages.

        }
    }

    /**
     * Get pages in a array.
     *
     * @return array of \Phile\Model\Page
     */
    public function toArray()
    {
        $this->load();
        return $this->pages;
    }

    public function getIterator()
    {
        $this->load();
        return new \ArrayIterator($this->pages);
    }

    public function offsetExists($offset)
    {
        $this->load();
        return isset($this->pages[$offset]);
    }

    public function offsetGet($offset)
    {
        $this->load();
        return $this->pages[$offset];
    }

    public function offsetSet($offset, $value)
    {
        $this->load();
        $this->pages[$offset] =    $value;
    }

    public function offsetUnset($offset)
    {
        $this->load();
        unset($this->pages[$offset]);
    }

    public function count()
    {
        $this->load();
        return count($this->pages);
    }
}