Vivait\TenantBundle\Locator\HostnameLocator

Complexity

Total Complexity

6

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getTenant()	0	10	2
A	getPattern()	0	4	1
A	setPattern()	0	6	1
A	getTenantFromRequest()	0	4	1

<?php

namespace Vivait\TenantBundle\Locator;

use Symfony\Component\HttpFoundation\Request;

/**
 * @see HostnameLocatorSpec
 */
class HostnameLocator implements TenantLocator
{
    private $pattern = '#^(?P<tenant>.+?)\.(.*?)\.#';
    private $request;

    public function __construct( Request $request )

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;
    }

    /**
     * @return string Tenant key
     */
    public function getTenant() {
        // Get the server name
        $host = $this->request->getHost();

        if (!preg_match( $this->pattern, $host, $matches )) {
            throw new \RuntimeException(sprintf('Could not match tenant for host "%s"', $host));
        }

        return $matches['tenant'];
    }

    /**
     * Gets the REGEX pattern used to match a tenant
     * @return string
     */
    public function getPattern()
    {
        return $this->pattern;
    }

    /**
     * Sets the REGEX pattern used to match a tenant
     * @param string $pattern
     * @return $this
     */
    public function setPattern( $pattern )
    {
        $this->pattern = $pattern;

        return $this;
    }

    /**
     * @param Request $request
     * @return string Tenant key
     */
    public static function getTenantFromRequest(Request $request) {
        $locator = new self($request);
        return $locator->getTenant();
    }
}