ClientScopeCommand::getArgOrGetUpset() - Code Metrics - Inspection of "remove example api call code from coverage and tes..." - delboy1978uk/oauth2-api - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( a54619...073016 )

by Derek Stephen

created 2018-12-08 01:04 UTC

ClientScopeCommand::getArgOrGetUpset() A

↳ Parent: ClientScopeCommand

Complexity

Conditions	2
Paths	2

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

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

<?php

namespace OAuth\Command;

use Exception;
use OAuth\Client;
use OAuth\Repository\ScopeRepository;
use OAuth\Scope;
use OAuth\Service\ClientService;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Helper\QuestionHelper;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Question\ChoiceQuestion;
use Symfony\Component\Console\Question\ConfirmationQuestion;
use Symfony\Component\Console\Question\Question;

/**
 * Class ClientCommand
 * @package OAuth\Command
 */
class ClientScopeCommand extends Command
{
    /**
     * @var ClientService $clientService
     */
    private $clientService;

    /**
     * @var ScopeRepository $scopeRepository
     */
    private $scopeRepository;

    /** @var QuestionHelper $helper */
    private $helper;


    public function __construct(ClientService $clientService, ScopeRepository $scopeRepository)
    {
        $this->clientService = $clientService;
        $this->scopeRepository = $scopeRepository;
        parent::__construct('client:scope');
        $this->addArgument('operation', InputArgument::REQUIRED, 'list, add, or remove.');
        $this->addArgument('client', InputArgument::OPTIONAL, 'The client identifier.');
        $this->addArgument('scope', InputArgument::OPTIONAL, 'The scope name when adding or removing.');
    }

    /**
     * configure options
     */
    protected function configure()
    {
        $this->setDescription('Add, remove, or list scopes for each client.');
        $this->setHelp('Client scope administration');
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     * @return int|void|null
     * @throws Exception
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $output->writeln('Client scope administration');
        $operation = $input->getArgument('operation');
        switch ($operation) {
            case 'list';
                $this->listScopes($input, $output);
                break;
            case 'add';
                $this->addScope($input, $output);
                break;
            case 'remove';
                $this->removeScope($input, $output);
                break;
        }
    }

    /**
     * @param string $argName
     * @param string $value
     * @throws Exception
     */
    private function getArgOrGetUpset(InputInterface $input, string $argName)
    {
        $value = $input->getArgument($argName);
        if (!$value) {
            throw new Exception('No ' . $argName . ' provided');
        }
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     */
    private function listScopes(InputInterface $input, OutputInterface $output)
    private function listScopes(/** @scrutinizer ignore-unused */ InputInterface $input, OutputInterface $output)
    {
        $output->writeln('List scopes for client');
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     * @throws Exception
     */
    private function addScope(InputInterface $input, OutputInterface $output)
    {
        $client = $this->getArgOrGetUpset($input, 'client');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        $scope = $this->getArgOrGetUpset($input, 'scope');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        $output->writeln('Scope '. $scope . ', client ' . $client);
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     */
    private function removeScope(InputInterface $input, OutputInterface $output)
    private function removeScope(/** @scrutinizer ignore-unused */ InputInterface $input, OutputInterface $output)
    {
        $output->writeln('Remove scope for client');
    }
}

delboy1978uk / oauth2-api

Push — master ( a54619...073016 )

ClientScopeCommand::getArgOrGetUpset() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like