BaseCommand::setRoles() - Code Metrics - Inspection of "Merge pull request #13 from alsar/configupdate" - matejvelikonja/LabbyBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6f37d5...aba46f )

by Matej

created 2016-03-21 14:26 UTC

BaseCommand::setRoles() A

↳ Parent: BaseCommand

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
c	3
b	0
f	0
dl	0
loc	6
ccs	3
cts	3
cp	1
rs	9.4285
cc	1
eloc	3
nc	1
nop	1
crap	1

<?php

namespace Velikonja\LabbyBundle\Command;

use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class BaseCommand
 *
 * @package Velikonja\LabbyBundle\Command
 * @author  Matej Velikonja <[email protected]>
 */
abstract class BaseCommand extends ContainerAwareCommand
{
    const ROLE_LOCAL  = 'local';
    const ROLE_REMOTE = 'remote';

    /**
     * @var array
     */
    private $roles = array();

    /**
     * Sets roles that are allowed to execute command.
     *
     * @param array $roles
     *
     * @return $this
     */
    public function setRoles(array $roles)
    {
        $this->roles = $roles;

        return $this;
    }

    public function run(InputInterface $input, OutputInterface $output)
    {
        $this->checkRoles();

        return parent::run($input, $output);
    }

    /**
     * TODO: check if this can be moved to event
     *
     * @throws \Exception
     *
     * @return bool
     */
    private function checkRoles()
    {
        $appRoles = $this->getContainer()->getParameter('velikonja_labby.config.roles');

        foreach ($appRoles as $appRole) {
            if (in_array($appRole, $this->roles)) {
                return true;
            }
        }

        throw new \Exception(sprintf(
            'App role ( %s ) is not allowed. Allowed roles: ( %s ).',
            implode(', ', $appRoles),
            implode(', ', $this->roles)
        ));
    }
}


1		<?php
2
3		namespace Velikonja\LabbyBundle\Command;
4
5		use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
6		use Symfony\Component\Console\Input\InputInterface;
7		use Symfony\Component\Console\Output\OutputInterface;
8
9		/**
10		* Class BaseCommand
11		*
12		* @package Velikonja\LabbyBundle\Command
13		* @author Matej Velikonja <[email protected]>
14		*/
15		abstract class BaseCommand extends ContainerAwareCommand
16		{
17		const ROLE_LOCAL = 'local';
18		const ROLE_REMOTE = 'remote';
19
20		/**
21		* @var array
22		*/
23		private $roles = array();
24
25		/**
26		* Sets roles that are allowed to execute command.
27		*
28		* @param array $roles
29		*
30		* @return $this
31		*/
32	10	public function setRoles(array $roles)
33		{
34	10	$this->roles = $roles;
35
36	10	return $this;
37		}
38
39	10	public function run(InputInterface $input, OutputInterface $output)
40		{
41	10	$this->checkRoles();
42
43	10	return parent::run($input, $output);
44		}
45
46		/**
47		* TODO: check if this can be moved to event
48		*
49		* @throws \Exception
50		*
51		* @return bool
52		*/
53	10	private function checkRoles()
54		{
55	10	$appRoles = $this->getContainer()->getParameter('velikonja_labby.config.roles');
56
57	10	foreach ($appRoles as $appRole) {
58	10	if (in_array($appRole, $this->roles)) {
59	10	return true;
60		}
61		}
62
63		throw new \Exception(sprintf(
64		'App role ( %s ) is not allowed. Allowed roles: ( %s ).',
65		implode(', ', $appRoles),
66		implode(', ', $this->roles)
67		));
68		}
69		}
70

matejvelikonja / LabbyBundle

Push — master ( 6f37d5...aba46f )

BaseCommand::setRoles() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like