irfaardy / lockout-account

src/Console/Commands/LockCommands.php

<?php

namespace Irfa\Lockout\Console\Commands;

use Illuminate\Console\Command;
use Irfa\Lockout\Func\Core;
use Symfony\Component\Console\Helper\Table;

class LockCommands extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'lockout:lock {username}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Lock Account';

    /**
     * Create a new command instance.
     *
     * @return void
     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle(Core $core)
    {
        $this->line('Locking '.$this->argument('username').'...');

Are you sure $this->argument('username') of type string|string[] can be used in concatenation?

        if($core->lock_account($this->argument('username'))!="error"){

It seems like $this->argument('username') can also be of type string[]; however, parameter $username of Irfa\Lockout\Func\Core::lock_account() does only seem to accept string, maybe add an additional type check?

            $table = new Table($this->output);
            $table->setRows([
                        ['<fg=green>'.$this->argument('username').' successfully locked.'],
                       
                    ]);
                        $table->render();
        } else{
                $this->line('<fg=red> Locking failed.');
        }
    }
    
}