Slick\Di\ContainerAwareMethods

Complexity

Total Complexity

2

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getContainer()	0	4	1
A	setContainer()	0	5	1

<?php

/**
 * This file is part of slick/di
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Slick\Di;

use Interop\Container\ContainerInterface as InteropContainer;

/**
 * Implementation methods for ContainerAwareInterface
 *
 * @package Slick\Di
 * @author  Filipe Silva <sivam.filipe@gmail.com>
 */
trait ContainerAwareMethods
{

    /**
     * @var ContainerInterface
     */
    protected $container;

    /**
     * Get container
     *
     * @return InteropContainer
     */
    public function getContainer()
    {
        return $this->container;
    }

    /**
     * Set container
     *
     * @param InteropContainer $container
     *
     * @return self|$this|ContainerAwareMethods

The return type ContainerAwareMethods is a trait, and thus cannot be used for type-hinting in PHP. Maybe consider adding an interface and use that for type-hinting?

     */
    public function setContainer(InteropContainer $container)
    {
        $this->container = $container;

$container is of type object<Interop\Container\ContainerInterface>, but the property $container was declared to be of type object<Slick\Di\ContainerInterface>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        return $this;
    }
}