kenmirrors/grocy
1
0
Fork 0
mirror of https://github.com/grocy/grocy.git synced 2025-08-17 19:16:37 +00:00
Code Issues Projects Releases Wiki Activity

Fix rounding error on total value calculation (#1347)

Browse Source 
* Fix rounding error on total value calculation

* Remove unused 'amountSum' calculation
This commit is contained in:
Lauri Niskanen
2021-02-21 19:10:41 +02:00
committed by GitHub
parent 3d82c9abbd
commit 33dcd17fbd
1 changed files with 1 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
public/viewjs/stockoverview.js
View File

@@ -214,12 +214,6 @@ function RefreshStatistics()
Grocy.Api.Get('stock', Grocy.Api.Get('stock',
function(result) function(result)
{ {
var amountSum = 0;
result.forEach(element =>
{
amountSum += parseInt(element.amount);
});
if (!Grocy.FeatureFlags.GROCY_FEATURE_FLAG_STOCK_PRICE_TRACKING) if (!Grocy.FeatureFlags.GROCY_FEATURE_FLAG_STOCK_PRICE_TRACKING)
{ {
$("#info-current-stock").text(__n(result.length, '%s Product', '%s Products')); $("#info-current-stock").text(__n(result.length, '%s Product', '%s Products'));
@@ -229,7 +223,7 @@ function RefreshStatistics()
var valueSum = 0; var valueSum = 0;
result.forEach(element => result.forEach(element =>
{ {
valueSum += parseInt(element.value); valueSum += parseFloat(element.value);
}); });
$("#info-current-stock").text(__n(result.length, '%s Product', '%s Products') + ", " + __t('%s total value', valueSum.toLocaleString(undefined, { style: "currency", currency: Grocy.Currency }))); $("#info-current-stock").text(__n(result.length, '%s Product', '%s Products') + ", " + __t('%s total value', valueSum.toLocaleString(undefined, { style: "currency", currency: Grocy.Currency })));
} }