Helldar\Roles\Traits\Commands

Complexity

Total Complexity

6

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	roleIsExist()	0	3	1
A	permissionIsDoesntExist()	0	3	1
A	roleIsDoesntExist()	0	3	1
A	permissionIsExist()	0	3	1
A	slug()	0	9	2

<?php

namespace Helldar\Roles\Traits;

use Helldar\Roles\Models\Permission;
use Helldar\Roles\Models\Role;
use Illuminate\Support\Str;

trait Commands
{
    use Searchable;

    protected $slug;

    protected function slug(): string
    {
        if (is_null($this->slug)) {
            $slug = $this->argument('slug');

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

            $this->slug = Str::slug($slug, '_');
        }

        return $this->slug;
    }

    protected function roleIsExist(): bool
    {
        return $this->searchExist(Role::class, $this->slug());
    }

    protected function roleIsDoesntExist(): bool
    {
        return ! $this->roleIsExist();
    }

    protected function permissionIsExist(): bool
    {
        return $this->searchExist(Permission::class, $this->slug());
    }

    protected function permissionIsDoesntExist(): bool
    {
        return ! $this->permissionIsExist();
    }
}