AuthorizationChecker::__construct() - Code Metrics - Inspection of "[WIP] V2.0.0" - LIN3S/WPFoundation - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#14)

by Gorka

created 2018-10-01 15:33 UTC

AuthorizationChecker::__construct() A

↳ Parent: AuthorizationChecker

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	15
rs	9.7666
c	0
b	0
f	0
cc	2
nc	2
nop	0

<?php

/*
 * This file is part of the WPFoundation library.
 *
 * Copyright (c) 2015-present LIN3S <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace LIN3S\WPFoundation;

/**
 * @author Beñat Espiña <[email protected]>
 */
final class AuthorizationChecker
{
    private const ROLE_ADMIN = 'administrator';
    private const ROLE_AUTHOR = 'author';
    private const ROLE_EDITOR = 'editor';

    private $user;

    public function __construct()
    {
        $this->user = wp_get_current_user();

        if ($this->isNotAdmin()) {
            add_filter('admin_menu', function () {

                remove_menu_page('profile.php');
                remove_menu_page('tools.php');
                remove_menu_page('options-general.php');
                remove_menu_page('edit.php');
                remove_menu_page('admin.php');
                remove_submenu_page('users.php', 'profile.php');
            });
        }
    }

    private function isNotAdmin() : bool
    {
        return !in_array(self::ROLE_ADMIN, $this->user->roles, true);
    }
}


1			<?php
2
3			/*
4			* This file is part of the WPFoundation library.
5			*
6			* Copyright (c) 2015-present LIN3S <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			declare(strict_types=1);
13
14			namespace LIN3S\WPFoundation;
15
16			/**
17			* @author Beñat Espiña <[email protected]>
18			*/
19			final class AuthorizationChecker
20			{
21			private const ROLE_ADMIN = 'administrator';
22			private const ROLE_AUTHOR = 'author';
23			private const ROLE_EDITOR = 'editor';
24
25			private $user;
26
27			public function __construct()
28			{
29			$this->user = wp_get_current_user();
30
31			if ($this->isNotAdmin()) {
32			add_filter('admin_menu', function () {
			0 ignored issues – show Unused Code introduced 2018-01-25 12:16 UTC by Report Bug Copy Issue Report The call to the function `add_filter()` seems unnecessary as the function has no side-effects. Loading history...
33			remove_menu_page('profile.php');
34			remove_menu_page('tools.php');
35			remove_menu_page('options-general.php');
36			remove_menu_page('edit.php');
37			remove_menu_page('admin.php');
38			remove_submenu_page('users.php', 'profile.php');
39			});
40			}
41			}
42
43			private function isNotAdmin() : bool
44			{
45			return !in_array(self::ROLE_ADMIN, $this->user->roles, true);
46			}
47			}
48

LIN3S / WPFoundation

Pull Request — master (#14)

AuthorizationChecker::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like