Apparat\Object\Domain\Model\Object\Collection

Complexity

Total Complexity

23

Size/Duplication

Total Lines	243
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

14.75%

Importance

Changes	6
Bugs	0	Features	0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	current()	0	4	1
A	next()	0	4	1
A	key()	0	4	1
A	valid()	0	4	1
A	rewind()	0	4	1
A	offsetExists()	0	4	1
A	offsetGet()	0	4	1
A	offsetSet()	0	4	1
A	offsetUnset()	0	6	1
A	count()	0	4	1
A	append()	0	5	1
B	__construct()	0	23	4
A	addObject()	0	14	3
A	removeObject()	0	18	3
A	_loadObject()	0	11	2

<?php

/**
 * apparat-object
 *
 * @category    Apparat
 * @package     Apparat\Object\Domain
 * @author      Joschi Kuphal <joschi@kuphal.net> / @jkphl
 * @copyright   Copyright © 2016 Joschi Kuphal <joschi@kuphal.net> / @jkphl
 * @license     http://opensource.org/licenses/MIT	The MIT License (MIT)

Spaces must be used for alignment; tabs are not allowed

 */

/***********************************************************************************
 *  The MIT License (MIT)
 *
 *  Copyright © 2016 Joschi Kuphal <joschi@kuphal.net> / @jkphl
 *
 *  Permission is hereby granted, free of charge, to any person obtaining a copy of
 *  this software and associated documentation files (the "Software"), to deal in
 *  the Software without restriction, including without limitation the rights to
 *  use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
 *  the Software, and to permit persons to whom the Software is furnished to do so,
 *  subject to the following conditions:
 *
 *  The above copyright notice and this permission notice shall be included in all
 *  copies or substantial portions of the Software.
 *
 *  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 *  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
 *  FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
 *  COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
 *  IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
 *  CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 ***********************************************************************************/

namespace Apparat\Object\Domain\Model\Object;

use Apparat\Object\Domain\Model\Path\LocalPath;
use Apparat\Object\Domain\Model\Path\PathInterface;
use Apparat\Object\Domain\Model\Path\RepositoryPath;

/**
 * Lazy loading object collection
 *
 * @package Apparat\Object
 * @subpackage Apparat\Object\Domain
 */
class Collection implements CollectionInterface
{
    /**
     * Objects
     *
     * @var ObjectInterface[]|RepositoryPath[]
     */
    protected $_objects = array();
    /**
     * Object IDs
     *
     * @var array
     */
    protected $_objectIds = array();
    /**
     * Internal object pointer
     *
     * @var int
     */
    protected $_pointer = 0;

    /*******************************************************************************
     * PUBLIC METHODS
     *******************************************************************************/

    /**
     * Collection constructor
     *
     * @param array $objects Collection objects
     * @throws InvalidArgumentException If the an invalid object or path is provided
     */
    public function __construct(array $objects = [])
    {
        foreach ($objects as $object) {

            // If it's an object
            if ($object instanceof ObjectInterface) {
                $this->_objects[$object->getId()->getId()] = $object;

                // Else if it's an object path
            } elseif ($object instanceof RepositoryPath) {
                $this->_objects[$object->getId()->getId()] = $object;

The method getId cannot be called on $object->getId() (of type integer).

                // Else: Error
            } else {
                throw new InvalidArgumentException(
                    'Invalid collection object or path',
                    InvalidArgumentException::INVALID_COLLECTION_OBJECT_OR_PATH
                );
            }
        }

        $this->_objectIds = array_keys($this->_objects);
    }

    /**
     * Return the current object
     *
     * @return ObjectInterface Current object
     */
    public function current()
    {
        return $this->_loadObject($this->_objectIds[$this->_pointer]);
    }

    /**
     * Move forward to next object
     *
     * @return void
     */
    public function next()
    {
        ++$this->_pointer;
    }

    /**
     * Return the ID of the current object
     *
     * @return int Object ID
     */
    public function key()
    {
        return $this->_objectIds[$this->_pointer];
    }

