AuthorizationCheckerWrapper::__construct() - Code Metrics - Inspection of "Security role" - ynloultratech/graphql-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#7)

by Yonel Ceruto

created 2018-01-18 18:21 UTC

AuthorizationCheckerWrapper::__construct() A

↳ Parent: AuthorizationCheckerWrapper

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	3
rs	10
c	0
b	0
f	0
cc	1
eloc	1
nc	1
nop	1

<?php
/*******************************************************************************
 *  This file is part of the GraphQL Bundle package.
 *
 *  (c) YnloUltratech <[email protected]>
 *
 *  For the full copyright and license information, please view the LICENSE
 *  file that was distributed with this source code.
 ******************************************************************************/

namespace Ynlo\GraphQLBundle\Security\Authorization;

use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
use Symfony\Component\Security\Core\Exception\AuthenticationCredentialsNotFoundException;

class AuthorizationCheckerWrapper implements AuthorizationCheckerInterface
{
    private static $cache = [];

    private $authorizationChecker;

    public function __construct(AuthorizationCheckerInterface $authorizationChecker)
    {
        $this->authorizationChecker = $authorizationChecker;
    }

    /**
     * {@inheritdoc}
     */
    public function isGranted($attributes, $subject = null): bool
    {
        if (null !== $subject) {
            $key = md5(json_encode($attributes).'/'.spl_object_hash($subject));
        } else {
            $key = md5(json_encode($attributes));
        }

        if (array_key_exists($key, static::$cache)) {

            return static::$cache[$key];
        }

        try {
            $isGranted = $this->authorizationChecker->isGranted($attributes, $subject);
        } catch (AuthenticationCredentialsNotFoundException $e) {
            $isGranted = false;
        }

        return static::$cache[$key] = $isGranted;
    }
}


1			<?php
2			/*******************************************************************************
3			* This file is part of the GraphQL Bundle package.
4			*
5			* (c) YnloUltratech <[email protected]>
6			*
7			* For the full copyright and license information, please view the LICENSE
8			* file that was distributed with this source code.
9			******************************************************************************/
10
11			namespace Ynlo\GraphQLBundle\Security\Authorization;
12
13			use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
14			use Symfony\Component\Security\Core\Exception\AuthenticationCredentialsNotFoundException;
15
16			class AuthorizationCheckerWrapper implements AuthorizationCheckerInterface
17			{
18			private static $cache = [];
19
20			private $authorizationChecker;
21
22			public function __construct(AuthorizationCheckerInterface $authorizationChecker)
23			{
24			$this->authorizationChecker = $authorizationChecker;
25			}
26
27			/**
28			* {@inheritdoc}
29			*/
30			public function isGranted($attributes, $subject = null): bool
31			{
32			if (null !== $subject) {
33			$key = md5(json_encode($attributes).'/'.spl_object_hash($subject));
34			} else {
35			$key = md5(json_encode($attributes));
36			}
37
38			if (array_key_exists($key, static::$cache)) {
			0 ignored issues – show Bug introduced 2018-01-18 18:32 UTC by Report Bug Copy Issue Report Since `$cache` is declared private, accessing it with `static` will lead to errors in possible sub-classes; you can either use `self`, or increase the visibility of `$cache` to at least protected. Loading history...
39			return static::$cache[$key];
40			}
41
42			try {
43			$isGranted = $this->authorizationChecker->isGranted($attributes, $subject);
44			} catch (AuthenticationCredentialsNotFoundException $e) {
45			$isGranted = false;
46			}
47
48			return static::$cache[$key] = $isGranted;
49			}
50			}
51

ynloultratech / graphql-bundle

Pull Request — master (#7)

AuthorizationCheckerWrapper::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like