Merge #6106

6106: bug(Fix Display Decimal point) r=jniles a=lomamech - Removal of the call of an unnecessary and undefined parameter in helpers finance in function currency() closes #6104 Co-authored-by: lomamech <lomamech@gmail.com>
IMA-WorldHealth · Nov 29, 2021 · ff79a25 · ff79a25
2 parents bfdfb5e + ce2319b
commit ff79a25
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/client/src/modules/reports/generate/account_report/account_report.config.js b/client/src/modules/reports/generate/account_report/account_report.config.js
@@ -8,7 +8,7 @@ AccountReportConfigController.$inject = [
 
 function AccountReportConfigController(
   $sce, Notify, SavedReports, AppCache, reportData, $state,
-  Moment, Session
+  Moment, Session,
 ) {
   const vm = this;
   const cache = new AppCache('configure_account_report');

diff --git a/server/controllers/finance/reports/reportAccounts/report.handlebars b/server/controllers/finance/reports/reportAccounts/report.handlebars
@@ -64,7 +64,6 @@
           </th>
 
           <th></th>
-
           <th class="text-right">
             {{debcred header.exchangedBalance params.currency_id }}
           </th>

diff --git a/server/lib/template/helpers/finance.js b/server/lib/template/helpers/finance.js
@@ -36,7 +36,9 @@ function currency(value = 0, currencyId, digit) {
   } else {
     // usage with custom digits for precision
     const format = JSON.parse(JSON.stringify(fmt));
-    format.precision = digit;
+
+    if (digit !== undefined) { format.precision = digit; }
+
     output = new Handlebars.SafeString(accountingjs.formatMoney(value, format));
   }