OAuth\Command\ClientCommand

Complexity

Total Complexity

5

Size/Duplication

Total Lines	78
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	8

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	configure()	0	6	1
B	execute()	0	42	3

<?php

namespace OAuth\Command;

use Del\Service\UserService;
use OAuth\Client;
use OAuth\Service\ClientService;
use OAuth\User;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Question\ConfirmationQuestion;
use Symfony\Component\Console\Question\Question;

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

    /**
     * @var UserService $userService
     */
    private $userService;

    public function __construct(ClientService $clientService, UserService $userService, ?string $name = null)
    {
        $this->clientService = $clientService;
        $this->userService = $userService;
        parent::__construct($name);
    }

    /**
     * configure options
     */
    protected function configure()
    {
        $this->setName('create-client');
        $this->setDescription('Creates a new user.');
        $this->setHelp('This command allows you to create a user...');
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     * @return int|null|Client
     * @throws \Doctrine\ORM\OptimisticLockException
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $output->writeln('Bone API client creator');
        $helper = $this->getHelper('question');

        $question = new Question('Enter the email of the account: ', false);
        $email = $helper->ask($input, $output, $question);

        $this->userService->setUserClass(User::class);
        /** @var User $user */
        $user = $this->userService->findUserByEmail($email);

        if (!$user) {
            $output->writeln('User not found. Exiting.');
            return null;
        }

        $question = new Question('Give a name for this application: ', false);
        $name = $helper->ask($input, $output, $question);

        $question = new Question('Give a redirect URI: ', false);
        $uri = $helper->ask($input, $output, $question);

        $question = new ConfirmationQuestion('Is this a phone app or JS client ? ', false);
        $public = $helper->ask($input, $output, $question);


        $client = new Client();
        $client->setName($name);
        $client->setIdentifier(md5($name));
        $client->setRedirectUri($uri);
        $client->setConfidential($public);
        $client->setUser($user);

        if ($public === false) {
            $this->clientService->generateSecret($client);
        }

        $this->clientService->getClientRepository()->create($client);

        return $client;

The return type of return $client; (OAuth\Client) is incompatible with the return type of the parent method Symfony\Component\Console\Command\Command::execute of type null|integer.

    }
}