ElfSundae\Laravel\Apps\Apps

Complexity

Total Complexity

11

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	id()	0	12	4
A	refreshId()	0	6	1
B	idForUrl()	0	18	5

<?php

namespace ElfSundae\Laravel\Apps;

use Illuminate\Contracts\Container\Container;

class Apps
{
    /**
     * The container instance.
     *
     * @var \Illuminate\Contracts\Container\Container
     */
    protected $container;

    /**
     * The current application identifier.
     *
     * @var string|null|false
     */
    protected $id = false;

    /**
     * Create a new Apps instance.
     *
     * @param  \Illuminate\Contracts\Container\Container  $container
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(Container $container)
    {
        $this->container = $container;

        $this->container->rebinding('request', function () {
            $this->refreshId();
        });
    }

    /**
     * Get or check the current application identifier.
     *
     * @return string|bool
     */
    public function id()
    {
        if ($this->id === false) {
            $this->id = $this->idForUrl($this->container['request']->getUri());

It seems like $this->idForUrl($this->container['request']->getUri()) can also be of type integer. However, the property $id is declared as type string|null|false. Maybe add an additional type check?

        }

        if (func_num_args() > 0) {
            return in_array($this->id, is_array(func_get_arg(0)) ? func_get_arg(0) : func_get_args());
        }

        return $this->id;
    }

    /**
     * Refresh the current application identifier.
     *
     * @return $this
     */
    public function refreshId()
    {
        $this->id = false;

        return $this;
    }

    /**
     * Get application identifier for the given URL.
     *
     * @param  string  $url
     * @return string
     */
    public function idForUrl($url)
    {
        $identifier = null;

        foreach ($this->container['config']['apps.url'] as $id => $root) {
            $root = preg_replace('~^https?://~i', '', $root);
            $pattern = '~^https?://'.preg_quote($root, '~').'([/\?#].*)?$~i';
            if (preg_match($pattern, $url)) {
                $len = strlen($root);
                if (! isset($length) || $length < $len) {
                    $length = $len;
                    $identifier = $id;
                }
            }
        }

        return $identifier;
    }
}