AdminGroups::isAdmin() - Code Metrics - Inspection of "Widget" - eXpansionPluginPack/eXpansion2 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#144)

unknown

created 2017-12-01 22:35 UTC

AdminGroups::isAdmin() A

↳ Parent: AdminGroups

Complexity

Conditions	2
Paths	2

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
ccs	0
cts	2
cp	0
cc	2
eloc	2
nc	2
nop	1
crap	6

<?php

namespace eXpansion\Framework\AdminGroups\Helpers;

use eXpansion\Framework\AdminGroups\Exceptions\UnknownGroupException;
use eXpansion\Framework\AdminGroups\Services\AdminGroupConfiguration;
use eXpansion\Framework\Core\Model\UserGroups\Group;
use eXpansion\Framework\Core\Plugins\UserGroups\Factory;
use eXpansion\Framework\Core\Storage\Data\Player;

/**
 * Class AdminGroupConfiguration
 *
 * @package eXpansion\Bundle\AdminGroupConfiguration\Helpers;
 * @author oliver de Cramer <[email protected]>
 */
class AdminGroups
{
    /** @var  AdminGroupConfiguration */
    protected $adminGroupConfiguration;

    /** @var  Factory */
    protected $userGroupFactory;

    /**
     * GroupsPlugin constructor.
     *
     * @param AdminGroupConfiguration $adminGroupConfiguration
     * @param Factory $userGroupFactory
     */
    public function __construct(
        AdminGroupConfiguration $adminGroupConfiguration,
        Factory $userGroupFactory
    ) {
        $this->adminGroupConfiguration = $adminGroupConfiguration;
        $this->userGroupFactory = $userGroupFactory;
    }

    /**
     * Get list of all user groups.
     * Can be useful for creating group based GUI widgets.
     *
     * @return Group[]
     */
    public function getUserGroups()
    {
        $groups = [];
        foreach ($this->adminGroupConfiguration->getGroups() as $groupName) {
            $groups[] = $this->getUserGroup("$groupName");
        }

        return $groups;
    }

    /**
     * Get the group in which a user is.
     * This is useful for gui actions.
     *
     * @param string $login
     *
     * @return Group
     */
    public function getLoginUserGroups($login)
    {
        $groupName = $this->adminGroupConfiguration->getLoginGroupName($login);
        if (empty($groupName)) {
            $groupName = 'guest';
        }

        return $this->getUserGroup("$groupName");
    }

    /**
     * Get (or create a new) admin user group
     *
     * @param string $groupName
     *
     * @return Group
     */
    protected function getUserGroup($groupName)
    {
        $groupName = "admin:$groupName";

        $group = $this->userGroupFactory->getGroup($groupName);
        if (!$group) {
            $this->userGroupFactory->create($groupName);
            $group = $this->userGroupFactory->getGroup($groupName);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        }

        return $group;
    }

    /**
     * Checks if group, a login or a player has a certain permission or not.
     *
     * @param string|Group|Player $recipient
     * @param string $permission The permission to check for.
     *
     * @return bool
     */
    public function hasPermission($recipient, $permission)
    {

        if ($recipient instanceof Group) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
            $check = true;
            foreach ($recipient->getLogins() as $login) {
                if ($this->hasLoginPermission($login, $permission) === false) {
                    $check = false;
                }
            }

            return $check;
        } else {
            return $this->hasLoginPermission($recipient, $permission);
        }
    }

    protected function hasLoginPermission($login, $permission)
    {
        if ($login instanceof Player) {
            $login = $login->getLogin();
        }

        return $this->adminGroupConfiguration->hasPermission($login, $permission);
    }


    /**
     * Check if a group has a certain permission or not.
     *
     * @param string $groupName The name of the group to check permissions for.
     * @param string $permission The permission to check for.
     *
     * @return bool
     * @throws UnknownGroupException Thrown when group isn't an admin group.
     */
    public function hasGroupPermission($groupName, $permission)
    {

        if (strpos($groupName, 'admin:') === 0) {
            $groupName = str_replace("admin:", '', $groupName);
        }

        $logins = $this->adminGroupConfiguration->getGroupLogins($groupName);

        if (!empty($logins)) {
            return $this->hasPermission($logins[0], $permission);
        }

        if (is_null($logins)) {
            throw new UnknownGroupException("'$groupName' admin group does not exist.");
        }

        return false;
    }

    /**
     * @param string $groupName
     * @return string
     */
    public function getGroupLabel($groupName)
    {
        if (strpos($groupName, 'admin:') === 0) {
            $groupName = str_replace("admin:", '', $groupName);
        }

        return $this->adminGroupConfiguration->getGroupLabel($groupName);

    }

    /**
     * gets if the player is admin
     *
     * @param string $login
     * @return bool
     */
    public function isAdmin($login)
    {
        return ($this->getLoginUserGroups($login)->getName() !== "admin:guest") ? true : false;
    }

}


