CreateRole::handle() - Code Metrics - Inspection of "Merge pull request #862 from drbyte/cli-create-per..." - spatie/laravel-permission - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 63ef87...71e28f )

by Chris

created 2018-09-06 03:52 UTC

CreateRole::handle() A

↳ Parent: CreateRole

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	10
rs	9.9332
c	0
b	0
f	0

<?php

namespace Spatie\Permission\Commands;

use Illuminate\Console\Command;
use Spatie\Permission\Contracts\Role as RoleContract;
use Spatie\Permission\Contracts\Permission as PermissionContract;

class CreateRole extends Command
{
    protected $signature = 'permission:create-role
        {name : The name of the role}
        {guard? : The name of the guard}
        {permissions? : A list of permissions to assign to the role, separated by | }';

    protected $description = 'Create a role';

    public function handle()
    {
        $roleClass = app(RoleContract::class);

        $role = $roleClass::findOrCreate($this->argument('name'), $this->argument('guard'));

        $role->givePermissionTo($this->makePermissions($this->argument('permissions')));

        $this->info("Role `{$role->name}` created");
    }

    protected function makePermissions($string = null)
    {
        $permissionClass = app(PermissionContract::class);

        $permissions = explode('|', $string);

        $models = [];

        foreach ($permissions as $permission) {
            $models[] = $permissionClass::findOrCreate(trim($permission), $this->argument('guard'));
        }

        return collect($models);
    }
}


1			<?php
2
3			namespace Spatie\Permission\Commands;
4
5			use Illuminate\Console\Command;
6			use Spatie\Permission\Contracts\Role as RoleContract;
7			use Spatie\Permission\Contracts\Permission as PermissionContract;
8
9			class CreateRole extends Command
10			{
11			protected $signature = 'permission:create-role
12			{name : The name of the role}
13			{guard? : The name of the guard}
14			{permissions? : A list of permissions to assign to the role, separated by \| }';
15
16			protected $description = 'Create a role';
17
18			public function handle()
19			{
20			$roleClass = app(RoleContract::class);
21
22			$role = $roleClass::findOrCreate($this->argument('name'), $this->argument('guard'));
23
24			$role->givePermissionTo($this->makePermissions($this->argument('permissions')));
25
26			$this->info("Role `{$role->name}` created");
27			}
28
29			protected function makePermissions($string = null)
30			{
31			$permissionClass = app(PermissionContract::class);
32
33			$permissions = explode('\|', $string);
34
35			$models = [];
36
37			foreach ($permissions as $permission) {
38			$models[] = $permissionClass::findOrCreate(trim($permission), $this->argument('guard'));
39			}
40
41			return collect($models);
42			}
43			}
44

spatie / laravel-permission

Push — master ( 63ef87...71e28f )

CreateRole::handle() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like