Move value formatting code

2 files changed, 38 insertions, 39 deletions

diff --git a/web/app/view/report/Report.js b/web/app/view/report/Report.js
index 8a1dae9c4..67547e536 100644
--- a/web/app/view/report/Report.js
+++ b/web/app/view/report/Report.js
@@ -78,23 +78,39 @@ Ext.define('Traccar.view.report.Report', {
         selectionchange: 'onSelectionChange'
     },
 
-    columns: [
-        { text: strings.positionValid, dataIndex: 'valid', flex: 1 },
-        { text: strings.positionTime, dataIndex: 'fixTime', flex: 1, xtype: 'datecolumn', format: styles.dateTimeFormat },
-        { text: strings.positionLatitude, dataIndex: 'latitude', flex: 1 },
-        { text: strings.positionLongitude, dataIndex: 'longitude', flex: 1 },
-        { text: strings.positionAltitude, dataIndex: 'altitude', flex: 1 },
-        {
-            text: strings.positionSpeed,
-            dataIndex: 'speed',
-            flex: 1,
-            renderer: function(value) {
-                var speedUnits = Ext.getStore('SpeedUnits');
-                var unit = Traccar.getApplication().getUser().get('speedUnit') || Traccar.getApplication().getServer().get('speedUnit') || '';
-                return speedUnits.formatValue(value, unit);
-            }
-        },
-        { text: strings.positionAddress, dataIndex: 'address', flex: 1 }
-    ]
-
+    columns: [{
+        text: strings.positionValid,
+        dataIndex: 'valid',
+        flex: 1
+    }, {
+        text: strings.positionTime,
+        dataIndex: 'fixTime',
+        flex: 1,
+        xtype: 'datecolumn',
+        //renderer: Traccar.app.getRenderer()
+    }, {
+        text: strings.positionLatitude,
+        dataIndex: 'latitude',
+        flex: 1,
+        //renderer: Traccar.app.getRenderer('latitude')
+    }, {
+        text: strings.positionLongitude,
+        dataIndex: 'longitude',
+        flex: 1,
+        //renderer: Traccar.app.getRenderer('latitude')
+    }, {
+        text: strings.positionAltitude,
+        dataIndex: 'altitude',
+        flex: 1,
+        //renderer: Traccar.app.getRenderer('altitude')
+    }, {
+        text: strings.positionSpeed,
+        dataIndex: 'speed',
+        flex: 1,
+        //renderer: Traccar.app.getRenderer('speed')
+    }, {
+        text: strings.positionAddress,
+        dataIndex: 'address',
+        flex: 1
+    }]
 });
diff --git a/web/app/view/state/StateController.js b/web/app/view/state/StateController.js
index 23ac72db3..91fa7714f 100644
--- a/web/app/view/state/StateController.js
+++ b/web/app/view/state/StateController.js
@@ -91,19 +91,10 @@ Ext.define('Traccar.view.state.StateController', {
 
         for (var key in position.data) {
             if (position.data.hasOwnProperty(key) && this.keys[key] !== undefined) {
-                value = position.get(key);
-                if (key === 'speed') {
-                    var speedUnits = Ext.getStore('SpeedUnits');
-                    unit = Traccar.getApplication().getUser().get('speedUnit') || Traccar.getApplication().getServer().get('speedUnit') || '';
-                    value = speedUnits.formatValue(value, unit);
-                } else if (value instanceof Date) {
-                    value = Ext.Date.format(value, styles.dateTimeFormat);
-                }
-                
                 store.add(Ext.create('Traccar.model.Attribute', {
                     priority: this.keys[key].priority,
                     name: this.keys[key].name,
-                    value: this.formatValue(value)
+                    value: Traccar.getApplication().getRenderer(key)(position.get(key))
                 }));
             }
         }
@@ -116,20 +107,12 @@ Ext.define('Traccar.view.state.StateController', {
         }
         for (var key in attributes) {
             if (attributes.hasOwnProperty(key)) {
-
-                value = attributes[key];
-                if (key === 'distance' || key === 'odometer') {
-                    var distanceUnits = Ext.getStore('DistanceUnits');
-                    unit = Traccar.getApplication().getUser().get('distanceUnit') || Traccar.getApplication().getServer().get('distanceUnit') || '';
-                    value = distanceUnits.formatValue(value, unit);
-                }
-
                 store.add(Ext.create('Traccar.model.Attribute', {
-                    priority: 999,
+                    priority: 1024,
                     name: key.replace(/^./, function (match) {
                         return match.toUpperCase();
                     }),
-                    value: this.formatValue(value)
+                    value: Traccar.getApplication().getRenderer(key)(attributes[key])
                 }));
             }
         }