Sco\Admin\Component\Concerns\HasNavigation

Complexity

Total Complexity

17

Size/Duplication

Total Lines	198
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	getPageId()	0	4	1
A	setPageId()	0	6	1
A	getIcon()	0	4	1
A	setIcon()	0	6	1
A	getParentPageId()	0	4	1
A	setParentPageId()	0	6	1
A	hasParentPageId()	0	4	1
A	getPriority()	0	4	1
A	setPriority()	0	6	1
A	getNavigation()	0	4	1
A	addToNavigation()	0	21	3
A	makePage()	0	19	3
A	getDisplayUrl()	0	6	1

<?php

namespace Sco\Admin\Component\Concerns;

use InvalidArgumentException;
use KodiComponents\Navigation\Contracts\BadgeInterface;
use Sco\Admin\Navigation\Badge;
use Sco\Admin\Navigation\Page;

trait HasNavigation
{
    /**
     * The page id name.
     *
     * @var string|null
     */
    protected $pageId;

    /**
     * The page icon class name.
     *
     * @var string|null
     */
    protected $icon;

    /**
     * The page belong to page id name.
     *
     * @var string
     */
    protected $parentPageId;

    /**
     * The page priority.
     *
     * @var int
     */
    protected $priority = 100;

    /**
     * Get the page id
     *
     * @return string|null
     */
    public function getPageId()
    {
        return $this->pageId;
    }

    /**
     * Set the page id
     *
     * @param string $pageId
     *
     * @return $this
     */
    public function setPageId(string $pageId)
    {
        $this->pageId = $pageId;

        return $this;
    }

    /**
     * Get the page icon class name.
     *
     * @return string|null
     */
    public function getIcon()
    {
        return $this->icon;
    }

    /**
     * Set the page icon class name.
     *
     * @param string $icon
     *
     * @return $this
     */
    public function setIcon(string $icon)
    {
        $this->icon = $icon;

        return $this;
    }

    /**
     *
     * @return string
     */
    public function getParentPageId(): string
    {
        return $this->parentPageId;
    }

    /**
     * @param string $parentPageId
     *
     * @return $this
     */
    public function setParentPageId(string $parentPageId)
    {
        $this->parentPageId = $parentPageId;

        return $this;
    }

    public function hasParentPageId()
    {
        return $this->parentPageId !== null;
    }

    /**
     * @return int
     */
    public function getPriority(): int
    {
        return $this->priority;
    }

    /**
     * @param int $priority
     *
     * @return $this
     */
    public function setPriority(int $priority)
    {
        $this->priority = $priority;

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function getNavigation()
    {
        return $this->app['admin.navigation'];

The property app does not exist. Did you maybe forget to declare it?

    }

    /**
     * {@inheritdoc}
     */
    public function addToNavigation($badge = null)
    {
        $nav = $this->getNavigation();
        if ($this->hasParentPageId()) {
            $nav = $nav->getPages()->findById($this->getParentPageId());
            if (is_null($nav)) {
                throw new InvalidArgumentException(
                    sprintf(
                        'Not Found "%s" navigation',
                        $this->getParentPageId()
                    )
                );
            }
        }

        $page = $this->makePage($badge);

        $nav->addPage($page);

        return $page;
    }

    /**
     * Make page
     *
     * @param string|Closure|null $badge
     *
     * @return \Sco\Admin\Navigation\Page
     */
    protected function makePage($badge = null)
    {
        $page = new Page($this);
        $page->setPriority($this->getPriority())
            ->setIcon($this->getIcon())
            ->setId($this->getPageId())
            ->setAccessLogic(function () {
                return $this->isDisplay();

It seems like isDisplay() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            });

        if ($badge) {
            if (! ($badge instanceof BadgeInterface)) {
                $badge = new Badge($badge);
            }
            $page->addBadge($badge);
        }

        return $page;
    }

    /**
     * Get the model display url.
     *
     * @param array $parameters
     *
     * @return string
     */
    public function getDisplayUrl(array $parameters = [])
    {
        array_unshift($parameters, $this->getName());

It seems like getName() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return route('admin.model.index', $parameters, false);
    }
}