Bytesfield\KeyManager\Commands\CreateClientCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	handle()	0	18	2

<?php

namespace Bytesfield\KeyManager\Commands;

use Bytesfield\KeyManager\KeyManagerInterface;
use Illuminate\Console\Command;

class CreateClientCommand extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'client:create {name} {type} {status=active}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Command to create client with API keys';

    /**
     * Create a new command instance.
     *
     * @return void
     */

    /**
     * The KeyManagerInterface.
     *
     * @var KeyManagerInterface
     */
    private $manager;

    /**
     * @param KeyManagerInterface $manager
     */
    public function __construct(KeyManagerInterface $manager)
    {
        $this->manager = $manager;

        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return void
     */
    public function handle(): void
    {
        $name = $this->argument('name');

        $type = $this->argument('type');

        $status = $this->argument('status');

        $client = $this->manager->createClient($name, $type, $status);

It seems like $name defined by $this->argument('name') on line 54 can also be of type array or null; however, Bytesfield\KeyManager\KeyManagerInterface::createClient() does only seem to accept string, maybe add an additional type check?

It seems like $type defined by $this->argument('type') on line 56 can also be of type array or null; however, Bytesfield\KeyManager\KeyManagerInterface::createClient() does only seem to accept string, maybe add an additional type check?

It seems like $status defined by $this->argument('status') on line 58 can also be of type array or null; however, Bytesfield\KeyManager\KeyManagerInterface::createClient() does only seem to accept string, maybe add an additional type check?

        $this->info($client->getData()->status == true ? 'Success' : 'Failed');

        $this->info($client->getData()->message);

        $key = $this->manager->getPrivateKey($client->getData()->data->id);

        $this->info('client key: '.$key->getData()->data->key);
    }
}