Modules\Authorization\Database\Seeders\AuthorizationSeeder

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	run()	0	5	1
A	__construct()	0	3	1
A	createRoles()	0	4	2
A	createPermissions()	0	4	1

<?php

namespace Modules\Authorization\Database\Seeders;

use Illuminate\Database\Seeder;
use Modules\Authorization\Contracts\AuthorizationContract;
use Modules\Authorization\Managers\PermissionManager;
use Modules\Authorization\Managers\RoleManager;
use Modules\Authorization\Services\AuthorizationService;

class AuthorizationSeeder extends Seeder
{
    /**
     * @var AuthorizationService
     */
    protected $service;

    /**
     * AuthorizationSeeder constructor.
     *
     * @param $service
     */
    public function __construct(AuthorizationContract $service)
    {
        $this->service = $service;

$service is of type Modules\Authorization\Contracts\AuthorizationContract, but the property $service was declared to be of type Modules\Authorization\Services\AuthorizationService. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * Run the database seeds.
     *
     * @return void
     */
    public function run()
    {
        $this->service->clearPermissionCache();
        $this->createPermissions();
        $this->createRoles();
    }

    protected function createPermissions(): void
    {
        $permissions = PermissionManager::getAllPermissions();
        $this->service->createPermissions(collect($permissions)->flatten()->toArray());
    }

    protected function createRoles(): void
    {
        foreach (RoleManager::ROLES as $role) {
            $this->service->createRole(strtolower($role::getRoleName()), $role::getPermissions());
        }
    }
}