1		<?php
2
3		namespace eXpansion\Framework\AdminGroups\Helpers;
4
5		use eXpansion\Framework\AdminGroups\Exceptions\UnknownGroupException;
6		use eXpansion\Framework\AdminGroups\Services\AdminGroupConfiguration;
7		use eXpansion\Framework\Core\Model\UserGroups\Group;
8		use eXpansion\Framework\Core\Plugins\UserGroups\Factory;
9		use eXpansion\Framework\Core\Storage\Data\Player;
10
11		/**
12		* Class AdminGroupConfiguration
13		*
14		* @package eXpansion\Bundle\AdminGroupConfiguration\Helpers;
15		* @author oliver de Cramer <[email protected]>
16		*/
17		class AdminGroups
18		{
19		/** @var AdminGroupConfiguration */
20		protected $adminGroupConfiguration;
21
22		/** @var Factory */
23		protected $userGroupFactory;
24
25		/**
26		* GroupsPlugin constructor.
27		*
28		* @param AdminGroupConfiguration $adminGroupConfiguration
29		* @param Factory $userGroupFactory
30		*/
31	6	public function __construct(
32		AdminGroupConfiguration $adminGroupConfiguration,
33		Factory $userGroupFactory
34		) {
35	6	$this->adminGroupConfiguration = $adminGroupConfiguration;
36	6	$this->userGroupFactory = $userGroupFactory;
37	6	}
38
39		/**
40		* Get list of all user groups.
41		* Can be useful for creating group based GUI widgets.
42		*
43		* @return Group[]
44		*/
45	1	public function getUserGroups()
46		{
47	1	$groups = [];
48	1	foreach ($this->adminGroupConfiguration->getGroups() as $groupName) {
49	1	$groups[] = $this->getUserGroup("$groupName");
50		}
51
52	1	return $groups;
53		}
54
55		/**
56		* Get the group in which a user is.
57		* This is useful for gui actions.
58		*
59		* @param string $login
60		*
61		* @return Group
62		*/
63	1	public function getLoginUserGroups($login)
64		{
65	1	$groupName = $this->adminGroupConfiguration->getLoginGroupName($login);
66	1	if (empty($groupName)) {
67	1	$groupName = 'guest';
68		}
69
70	1	return $this->getUserGroup("$groupName");
71		}
72
73		/**
74		* Get (or create a new) admin user group
75		*
76		* @param string $groupName
77		*
78		* @return Group
79		*/
80	2	protected function getUserGroup($groupName)
81		{
82	2	$groupName = "admin:$groupName";
83
84	2	$group = $this->userGroupFactory->getGroup($groupName);
85	2	if (!$group) {
86	2	$this->userGroupFactory->create($groupName);
87	2	$group = $this->userGroupFactory->getGroup($groupName);
		0 ignored issues – show Bug introduced 2017-05-13 11:19 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$group` is correct as `$this->userGroupFactory->getGroup($groupName)` (which targets `eXpansion\Framework\Core...ups\Factory::getGroup()`) seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
88		}
89
90	2	return $group;
91		}
92
93		/**
94		* Checks if group, a login or a player has a certain permission or not.
95		*
96		* @param string\|Group\|Player $recipient
97		* @param string $permission The permission to check for.
98		*
99		* @return bool
100		*/
101	3	public function hasPermission($recipient, $permission)
102		{
103
104	3	if ($recipient instanceof Group) {
		0 ignored issues – show Bug introduced 2017-11-25 14:38 UTC by Report Bug Copy Issue Report The class `eXpansion\Framework\Core\Model\UserGroups\Group` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
105		$check = true;
106		foreach ($recipient->getLogins() as $login) {
107		if ($this->hasLoginPermission($login, $permission) === false) {
108		$check = false;
109		}
110		}
111
112		return $check;
113		} else {
114	3	return $this->hasLoginPermission($recipient, $permission);
115		}
116		}
117
118	3	protected function hasLoginPermission($login, $permission)
119		{
120	3	if ($login instanceof Player) {
121		$login = $login->getLogin();
122		}
123
124	3	return $this->adminGroupConfiguration->hasPermission($login, $permission);
125		}
126
127
128		/**
129		* Check if a group has a certain permission or not.
130		*
131		* @param string $groupName The name of the group to check permissions for.
132		* @param string $permission The permission to check for.
133		*
134		* @return bool
135		* @throws UnknownGroupException Thrown when group isn't an admin group.
136		*/
137	2	public function hasGroupPermission($groupName, $permission)
138		{
139
140	2	if (strpos($groupName, 'admin:') === 0) {
141	1	$groupName = str_replace("admin:", '', $groupName);
142		}
143
144	2	$logins = $this->adminGroupConfiguration->getGroupLogins($groupName);
145
146	2	if (!empty($logins)) {
147	1	return $this->hasPermission($logins[0], $permission);
148		}
149
150	2	if (is_null($logins)) {
151	2	throw new UnknownGroupException("'$groupName' admin group does not exist.");
152		}
153
154	1	return false;
155		}
156
157		/**
158		* @param string $groupName
159		* @return string
160		*/
161		public function getGroupLabel($groupName)
162		{
163		if (strpos($groupName, 'admin:') === 0) {
164		$groupName = str_replace("admin:", '', $groupName);
165		}
166
167		return $this->adminGroupConfiguration->getGroupLabel($groupName);
168
169		}
170
171		/**
172		* gets if the player is admin
173		*
174		* @param string $login
175		* @return bool
176		*/
177		public function isAdmin($login)
178		{
179		return ($this->getLoginUserGroups($login)->getName() !== "admin:guest") ? true : false;
180		}
181
182		}
183

eXpansionPluginPack / eXpansion2

Pull Request — master (#144)

AdminGroups::isAdmin() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like