Thinktomorrow\Chief\Management\Nomadic::nomadicCan()

Complexity

Conditions	5
Paths	3

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function nomadicCan($verb)
    {
        if ($this->indexCollection()->count() > 0 && in_array($verb, ['create', 'store'])) {

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

            throw NotAllowedManagerRoute::create($this);

$this of type Thinktomorrow\Chief\Management\Nomadic is incompatible with the type Thinktomorrow\Chief\Management\Manager expected by parameter $manager of Thinktomorrow\Chief\Management\Exceptions\NotAllowedManagerRoute::create().

        }

        if (! auth()->guard('chief')->user()->hasRole('developer') && in_array($verb, ['create', 'store', 'delete'])) {

The method hasRole() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

            throw NotAllowedManagerRoute::notAllowedVerb($verb, $this);

$this of type Thinktomorrow\Chief\Management\Nomadic is incompatible with the type Thinktomorrow\Chief\Management\Manager expected by parameter $manager of Thinktomorrow\Chief\Management\Exceptions\NotAllowedManagerRoute::notAllowedVerb().

        }