RoleModel::removeRole() - Code Metrics - Inspection of "removed roles manager from user" - omatech/mage-core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 880107...7e2e0f )

by Christian

created 2019-10-11 13:05 UTC

RoleModel::removeRole() A

↳ Parent: RoleModel

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	15
ccs	8
cts	8
cp	1
rs	9.7666
c	0
b	0
f	0
cc	2
nc	2
nop	2
crap	2

<?php

namespace Omatech\Mage\Core\Domains\Roles;

use Omatech\Mage\Core\Domains\Roles\Contracts\RoleInterface;
use Omatech\Mage\Core\Domains\Roles\Exceptions\RoleIsNotSavedException;

class RoleModel
{
    public function assignRole(array $currentRoles, RoleInterface $assignRole): array
    {
        if ($assignRole->getId() === null) {
            throw new RoleIsNotSavedException;
        }

        if (!in_array($assignRole, $currentRoles, true)) {
            $currentRoles[] = $assignRole;
        }

        return $currentRoles;
    }

    public function assignRoles(array $currentRoles, array $assignRoles): array
    {
        foreach ($assignRoles as $role) {
            if ($role instanceof RoleInterface) {
                $currentRoles = $this->assignRole($currentRoles, $role);
            }
        }

        return $currentRoles;
    }

    public function removeRole(array $currentRoles, RoleInterface $deleteRole): array
    {
        if ($deleteRole->getId() === null) {
            throw new RoleIsNotSavedException;
        }

        $currentRoles = array_values(array_filter(
            $currentRoles,
            static function ($currentRole) use ($deleteRole) {
                return $currentRole !== $deleteRole;
            }
        ));

        return $currentRoles;
    }

    public function removeRoles(array $currentRoles, array $deleteRoles): array
    {
        foreach ($deleteRoles as $role) {
            if ($role instanceof RoleInterface) {
                $currentRoles = $this->removeRole($currentRoles, $role);
            }
        }

        return $currentRoles;
    }
}


1		<?php
2
3		namespace Omatech\Mage\Core\Domains\Roles;
4
5		use Omatech\Mage\Core\Domains\Roles\Contracts\RoleInterface;
6		use Omatech\Mage\Core\Domains\Roles\Exceptions\RoleIsNotSavedException;
7
8		class RoleModel
9		{
10	7	public function assignRole(array $currentRoles, RoleInterface $assignRole): array
11		{
12	7	if ($assignRole->getId() === null) {
13	1	throw new RoleIsNotSavedException;
14		}
15
16	6	if (!in_array($assignRole, $currentRoles, true)) {
17	6	$currentRoles[] = $assignRole;
18		}
19
20	6	return $currentRoles;
21		}
22
23	7	public function assignRoles(array $currentRoles, array $assignRoles): array
24		{
25	7	foreach ($assignRoles as $role) {
26	6	if ($role instanceof RoleInterface) {
27	6	$currentRoles = $this->assignRole($currentRoles, $role);
28		}
29		}
30
31	6	return $currentRoles;
32		}
33
34	3	public function removeRole(array $currentRoles, RoleInterface $deleteRole): array
35		{
36	3	if ($deleteRole->getId() === null) {
37	1	throw new RoleIsNotSavedException;
38		}
39
40	2	$currentRoles = array_values(array_filter(
41	2	$currentRoles,
42		static function ($currentRole) use ($deleteRole) {
43	2	return $currentRole !== $deleteRole;
44	2	}
45		));
46
47	2	return $currentRoles;
48		}
49
50	2	public function removeRoles(array $currentRoles, array $deleteRoles): array
51		{
52	2	foreach ($deleteRoles as $role) {
53	2	if ($role instanceof RoleInterface) {
54	2	$currentRoles = $this->removeRole($currentRoles, $role);
55		}
56		}
57
58	1	return $currentRoles;
59		}
60		}
61

omatech / mage-core

Push — master ( 880107...7e2e0f )

RoleModel::removeRole() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like