Larabros\Laranav\Manager

Complexity

Total Complexity

8

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

6 Methods

Rating	Name	Duplication	Size	Complexity
A	menu()	0	5	2
A	getDefaultDriver()	0	4	1
A	driver()	0	4	2
A	createDriver()	0	13	1
A	getConfig()	0	4	1
A	getMenuItems()	0	4	1

<?php

namespace Larabros\Laranav;

use Larabros\Laranav\Menus\Menu;
use Illuminate\Support\Manager as AbstractManager;
use Illuminate\Contracts\Foundation\Application;

class Manager extends AbstractManager
{
    /**
     * The array of menus as defined in the config files.
     *
     * @var array
     */
    protected $menus;

    /**
     * Get a `Menu` instance.
     *
     * @param  string $name

Should the type for parameter $name not be string|null?

     *
     * @return Menu
     */
    public function menu($name = null)
    {
        $name = $name ?: $this->getDefaultDriver();
        return $this->menus[$name] = $this->driver($name);
    }

    /**
     * @inheritDoc
     */
    public function getDefaultDriver()
    {
        return 'default';
    }

    /**
     * @inheritDoc
     */
    public function driver($name = null)

The return type could not be reliably inferred; please add a @return annotation.

    {
        return isset($this->menus[$name]) ? $this->menus[$name] : $this->createDriver($name);
    }

    /**
     * @inheritDoc
     */
    protected function createDriver($name)
    {
        $config = array_merge($this->getConfig($this->getDefaultDriver()), $this->getConfig($name));
        $items  = $this->getMenuItems($name);

        return new Menu(
            $name,
            $items,
            $config,
            $this->app->make('url'),
            $this->app->make('view')
        );
    }

    /**
     * Get a menu's configuration.
     *
     * @param  string  $name
     *
     * @return array
     */
    protected function getConfig($name)
    {
        return $this->app->make('config')->get("laranav.config.{$name}");

As per coding-style, please use concatenation or sprintf for the variable $name instead of interpolation.

    }

    /**
     * Get a menu's items.
     *
     * @param  string  $name
     *
     * @return array
     */
    protected function getMenuItems($name)
    {
        return $this->app->make('config')->get("laranav.menus.{$name}");

As per coding-style, please use concatenation or sprintf for the variable $name instead of interpolation.

    }
}