SAML2_Compat_ContainerSingleton

Complexity

Total Complexity

4

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	ContainerSingleton::getInstance()	0	7	2
A	ContainerSingleton::setContainer()	0	5	1

<?php

namespace SAML2\Compat;

use SAML2\Compat\Ssp\Container;

class ContainerSingleton
{
    /**
     * @var \SAML2\Compat\Ssp\Container
     */
    protected static $container;

    /**
     * @return \SAML2\Compat\Ssp\Container
     */
    public static function getInstance()
    {
        if (!self::$container) {
            self::setContainer(self::initSspContainer());
        }
        return self::$container;
    }

    /**
     * Set a container to use.
     *
     * @param \SAML2\Compat\AbstractContainer $container
     * @return \SAML2\Compat\AbstractContainer
     */
    public static function setContainer(AbstractContainer $container)
    {
        self::$container = $container;

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

        return $container;
    }

    /**
     * @return \SAML2\Compat\Ssp\Container
     */
    public static function initSspContainer()
    {
        return new Container();
    }
}