Thinktomorrow\Chief\App\View\Components\Nav

Complexity

Total Complexity

5

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	items()	0	12	3
A	render()	0	3	1
A	__construct()	0	4	1

<?php
declare(strict_types=1);

namespace Thinktomorrow\Chief\App\View\Components;

use Illuminate\Support\Collection;
use Illuminate\View\Component;
use Thinktomorrow\Chief\Admin\Nav\Nav as NavItems;

final class Nav extends Component
{
    public ?string $title;

    /** @var NavItems */
    private NavItems $nav;

    public function __construct(NavItems $nav, ?string $title = null)
    {
        $this->title = $title;
        $this->nav = $nav;
    }

    /**
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
     */
    public function render()
    {
        return view('chief::components.nav.index');
    }

    public function items(): Collection
    {
        // Attribute bag is filled at moment of method call (inside the component view)
        if ($this->attributes->has('untagged')) {
            return collect($this->nav->untagged()->all());

$this->nav->untagged()->all() of type array is incompatible with the type Illuminate\Contracts\Support\Arrayable expected by parameter $value of collect().

        }

        if ($this->attributes->has('tagged')) {
            return collect($this->nav->tagged(explode(',', $this->attributes->get('tagged')))->all());

It seems like $this->attributes->get('tagged') can also be of type null; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

        }

        return collect($this->nav->all());
    }
}