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

Complexity

Conditions	1
Paths	1

Size

Total Lines	23
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	6
Bugs	1	Features	0

<?php
    public function handle(Currencylayer $currencylayer): void
    {
        /**
         * @var array|float $rates
         */
        $rates = $currencylayer->rate(
            /**
             * @var string $source
             */
            $source = $this->argument('source'),

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

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

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

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

        $this->output(
            $date.' '.$source.' rates',
            $this->prepareRows($currencies, $rates)
        );