diff options
| author | Abyss777 <abyss@fox5.ru> | 2018-05-31 10:18:29 +0500 |
|---|---|---|
| committer | Abyss777 <abyss@fox5.ru> | 2018-05-31 10:18:29 +0500 |
| commit | 786f101fab6a81007f67e868d381302f4a502d77 (patch) | |
| tree | 89c06091207e99ae73ff62ecff377847b7960ade /src/org/traccar/reports | |
| parent | 3d14b6504b01b26e529717b88318be54094b8816 (diff) | |
| download | trackermap-server-786f101fab6a81007f67e868d381302f4a502d77.tar.gz trackermap-server-786f101fab6a81007f67e868d381302f4a502d77.tar.bz2 trackermap-server-786f101fab6a81007f67e868d381302f4a502d77.zip | |
- Rename switch and variable
- Add temporary comment
Diffstat (limited to 'src/org/traccar/reports')
| -rw-r--r-- | src/org/traccar/reports/ReportUtils.java | 3 | ||||
| -rw-r--r-- | src/org/traccar/reports/Summary.java | 7 |
2 files changed, 6 insertions, 4 deletions
diff --git a/src/org/traccar/reports/ReportUtils.java b/src/org/traccar/reports/ReportUtils.java index 94cc7664f..ea383d598 100644 --- a/src/org/traccar/reports/ReportUtils.java +++ b/src/org/traccar/reports/ReportUtils.java @@ -266,7 +266,8 @@ public final class ReportUtils { if (startStop.getAttributes().containsKey(Position.KEY_HOURS) && endStop.getAttributes().containsKey(Position.KEY_HOURS)) { engineHours = endStop.getLong(Position.KEY_HOURS) - startStop.getLong(Position.KEY_HOURS); - } else if (Context.getConfig().getBoolean("engineHours.enable")) { + } else if (Context.getConfig().getBoolean("processing.engineHours.enable")) { + // Temporary fallback for old data, to be removed in May 2019 for (int i = startIndex + 1; i <= endIndex; i++) { if (positions.get(i).getBoolean(Position.KEY_IGNITION) && positions.get(i - 1).getBoolean(Position.KEY_IGNITION)) { diff --git a/src/org/traccar/reports/Summary.java b/src/org/traccar/reports/Summary.java index f57372e8d..9810424d8 100644 --- a/src/org/traccar/reports/Summary.java +++ b/src/org/traccar/reports/Summary.java @@ -44,14 +44,15 @@ public final class Summary { Position firstPosition = null; Position previousPosition = null; double speedSum = 0; - boolean needCalculateEngineHours = Context.getConfig().getBoolean("engineHours.enable"); + boolean engineHoursEnabled = Context.getConfig().getBoolean("processing.engineHours.enable"); for (Position position : positions) { if (firstPosition == null) { firstPosition = position; } - if (needCalculateEngineHours && previousPosition != null + if (engineHoursEnabled && previousPosition != null && position.getBoolean(Position.KEY_IGNITION) && previousPosition.getBoolean(Position.KEY_IGNITION)) { + // Temporary fallback for old data, to be removed in May 2019 result.addEngineHours(position.getFixTime().getTime() - previousPosition.getFixTime().getTime()); } @@ -65,7 +66,7 @@ public final class Summary { result.setAverageSpeed(speedSum / positions.size()); result.setSpentFuel(ReportUtils.calculateFuel(firstPosition, previousPosition)); - if (needCalculateEngineHours + if (engineHoursEnabled && firstPosition.getAttributes().containsKey(Position.KEY_HOURS) && previousPosition.getAttributes().containsKey(Position.KEY_HOURS)) { result.setEngineHours( |