User::getRoles() - Code Metrics - Inspection of "[WIP] [SettingsBundle][UserBundle] introduce new b..." - ahilles107/web-publisher - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9c95f5...e428dc )

by Paweł

created 2017-04-05 09:00 UTC

User::getRoles() A

↳ Parent: User

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	11
rs	9.4285
c	0
b	0
f	0
ccs	0
cts	0
cp	0
cc	2
eloc	6
nc	2
nop	0
crap	6

<?php

declare(strict_types=1);

/*
 * This file is part of the Superdesk Web Publisher Core Bundle.
 *
 * Copyright 2016 Sourcefabric z.ú. and contributors.
 *
 * For the full copyright and license information, please see the
 * AUTHORS and LICENSE files distributed with this source code.
 *
 * @copyright 2016 Sourcefabric z.ú
 * @license http://www.superdesk.org/license
 */

namespace SWP\Bundle\CoreBundle\Model;

use SWP\Bundle\UserBundle\Model\User as BaseUser;
use SWP\Component\MultiTenancy\Model\TenantAwareTrait;

class User extends BaseUser implements UserInterface
{
    use TenantAwareTrait;

    /**
     * {@inheritdoc}
     */
    public function addRole($role)
    {
        $role = strtoupper($role);
        if ($role === static::ROLE_READER) {
            return $this;
        }

        if (!in_array($role, $this->roles, true)) {
            $this->roles[] = $role;
        }

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function getRoles()
    {
        $roles = $this->roles;
        foreach ($this->getGroups() as $group) {
            $roles = array_merge($roles, $group->getRoles());
        }
        // we need to make sure to have at least one role
        $roles[] = static::ROLE_READER;

        return array_unique($roles);
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		/*
6		* This file is part of the Superdesk Web Publisher Core Bundle.
7		*
8		* Copyright 2016 Sourcefabric z.ú. and contributors.
9		*
10		* For the full copyright and license information, please see the
11		* AUTHORS and LICENSE files distributed with this source code.
12		*
13		* @copyright 2016 Sourcefabric z.ú
14		* @license http://www.superdesk.org/license
15		*/
16
17		namespace SWP\Bundle\CoreBundle\Model;
18
19		use SWP\Bundle\UserBundle\Model\User as BaseUser;
20		use SWP\Component\MultiTenancy\Model\TenantAwareTrait;
21
22		class User extends BaseUser implements UserInterface
23		{
24		use TenantAwareTrait;
25
26		/**
27		* {@inheritdoc}
28		*/
29		public function addRole($role)
30	100	{
31		$role = strtoupper($role);
32	100	if ($role === static::ROLE_READER) {
33		return $this;
34	100	}
35	100
36		if (!in_array($role, $this->roles, true)) {
37		$this->roles[] = $role;
38		}
39
40		return $this;
41		}
42
43		/**
44		* {@inheritdoc}
45		*/
46		public function getRoles()
47		{
48		$roles = $this->roles;
49		foreach ($this->getGroups() as $group) {
50		$roles = array_merge($roles, $group->getRoles());
51		}
52		// we need to make sure to have at least one role
53		$roles[] = static::ROLE_READER;
54
55		return array_unique($roles);
56		}
57		}
58

ahilles107 / web-publisher

Push — master ( 9c95f5...e428dc )

User::getRoles() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like