From b77d0f87b24d90d5e0d8da43aa11d08690880755 Mon Sep 17 00:00:00 2001
From: Matjaž Črnko <m.crnko@txt.si>
Date: Fri, 16 Jun 2023 11:34:23 +0200
Subject: fix: Modern sync Consumption, Volume Position Attribute formatter

---
 modern/src/common/components/PositionValue.js | 23 +++++++++++++++++++++--
 modern/src/common/util/formatter.js           |  2 ++
 2 files changed, 23 insertions(+), 2 deletions(-)

(limited to 'modern/src')

diff --git a/modern/src/common/components/PositionValue.js b/modern/src/common/components/PositionValue.js
index 9711a7bb..58841939 100644
--- a/modern/src/common/components/PositionValue.js
+++ b/modern/src/common/components/PositionValue.js
@@ -3,7 +3,21 @@ import { useSelector } from 'react-redux';
 import { Link } from '@mui/material';
 import { Link as RouterLink } from 'react-router-dom';
 import {
-  formatAlarm, formatAltitude, formatBoolean, formatCoordinate, formatCourse, formatDistance, formatNumber, formatNumericHours, formatPercentage, formatSpeed, formatTime, formatTemperature, formatVoltage,
+  formatAlarm,
+  formatAltitude,
+  formatBoolean,
+  formatCoordinate,
+  formatCourse,
+  formatDistance,
+  formatNumber,
+  formatNumericHours,
+  formatPercentage,
+  formatSpeed,
+  formatTime,
+  formatTemperature,
+  formatVoltage,
+  formatVolume,
+  formatConsumption,
 } from '../util/formatter';
 import { useAttributePreference, usePreference } from '../util/preferences';
 import { useTranslation } from './LocalizationProvider';
@@ -24,6 +38,7 @@ const PositionValue = ({ position, property, attribute }) => {
   const distanceUnit = useAttributePreference('distanceUnit');
   const altitudeUnit = useAttributePreference('altitudeUnit');
   const speedUnit = useAttributePreference('speedUnit');
+  const volumeUnit = useAttributePreference('volumeUnit');
   const coordinateFormat = usePreference('coordinateFormat');
   const hours12 = usePreference('twelveHourFormat');
 
@@ -39,7 +54,7 @@ const PositionValue = ({ position, property, attribute }) => {
         return formatCoordinate('longitude', value, coordinateFormat);
       case 'speed':
       case 'obdSpeed':
-        return formatSpeed(value, speedUnit, t);
+        return value != null ? formatSpeed(value, speedUnit, t) : '';
       case 'course':
         return formatCourse(value);
       case 'altitude':
@@ -49,6 +64,10 @@ const PositionValue = ({ position, property, attribute }) => {
         return formatVoltage(value, t);
       case 'batteryLevel':
         return value != null ? formatPercentage(value, t) : '';
+      case 'volume':
+        return value != null ? formatVolume(value, volumeUnit, t) : '';
+      case 'fuelConsumption':
+        return value != null ? formatConsumption(value, t) : '';
       case 'coolantTemp':
         return formatTemperature(value);
       case 'alarm':
diff --git a/modern/src/common/util/formatter.js b/modern/src/common/util/formatter.js
index 3ef83dba..0f7d2cc8 100644
--- a/modern/src/common/util/formatter.js
+++ b/modern/src/common/util/formatter.js
@@ -21,6 +21,8 @@ export const formatTemperature = (value) => `${value}°C`;
 
 export const formatVoltage = (value, t) => `${value} ${t('sharedVoltAbbreviation')}`;
 
+export const formatConsumption = (value, t) => `${value} ${t('sharedLiterPerHourAbbreviation')}`;
+
 export const formatTime = (value, format, hours12) => {
   if (value) {
     const m = moment(value);
-- 
cgit v1.2.3