    /**
     * Checks if current position is valid
     *
     * @return boolean The current position is valid
     */
    public function valid()
    {
        return isset($this->_objectIds[$this->_pointer]);
    }

    /**
     * Rewind the Iterator to the first object
     *
     * @return void
     */
    public function rewind()
    {
        $this->_pointer = 0;
    }

    /**
     * Whether an object ID exists
     *
     * @param int $offset Object ID
     * @return boolean Whether the object ID exists
     */
    public function offsetExists($offset)
    {
        return isset($this->_objects[$offset]);
    }

    /**
     * Get an object with a particular ID
     *
     * @param int $offset Object ID
     * @return ObjectInterface Object
     */
    public function offsetGet($offset)
    {
        return $this->_objects[$offset];

The expression $this->_objects[$offset]; of type Apparat\Object\Domain\Model\Object\ObjectInterface|Apparat\Object\Domain\Model\Path\RepositoryPath adds the type Apparat\Object\Domain\Model\Path\RepositoryPath to the return on line 172 which is incompatible with the return type documented by Apparat\Object\Domain\Model\Object\Collection::offsetGet of type Apparat\Object\Domain\Model\Object\ObjectInterface.

    }

    /**
     * Set an object by ID
     *
     * @param int $offset Object ID
     * @param ObjectInterface $value Object
     * @return void
     * @throws RuntimeException When an object should be set by ID
     */
    public function offsetSet($offset, $value)

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

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

    {

    }

    /**
     * Unset an object by ID
     *
     * @param int $offset Object ID
     * @return Collection Object collection with the object removed
     */
    public function offsetUnset($offset)
    {
        $objects = $this->_objects;
        unset($objects[$offset]);
        return new self($objects);
    }

    /**
     * Add an object to the collection
     *
     * @param string|ObjectInterface $object Object or object URL
     * @return Collection Modified object collection
     */
    public function addObject($object)
    {

        // If the object is not yet an object instance
        if (!($object instanceof ObjectInterface)) {
            if (!($object instanceof PathInterface)) {
                $object = new LocalPath(strval($object));
            }
        }

        $objects = $this->_objects;
        $objects[] = $object;
        return new self(array_values($objects));
    }

    /**
     * Remove an object out of this collection
     *
     * @param string|ObjectInterface $object Object or object ID
     * @return Collection Modified object collection
     */
    public function removeObject($object)
    {
        if ($object instanceof ObjectInterface) {
            $object = $object->getId();
        } else {
            $object = intval($object);
        }
        if (empty($this->_objects[$object])) {
            throw new InvalidArgumentException(
                sprintf('Unknown object ID "%s"', $object),
                InvalidArgumentException::UNKNOWN_OBJECT_ID
            );
        }

        $objects = $this->_objects;
        unset($objects[$object]);
        return new self(array_values($objects));
    }

    /**
     * Count objects in this collection
     *
     * @return int The number of objects in this collection
     */
    public function count()
    {
        return count($this->_objects);
    }

    /**
     * Append another collection
     *
     * @param Collection $collection Collection
     * @return Collection Combined collections
     */
    public function append(Collection $collection)
    {
        $objects = array_merge($this->_objects, $collection->_objects);
        return new self(array_values($objects));
    }

    /*******************************************************************************
     * PRIVATE METHODS
     *******************************************************************************/

    /**
     * Load and return an object by ID
     *
     * @param int $objectId Object ID
     * @return ObjectInterface Object
     */
    protected function _loadObject($objectId)
    {
        // Lazy-load the object once
        if ($this->_objects[$objectId] instanceof RepositoryPath) {
            $this->_objects[$objectId] = $this->_objects[$objectId]->getRepository()->loadObject(

The method getRepository does only exist in Apparat\Object\Domain\Model\Path\RepositoryPath, but not in Apparat\Object\Domain\Model\Object\ObjectInterface.

                $this->_objects[$objectId]
            );
        }

        return $this->_objects[$objectId];
    }
}