FeatureSecurity::isGranted() - Code Metrics - Inspection of "Move status check before token extraction on empty..." - adespresso/FeatureBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#46)

by Emanuele

created 2019-01-18 09:38 UTC

FeatureSecurity::isGranted() B

↳ Parent: FeatureSecurity

Complexity

Conditions	9
Paths	10

Size

Total Lines	29
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	9.7971

Importance

Changes

Metric	Value
cc	9
eloc	13
nc	10
nop	1
dl	0
loc	29
ccs	11
cts	14
cp	0.7856
crap	9.7971
rs	8.0555
c	0
b	0
f	0

<?php

namespace Ae\FeatureBundle\Security;

use Ae\FeatureBundle\Entity\Feature;
use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;

/**
 * Controls access to a Feature.
 *
 * @author Carlo Forghieri <[email protected]>
 */
class FeatureSecurity
{
    /**
     * @param AuthorizationCheckerInterface|null
     */
    protected $context;

    /**
     * @param AuthorizationCheckerInterface $context
     */
    public function __construct(AuthorizationCheckerInterface $context = null)
    {
        $this->context = $context;
    }

    /**
     * @param Feature $feature
     *
     * @return bool
     */
    public function isGranted(Feature $feature)
    {
        // feature is enabled without required roles
        // there's no need to check on user roles
        if (empty($feature->getRole()) && empty($feature->getParentRole())) {
            return $feature->isEnabled();
        }

        if (null === $this->context) {
            return false;
        }

        if (!$feature->isEnabled()) {
            return false;
        }

        if ($feature->getRole()) {
            if (!$this->context->isGranted($feature->getRole())) {
                return false;
            }
        }

        if ('' !== trim($feature->getParentRole())) {
            if (!$this->context->isGranted($feature->getParentRole())) {
                return false;
            }
        }

        return true;
    }
}


1		<?php
2
3		namespace Ae\FeatureBundle\Security;
4
5		use Ae\FeatureBundle\Entity\Feature;
6		use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
7
8		/**
9		* Controls access to a Feature.
10		*
11		* @author Carlo Forghieri <[email protected]>
12		*/
13		class FeatureSecurity
14		{
15		/**
16		* @param AuthorizationCheckerInterface\|null
17		*/
18		protected $context;
19
20		/**
21		* @param AuthorizationCheckerInterface $context
22		*/
23	12	public function __construct(AuthorizationCheckerInterface $context = null)
24		{
25	12	$this->context = $context;
26	12	}
27
28		/**
29		* @param Feature $feature
30		*
31		* @return bool
32		*/
33	12	public function isGranted(Feature $feature)
34		{
35		// feature is enabled without required roles
36		// there's no need to check on user roles
37	12	if (empty($feature->getRole()) && empty($feature->getParentRole())) {
38	8	return $feature->isEnabled();
39		}
40
41	4	if (null === $this->context) {
42		return false;
43		}
44
45	4	if (!$feature->isEnabled()) {
46		return false;
47		}
48
49	4	if ($feature->getRole()) {
50	4	if (!$this->context->isGranted($feature->getRole())) {
51		return false;
52		}
53		}
54
55	4	if ('' !== trim($feature->getParentRole())) {
56	2	if (!$this->context->isGranted($feature->getParentRole())) {
57	2	return false;
58		}
59		}
60
61	2	return true;
62		}
63		}
64

adespresso / FeatureBundle

Pull Request — master (#46)

FeatureSecurity::isGranted() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like