App\Classes\Library\PageLoader\NavItem

Complexity

Total Complexity

7

Size/Duplication

Total Lines	89
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
A	addChild()	0	4	1
A	classState()	0	4	2
A	url()	0	4	1
A	title()	0	4	1
A	target()	0	4	1

<?php
/**
 * Created by PhpStorm.
 * User: Marky
 * Date: 30/12/2017
 * Time: 18:47.
 */

namespace App\Classes\Library\PageLoader;

/*
 * Class NavItem
 *
 * @package App\Classes\Library\PageLoader
 */

use App\Model\Menu;
use Illuminate\Support\Collection;

/**
 * Class NavItem.
 */
class NavItem
{
    /**
     * @var string
     */
    protected $text;

    /**
     * @var string
     */
    protected $url = '#';

    /**
     * @var int
     */
    protected $order;

    /**
     * @var string
     */
    protected $target;

    /**
     * @var bool
     */
    protected $active = false;

    /**
     * @var Collection
     */
    public $children = [];

    /**
     * NavItem constructor.
     *
     * @param Menu $menu
     * @param bool $active
     */
    public function __construct(Menu $menu, bool $active = false)
    {
        $this->text = $menu->title;

        $this->url = $menu->link();

        $this->active = $active;

        $this->children = new Collection;
    }

    /**
     * @param NavItem $navItem

Should the type for parameter $navItem not be \self?

     */
    public function addChild(self $navItem)
    {
        $this->children->put($navItem->text, $navItem);
    }

    /**
     * @return bool

Should the return type not be string?

     */
    public function classState()
    {
        return $this->active ? 'active' : 'inactive';
    }

    /**
     * @return string

Should the return type not be \Illuminate\Contracts\Routing\UrlGenerator|string?

     */
    public function url()
    {
        return url($this->url);
    }

    /**
     * @return string
     */
    public function title()
    {
        return $this->text;
    }

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use string.

     */
    public function target()
    {
        return $this->target;
    }
}