ClientScopeCommand::fetchClient() - Code Metrics - Inspection of "Scope command add to client, list, remove" - delboy1978uk/oauth2-api - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( 073016...e095ce )

by Derek Stephen
created 2018-12-16 19:25 UTC
ClientScopeCommand::fetchClient() A

↳ Parent: ClientScopeCommand
Complexity

Conditions	1
Paths	1
Size

Total Lines	9
Code Lines	5
Duplication

Lines	0
Ratio	0 %
Code Coverage

Tests	0
CRAP Score	2
Importance

Changes
Metric	Value
cc	1
eloc	5
nc	1
nop	2
dl	0
loc	9
ccs	0
cts	6
cp	0
crap	2
rs	10
c	0
b	0
f	0
<?php

namespace OAuth\Command;

use Doctrine\Common\Collections\Collection;
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(' ');
        $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;
        }
        $output->writeln(' ');
    }

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

        return $value;
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     */
    private function listScopes(InputInterface $input, OutputInterface $output)
    {
        $clientId = $input->getArgument('client');

        $client = $this->fetchClient($output, $clientId);
        $client = $this->fetchClient($output, /** @scrutinizer ignore-type */ $clientId);
        if (!$client instanceof Client) {

            $output->writeln('No client found');
            return;
        }

        $scopes = $client->getScopes();
        $this->outputScopes($output, $client, $scopes);

        return;
    }

    /**
     * @param OutputInterface $output
     * @param Collection $scopes
     */
    private function outputScopes(OutputInterface $output, Client $client, Collection $scopes)
    {
        $output->writeln('Listing scopes for ' . $client->getName() . '.');

        if ($scopes->count()) {
            /** @var Scope $scope */
            foreach ($scopes as $scope) {
                $output->writeln(' - ' . $scope->getIdentifier());
            }
        } else {
            $output->writeln('No scopes set for ' . $client->getName() . '.');
        }
    }

    /**
     * @param string $id
     * @return Client
     */
    private function fetchClient(OutputInterface $output, string $id)
    {
        $output->writeln('Fetching client ' . $id .'...');
        /** @var Client $client */
        $client = $this->clientService
                    ->getClientRepository()
                    ->getClientEntity( $id, null, null, false);

        return $client;
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     * @throws Exception
     */
    private function addScope(InputInterface $input, OutputInterface $output)
    {
        $clientId = $input->getArgument('client');
        $scopeId = $this->getArgOrGetUpset($input, 'scope');

        $client = $this->fetchClient($output, $clientId);
        $client = $this->fetchClient($output, /** @scrutinizer ignore-type */ $clientId);
        if (!$client instanceof Client) {

            $output->writeln('No client found');
            return;
        }

        $scope = $this->scopeRepository->getScopeEntityByIdentifier($scopeId);
        $scope = $this->scopeRepository->getScopeEntityByIdentifier(/** @scrutinizer ignore-type */ $scopeId);
        if (!$scope instanceof Scope) {

            $output->writeln('No scope found.');
            return;
        }

        $output->writeln('Adding '. $scopeId . ' scope to ' . $client->getName() . '...');
        $output->writeln('Adding '. /** @scrutinizer ignore-type */ $scopeId . ' scope to ' . $client->getName() . '...');
        $client->getScopes()->add($scope);
        $this->clientService->getClientRepository()->save($client);
        $output->writeln($scopeId . ' scope added.');

    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     * @throws Exception
     */
    private function removeScope(InputInterface $input, OutputInterface $output)
    {
        $clientId = $input->getArgument('client');
        $scopeId = $this->getArgOrGetUpset($input, 'scope');

        $client = $this->fetchClient($output, $clientId);
        $client = $this->fetchClient($output, /** @scrutinizer ignore-type */ $clientId);
        if (!$client instanceof Client) {

            $output->writeln('No client found');
            return;
        }

        $scopes = $client->getScopes();
        /** @var Scope $scope */
        foreach ($scopes as $key => $scope) {
            if ($scope->getIdentifier() == $scopeId) {
                $scopes->remove($key);
                break;
            }
        }

        $this->clientService->getClientRepository()->save($client);
        $output->writeln($scopeId . ' scope removed.');
    }
}
delboy1978uk / oauth2-api

Push — master ( 073016...e095ce )

ClientScopeCommand::fetchClient() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like
