WebinoAppLib\Application\Traits\ServicesTrait

Complexity

Total Complexity

19

Size/Duplication

Total Lines	117
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getServices()	0	4	1
A	get()	0	9	2
D	set()	0	28	9
A	has()	0	4	1
A	requireService()	0	8	2
A	optionalService()	0	4	2
A	setService()	0	4	1
A	setServicesService()	0	7	1

<?php
/**
 * Webino™ (http://webino.sk)
 *
 * @link        https://github.com/webino for the canonical source repository
 * @copyright   Copyright (c) 2015-2017 Webino, s.r.o. (http://webino.sk)
 * @author      Peter Bačinský <peter@bacinsky.sk>
 * @license     BSD-3-Clause
 */

namespace WebinoAppLib\Application\Traits;

use WebinoAppLib\Exception;
use Zend\ServiceManager\Exception\ServiceNotFoundException;
use Zend\ServiceManager\FactoryInterface;
use Zend\ServiceManager\ServiceManager;

/**
 * Trait ServiceProviderTrait
 */
trait ServicesTrait
{
    /**
     * @var ServiceManager
     */
    private $services;

    /**
     * @return ServiceManager
     */
    public function getServices()
    {
        return $this->services;
    }

    /**
     * Return registered service
     *
     * @param string $service Service name
     * @return mixed
     * @throws Exception\UnknownServiceException
     */
    public function get($service)
    {
        try {
            return $this->getServices()->get($service);
        } catch (ServiceNotFoundException $exc) {
            throw (new Exception\UnknownServiceException('Unable to get an instance for %s', null, $exc))
                ->format($service);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function set($service, $factory = null)
    {
        $services = $this->getServices();

        if ($factory instanceof FactoryInterface
            || is_callable($factory)
            || (is_string($factory) && class_exists($factory))
        ) {
            // factory
            $services->setFactory($service, $factory);

It seems like $factory can also be of type object<Zend\ServiceManager\FactoryInterface>; however, Zend\ServiceManager\ServiceManager::setFactory() does only seem to accept callable, maybe add an additional type check?

            return $this;
        }

        if (null !== $factory && is_string($service)) {
            // service object
            $services->setService($service, $factory);
            return $this;
        }

        // invokable
        if (is_array($service)) {
            $services->setInvokableClass(key($service), current($service));
        } elseif (is_string($service)) {
            $services->setInvokableClass($service, $service);
        }

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function has($service)
    {
        return $this->getServices()->has((string) $service);
    }

    /**
     * Require service from services into application
     *
     * @param string $service Service name
     * @throws Exception\DomainException Unable to get service
     */
    protected function requireService($service)
    {
        if (!$this->services->has($service)) {
            throw (new Exception\DomainException('Unable to get required application service %s'))->format($service);
        }

        $this->setService($service);
    }

    /**
     * Set optional service from services into application
     *
     * @param string $service Service name
     */
    protected function optionalService($service)
    {
        $this->services->has($service) and $this->setService($service);

Using logical operators such as and instead of && is generally not recommended.

    }

    /**
     * @param $service
     */
    private function setService($service)
    {
        call_user_func([$this, 'set' . $service], $this->services->get($service), false);
    }

    /**
     * @param string $name
     * @param mixed $service
     */
    protected function setServicesService($name, $service)
    {
        $this->services
            ->setAllowOverride(true)
            ->setService($name, $service)
            ->setAllowOverride(false);
    }
}