Orkhanahmadov\LaravelCurrencylayer\Commands\LiveCommand::handle()

Complexity

Conditions	1
Paths	1

Size

Total Lines	19
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(CurrencyService $currencyService): void
    {
        /**
         * @var array|float
         */
        $rates = $currencyService->live(
            /**
             * @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::live() does only seem to accept Orkhanahmadov\LaravelCurrencylayer\Models\Currency|string, maybe add an additional type check?

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

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

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?

        );