Merge pull request #18206 from chamilwijesooriya/issue-1959

dev/core#1959 Brick\Math\Exception\RoundingNecessaryException
civicrm · Aug 21, 2020 · 482e99a · 482e99a
2 parents 5fbba0e + f1b7204
commit 482e99a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/CRM/Utils/Money.php b/CRM/Utils/Money.php
@@ -139,8 +139,9 @@ public static function getCurrencyPrecision($currency = NULL) {
    */
   public static function subtractCurrencies($leftOp, $rightOp, $currency) {
     if (is_numeric($leftOp) && is_numeric($rightOp)) {
-      $money = Money::of($leftOp, $currency, new DefaultContext(), RoundingMode::CEILING);
-      return $money->minus($rightOp)->getAmount()->toFloat();
+      $leftMoney = Money::of($leftOp, $currency, new DefaultContext(), RoundingMode::CEILING);
+      $rightMoney = Money::of($rightOp, $currency, new DefaultContext(), RoundingMode::CEILING);
+      return $leftMoney->minus($rightMoney)->getAmount()->toFloat();
     }
   }