Orkhanahmadov\LaravelCurrencylayer\Commands\RateCommand

Complexity

Total Complexity

1

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes	8
Bugs	2	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	23	1

<?php

namespace Orkhanahmadov\LaravelCurrencylayer\Commands;

use Orkhanahmadov\LaravelCurrencylayer\Contracts\CurrencyService;

class RateCommand extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'currencylayer:rate 
                            {source : Source currency code} 
                            {date : Date for currency rates} 
                            {currencies* : Target currency codes}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Gets live rates for currencies';

    /**
     * Execute the console command.
     *
     * @param CurrencyService $currencyService
     */
    public function handle(CurrencyService $currencyService): void
    {
        /**
         * @var array|float
         */
        $rates = $currencyService->rate(
            /**
             * @var string
             */
            $source = $this->argument('source'),

It seems like $source = $this->argument('source') can also be of type string[]; however, parameter $source of Orkhanahmadov\LaravelCurrencylayer\Contracts\CurrencyService::rate() does only seem to accept Orkhanahmadov\LaravelCurrencylayer\Models\Currency|string, maybe add an additional type check?

            /**
             * @var string
             */
            $date = $this->argument('date'),

It seems like $date = $this->argument('date') can also be of type string[]; however, parameter $date of Orkhanahmadov\LaravelCurrencylayer\Contracts\CurrencyService::rate() does only seem to accept Carbon\Carbon|string, maybe add an additional type check?

            /**
             * @var array
             */
            $currencies = $this->argument('currencies')
        );

        $this->output(
            $date.' '.$source.' rates',

Are you sure $date of type null|string|string[] can be used in concatenation?

Are you sure $source of type null|string|string[] can be used in concatenation?

            $this->prepareRows($currencies, $rates)

It seems like $currencies can also be of type null and string; however, parameter $currencies of Orkhanahmadov\LaravelCurrencylayer\Commands\Command::prepareRows() does only seem to accept array, maybe add an additional type check?

        );
    }
}