App\Services\ExchangeRateUpdater

Complexity

Total Complexity

2

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	update()	0	5	1
A	__construct()	0	4	1

<?php


namespace App\Services;


use App\Entity\PriceInformations\Currency;
use Brick\Math\BigDecimal;
use Swap\Swap;

class ExchangeRateUpdater
{
    private $base_currency;
    private $swap;

    public function __construct(string $base_currency, Swap $swap)
    {
        $this->base_currency = $base_currency;
        $this->swap = $swap;
    }

    /**
     * Updates the exchange rate of the given currency using the globally configured providers.
     * @param  Currency  $currency
     * @return Currency
     */
    public function update(Currency $currency): Currency
    {
        $rate = $this->swap->latest($currency->getIsoCode().'/'.$this->base_currency);
        $currency->setExchangeRate(BigDecimal::of($rate->getValue()));

It seems like Brick\Math\BigDecimal::of($rate->getValue()) can also be of type Brick\Math\BigInteger and Brick\Math\BigRational; however, parameter $exchange_rate of App\Entity\PriceInformations\Currency::setExchangeRate() does only seem to accept Brick\Math\BigDecimal|null, maybe add an additional type check?

        return $currency;
    }

}