Tidal\WampWatch\Model\Property\Collection\HasProceduresTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	addProcedure()	0	4	1
A	getProcedures()	0	4	1
A	hasProcedure()	0	4	1
A	getProcedure()	0	4	1
A	removeProcedure()	0	4	1
A	listProcedures()	0	6	2
A	setProcedures()	0	5	1

<?php

/*
 * This file is part of the Tidal/WampWatch package.
 *   (c) 2016 Timo Michna <timomichna/yahoo.de>
 *
 *  For the full copyright and license information, please view the LICENSE
 *  file that was distributed with this source code.
 *
 */

namespace Tidal\WampWatch\Model\Property\Collection;

use Tidal\WampWatch\Model\Contract;
use Tidal\WampWatch\Model\Contract\Property\ObjectCollectionInterface;

/**
 * Class HasProceduresTrait.
 *
 * Important! Classes using this trait have to also use trait
 * Tidal\WampWatch\Model\Behavior\Property\HasCollectionsTrait
 * for this trait to work;
 */
trait HasProceduresTrait
{
    protected static $proceduresPropertyName = 'procedures';

    /**
     * @var ObjectCollectionInterface
     */
    private $procedures;

    public function addProcedure(Contract\ProcedureInterface $procedure)
    {
        $this->getProcedures()->set($procedure->getUri(), $procedure);
    }

    /**
     * @return ObjectCollectionInterface
     */
    private function getProcedures()
    {
        return $this->getCollection(static::$proceduresPropertyName);

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

    }

    public function hasProcedure($uri)
    {
        return $this->getProcedures()->has($uri);
    }

    /**
     * @param string $uri
     *
     * @return Contract\ProcedureInterface
     */
    public function getProcedure($uri)
    {
        return $this->getProcedures()->get($uri);
    }

    /**
     * @param string $uri
     */
    public function removeProcedure($uri)
    {
        $this->getProcedures()->offsetUnset($uri);
    }

    /**
     * @return \Generator
     */
    public function listProcedures()
    {
        foreach ($this->getProcedures()->getIterator() as $uri => $procedure) {
            yield $uri => $procedure;
        }
    }

    /**
     * @param ObjectCollectionInterface $procedures
     */
    private function setProcedures(ObjectCollectionInterface $procedures)
    {
        $this->initCollection(static::$proceduresPropertyName, $procedures);

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

        $procedures->setObjectConstrain(Contract\ProcedureInterface::class);
    }
}