Vivait\TenantBundle\Locator\GetParameterLocator

Complexity

Total Complexity

5

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getTenant()	0	14	3
A	getTenantFromRequest()	0	6	1

<?php

namespace Vivait\TenantBundle\Locator;

use Symfony\Component\HttpFoundation\Request;
use Vivait\TenantBundle\Registry\TenantRegistry;

class GetParameterLocator
{

    /**
     * @var Request
     */
    private $request;

    /**
     * @var string
     */
    private $parameterName;
    /**
     * @var TenantRegistry
     */
    private $registry;

    /**
     * @param Request        $request
     * @param string         $parameterName
     * @param TenantRegistry $registry
     */
    public function __construct(Request $request, $parameterName, TenantRegistry $registry)

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

    {
        $this->request = $request;
        $this->parameterName = $parameterName;
        $this->registry = $registry;
    }

    /**
     * @throws \RuntimeException
     * @throws \OutOfBoundsException
     * @return string
     */
    public function getTenant()
    {
        $tenantName = $this->request->get($this->parameterName, null);

        if ( ! isset($tenantName)) {
            throw new \RuntimeException;
        }

        if ( ! $this->registry->contains($tenantName)) {
            throw new \OutOfBoundsException;
        }

        return $tenantName;
    }

    /**
     * @param Request        $request
     * @param string         $parameterName
     * @param TenantRegistry $registry
     *
     * @return mixed
     */
    public static function getTenantFromRequest(Request $request, $parameterName = 'tenant', TenantRegistry $registry)
    {
        $locator = new self($request, $parameterName, $registry);

        return $locator->getTenant();
    }
}