Vivait\TenantBundle\Locator\HeaderLocator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getTenant()	0	8	2
A	getTenantFromRequest()	0	6	1

<?php

namespace Vivait\TenantBundle\Locator;

use Symfony\Component\HttpFoundation\Request;

class HeaderLocator implements TenantLocator
{
    private $request;

    /**
     * @var string
     */
    private $headerKey;

    public function __construct(Request $request, $headerKey)

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->headerKey = $headerKey;
    }

    /**
     * @return string Tenant key
     */
    public function getTenant()
    {
        if (!$this->request->headers->has($this->headerKey)) {
            throw new \RuntimeException('Could not locate a tenant header in the request');
        }

        return $this->request->headers->get($this->headerKey);
    }

    /**
     * @param Request $request
     * @param string $headerKey
     * @return string Tenant key
     */
    public static function getTenantFromRequest(Request $request, $headerKey = 'tenant')
    {
        $locator = new self($request, $headerKey);

        return $locator->getTenant();
    }
}