Bytesfield\KeyManager\Commands\SuspendClientCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	47
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	5	5	1
A	handle()	8	8	2

<?php

namespace Bytesfield\KeyManager\Commands;

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

class SuspendClientCommand extends Command

This class seems to be duplicated in your project.

{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'client:suspend {clientId}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Command to suspend a client';

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

    /**
     * @param KeyManagerInterface $manager
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(KeyManagerInterface $manager)
    {
        $this->manager = $manager;
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return int
     */
    public function handle()
    {
        $key = $this->manager->suspendClient($this->argument('clientId'));

$this->argument('clientId') is of type string|array|null, but the function expects a integer.

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

        $this->info($key->getData()->message);
    }
}