MultiStrategy::__construct() - Code Metrics - Inspection of "Authentication Strategy Multi, configurable" - libgraviton/graviton - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#619)

unknown

created 2017-10-16 12:17 UTC

MultiStrategy::__construct() A

↳ Parent: MultiStrategy

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

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

<?php
/**
 * strategy combining a set of strategies to be applied
 */

namespace Graviton\SecurityBundle\Authentication\Strategies;

use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Security\Core\Role\Role;

/**
 * Class MultiStrategy
 *
 * @package Graviton\SecurityBundle\Authentication\Strategies
 * @author   List of contributors <https://github.com/libgraviton/graviton/graphs/contributors>
 * @license  http://opensource.org/licenses/gpl-license.php GNU Public License
 * @link     http://swisscom.ch
 */
class MultiStrategy implements StrategyInterface
{
    /** @var StrategyInterface[]  */
    private $strategies = [];

    /** @var Role[] */
    private $roles = [];

    /**
     * MultiStrategy add.
     *
     * @param StrategyInterface $strategy strategy to be applied.
     * @return void
     */
    public function addStrategy(StrategyInterface $strategy)
    {
        $this->strategies[] = $strategy;
    }

    /**
     * Applies the defined strategies on the provided request.
     *
     * @param Request $request request to handle
     *
     * @return string

     */
    public function apply(Request $request)
    {
        foreach ($this->strategies as $strategy) {
            $name = $strategy->apply($request);
            if ($strategy->stopPropagation()) {
                $this->roles = $strategy->getRoles();
                return $name;
            }
        }

        return false;
    }

    /**
     * Decider to stop other strategies running after from being considered.
     *
     * @return boolean
     */
    public function stopPropagation()
    {
        return false;
    }

    /**
     * Provides the list of registered roles.
     *
     * @return Role[]
     */
    public function getRoles()
    {
        return array_unique($this->roles);
    }
}


1		<?php
2		/**
3		* strategy combining a set of strategies to be applied
4		*/
5
6		namespace Graviton\SecurityBundle\Authentication\Strategies;
7
8		use Symfony\Component\HttpFoundation\Request;
9		use Symfony\Component\Security\Core\Role\Role;
10
11		/**
12		* Class MultiStrategy
13		*
14		* @package Graviton\SecurityBundle\Authentication\Strategies
15		* @author List of contributors <https://github.com/libgraviton/graviton/graphs/contributors>
16		* @license http://opensource.org/licenses/gpl-license.php GNU Public License
17		* @link http://swisscom.ch
18		*/
19		class MultiStrategy implements StrategyInterface
20		{
21		/** @var StrategyInterface[] */
22		private $strategies = [];
23
24		/** @var Role[] */
25		private $roles = [];
26
27		/**
28		* MultiStrategy add.
29		*
30		* @param StrategyInterface $strategy strategy to be applied.
31		* @return void
32		*/
33	2	public function addStrategy(StrategyInterface $strategy)
34		{
35	2	$this->strategies[] = $strategy;
36	2	}
37
38		/**
39		* Applies the defined strategies on the provided request.
40		*
41		* @param Request $request request to handle
42		*
43		* @return string
		0 ignored issues – show Documentation introduced 2017-10-12 15:04 UTC by Report Bug Copy Issue Report Should the return type not be `string\|false`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
44		*/
45	2	public function apply(Request $request)
46		{
47	2	foreach ($this->strategies as $strategy) {
48	2	$name = $strategy->apply($request);
49	2	if ($strategy->stopPropagation()) {
50	2	$this->roles = $strategy->getRoles();
51	2	return $name;
52	1	}
53	1	}
54
55		return false;
56		}
57
58		/**
59		* Decider to stop other strategies running after from being considered.
60		*
61		* @return boolean
62		*/
63		public function stopPropagation()
64		{
65		return false;
66		}
67
68		/**
69		* Provides the list of registered roles.
70		*
71		* @return Role[]
72		*/
73	2	public function getRoles()
74		{
75	2	return array_unique($this->roles);
76		}
77		}
78

libgraviton / graviton

Pull Request — develop (#619)

MultiStrategy::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like