Hyde\Framework\Features\Navigation\BaseNavigationMenu

Complexity

Total Complexity

7

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes	33
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	removeDuplicateItems()	0	5	1
A	canAddRoute()	0	3	1
A	generate()	0	11	2
A	__construct()	0	3	1
A	sortByPriority()	0	3	1
A	create()	0	9	1

<?php

declare(strict_types=1);

namespace Hyde\Framework\Features\Navigation;

use Hyde\Facades\Config;
use Hyde\Support\Models\Route;
use Hyde\Foundation\Facades\Routes;
use Illuminate\Support\Collection;
use function collect;

/**
 * @see \Hyde\Framework\Testing\Feature\NavigationMenuTest
 */
abstract class BaseNavigationMenu
{
    /** @var \Illuminate\Support\Collection<\Hyde\Framework\Features\Navigation\NavItem> */
    public Collection $items;

    final protected function __construct()
    {
        $this->items = new Collection();
    }

    public static function create(): static
    {
        $menu = new static();

        $menu->generate();
        $menu->removeDuplicateItems();
        $menu->sortByPriority();

        return $menu;
    }

    protected function generate(): void
    {
        Routes::each(function (Route $route): void {

The method each() does not exist on Hyde\Foundation\Facades\Routes. Since you implemented __callStatic, consider adding a @method annotation.

            if ($this->canAddRoute($route)) {
                $this->items->put($route->getRouteKey(), NavItem::fromRoute($route));

$route->getRouteKey() of type string is incompatible with the type Illuminate\Support\TKey expected by parameter $key of Illuminate\Support\Collection::put().

            }
        });

        collect(Config::getArray('hyde.navigation.custom', []))->each(function (NavItem $item): void {

Hyde\Facades\Config::getArray('hyde.navigation.custom', array()) of type array is incompatible with the type Illuminate\Contracts\Support\Arrayable expected by parameter $value of collect().

            // Since these were added explicitly by the user, we can assume they should always be shown
            $this->items->push($item);
        });
    }

    protected function canAddRoute(Route $route): bool
    {
        return $route->getPage()->showInNavigation();
    }

    protected function removeDuplicateItems(): void
    {
        $this->items = $this->items->unique(function (NavItem $item): string {
            // Filter using a combination of the group and label to allow duplicate labels in different groups
            return $item->getGroup().$item->label;
        });
    }

    protected function sortByPriority(): void
    {
        $this->items = $this->items->sortBy('priority')->values();
    }
}