From 47c9d9fb21e897733f7aaa29bb5797036ccc4476 Mon Sep 17 00:00:00 2001
From: Anton Tananaev <anton@traccar.org>
Date: Thu, 2 Feb 2023 13:56:18 -0800
Subject: Add event positions

---
 .../org/traccar/reports/CombinedReportProvider.java    | 18 +++++++++++++-----
 .../org/traccar/reports/model/CombinedReportItem.java  | 11 +++++++++++
 2 files changed, 24 insertions(+), 5 deletions(-)

(limited to 'src/main/java/org/traccar')

diff --git a/src/main/java/org/traccar/reports/CombinedReportProvider.java b/src/main/java/org/traccar/reports/CombinedReportProvider.java
index ca290f7db..d47a91bef 100644
--- a/src/main/java/org/traccar/reports/CombinedReportProvider.java
+++ b/src/main/java/org/traccar/reports/CombinedReportProvider.java
@@ -53,16 +53,24 @@ public class CombinedReportProvider {
         for (Device device: reportUtils.getAccessibleDevices(userId, deviceIds, groupIds)) {
             CombinedReportItem item = new CombinedReportItem();
             item.setDeviceId(device.getId());
-            item.setRoute(PositionUtil.getPositions(storage, device.getId(), from, to)
-                    .stream()
-                    .map(p -> new double[] { p.getLongitude(), p.getLatitude() })
+            var positions = PositionUtil.getPositions(storage, device.getId(), from, to);
+            item.setRoute(positions.stream()
+                    .map(p -> new double[] {p.getLongitude(), p.getLatitude()})
                     .collect(Collectors.toList()));
-            item.setEvents(storage.getObjects(Event.class, new Request(
+            var events = storage.getObjects(Event.class, new Request(
                     new Columns.All(),
                     new Condition.And(
                             new Condition.Equals("deviceId", device.getId()),
                             new Condition.Between("eventTime", "from", from, "to", to)),
-                    new Order("eventTime"))));
+                    new Order("eventTime")));
+            item.setEvents(events);
+            var eventPositions = events.stream()
+                    .map(Event::getPositionId)
+                    .filter(positionId -> positionId > 0)
+                    .collect(Collectors.toSet());
+            item.setPositions(positions.stream()
+                    .filter(p -> eventPositions.contains(p.getId()))
+                    .collect(Collectors.toList()));
             result.add(item);
         }
         return result;
diff --git a/src/main/java/org/traccar/reports/model/CombinedReportItem.java b/src/main/java/org/traccar/reports/model/CombinedReportItem.java
index 00f2cc08e..810e00916 100644
--- a/src/main/java/org/traccar/reports/model/CombinedReportItem.java
+++ b/src/main/java/org/traccar/reports/model/CombinedReportItem.java
@@ -16,6 +16,7 @@
 package org.traccar.reports.model;
 
 import org.traccar.model.Event;
+import org.traccar.model.Position;
 
 import java.util.List;
 
@@ -51,4 +52,14 @@ public class CombinedReportItem {
         this.events = events;
     }
 
+    private List<Position> positions;
+
+    public List<Position> getPositions() {
+        return positions;
+    }
+
+    public void setPositions(List<Position> positions) {
+        this.positions = positions;
+    }
+
 }
-- 
cgit v1.2.3