Merge pull request 'use the DateTime object when comparing dates in ExhcnageRateCalculator' (!49) from bugfix/fix-current-exchange-rate-calculation into master
All checks were successful
rvr-nextgen/pipeline/head This commit looks good
All checks were successful
rvr-nextgen/pipeline/head This commit looks good
Reviewed-on: #49
This commit is contained in:
commit
361671a362
@ -41,7 +41,7 @@ class ExchangeRateCalculator
|
|||||||
|
|
||||||
$currentExchangeRate = null;
|
$currentExchangeRate = null;
|
||||||
foreach ($this->exchangeRates[$currency->getId()] as $exchangeRate) {
|
foreach ($this->exchangeRates[$currency->getId()] as $exchangeRate) {
|
||||||
if ($exchangeRate->getValidFrom() > $time) {
|
if ($exchangeRate->getValidFromDate() > $time) {
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
$currentExchangeRate = $exchangeRate;
|
$currentExchangeRate = $exchangeRate;
|
||||||
|
Loading…
Reference in New Issue
Block a user