Merge pull request #2211 from Abyss777/distance

Changed way of summary distance calculation

1 files changed, 31 insertions, 17 deletions

diff --git a/src/org/traccar/reports/Summary.java b/src/org/traccar/reports/Summary.java
index 68733ded6..402e6d432 100644
--- a/src/org/traccar/reports/Summary.java
+++ b/src/org/traccar/reports/Summary.java
@@ -26,7 +26,6 @@ import javax.json.Json;
 import javax.json.JsonArrayBuilder;
 
 import org.traccar.Context;
-import org.traccar.helper.DistanceCalculator;
 import org.traccar.model.Position;
 import org.traccar.reports.model.SummaryReport;
 import org.traccar.web.CsvBuilder;
@@ -37,33 +36,48 @@ public final class Summary {
     private Summary() {
     }
 
-    private static SummaryReport calculateGeneralResult(long deviceId, Date from, Date to) throws SQLException {
+    private static SummaryReport calculateSummaryResult(long deviceId, Date from, Date to) throws SQLException {
         SummaryReport result = new SummaryReport();
         result.setDeviceId(deviceId);
         result.setDeviceName(Context.getDeviceManager().getDeviceById(deviceId).getName());
         Collection<Position> positions = Context.getDataManager().getPositions(deviceId, from, to);
         if (positions != null && !positions.isEmpty()) {
+            Position firstPosition = null;
             Position previousPosition = null;
             double speedSum = 0;
             for (Position position : positions) {
-                if (previousPosition != null) {
-                    result.addDistance(DistanceCalculator.distance(previousPosition.getLatitude(),
-                            previousPosition.getLongitude(), position.getLatitude(), position.getLongitude()));
-                    if (position.getAttributes().get(Position.KEY_IGNITION) != null
-                            && Boolean.parseBoolean(position.getAttributes().get(Position.KEY_IGNITION).toString())
-                            && previousPosition.getAttributes().get(Position.KEY_IGNITION) != null
-                            && Boolean.parseBoolean(previousPosition.getAttributes()
-                                    .get(Position.KEY_IGNITION).toString())) {
-                        result.addEngineHours(position.getFixTime().getTime()
-                                - previousPosition.getFixTime().getTime());
-                    }
+                if (firstPosition == null) {
+                    firstPosition = position;
+                }
+                if (previousPosition != null
+                        && position.getAttributes().get(Position.KEY_IGNITION) != null
+                        && Boolean.parseBoolean(position.getAttributes().get(Position.KEY_IGNITION).toString())
+                        && previousPosition.getAttributes().get(Position.KEY_IGNITION) != null
+                        && Boolean.parseBoolean(previousPosition.getAttributes()
+                                .get(Position.KEY_IGNITION).toString())) {
+                    result.addEngineHours(position.getFixTime().getTime()
+                            - previousPosition.getFixTime().getTime());
                 }
                 previousPosition = position;
                 speedSum += position.getSpeed();
                 result.setMaxSpeed(position.getSpeed());
             }
-            result.setAverageSpeed(speedSum / positions.size());
-            result.setDistance(new BigDecimal(result.getDistance()).setScale(2, RoundingMode.HALF_UP).doubleValue());
+            if (firstPosition.getAttributes().containsKey(Position.KEY_ODOMETER)
+                    && previousPosition.getAttributes().containsKey(Position.KEY_ODOMETER)) {
+                result.setDistance((Integer.parseInt(previousPosition.getAttributes().get(Position.KEY_ODOMETER)
+                        .toString())
+                        - Integer.parseInt(firstPosition.getAttributes().get(Position.KEY_ODOMETER).toString()))
+                        * 1000);
+            } else if (firstPosition.getAttributes().containsKey(Position.KEY_TOTAL_DISTANCE)
+                    && previousPosition.getAttributes().containsKey(Position.KEY_TOTAL_DISTANCE)) {
+                result.setDistance(((Number) previousPosition.getAttributes().get(Position.KEY_TOTAL_DISTANCE))
+                        .doubleValue()
+                        - ((Number) firstPosition.getAttributes().get(Position.KEY_TOTAL_DISTANCE)).doubleValue());
+            }
+            result.setDistance(new BigDecimal(result.getDistance())
+                    .setScale(2, RoundingMode.HALF_EVEN).doubleValue());
+            result.setAverageSpeed(new BigDecimal(speedSum / positions.size())
+                    .setScale(3, RoundingMode.HALF_EVEN).doubleValue());
         }
         return result;
     }
@@ -73,7 +87,7 @@ public final class Summary {
         JsonArrayBuilder json = Json.createArrayBuilder();
         for (long deviceId: ReportUtils.getDeviceList(deviceIds, groupIds)) {
             Context.getPermissionsManager().checkDevice(userId, deviceId);
-            json.add(JsonConverter.objectToJson(calculateGeneralResult(deviceId, from, to)));
+            json.add(JsonConverter.objectToJson(calculateSummaryResult(deviceId, from, to)));
         }
         return json.build().toString();
     }
@@ -84,7 +98,7 @@ public final class Summary {
         csv.addHeaderLine(new SummaryReport());
         for (long deviceId: ReportUtils.getDeviceList(deviceIds, groupIds)) {
             Context.getPermissionsManager().checkDevice(userId, deviceId);
-            csv.addLine(calculateGeneralResult(deviceId, from, to));
+            csv.addLine(calculateSummaryResult(deviceId, from, to));
         }
         return csv.build();
     }