#31022 Setting host Id in Url Map events

dotCMS · Jan 9, 2025 · d3302a2 · d3302a2
1 parent 3dbb005
commit d3302a2
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 0 deletions.
diff --git a/dotCMS/src/main/java/com/dotcms/analytics/track/collectors/PageDetailCollector.java b/dotCMS/src/main/java/com/dotcms/analytics/track/collectors/PageDetailCollector.java
@@ -95,6 +95,7 @@ public CollectorPayloadBean collect(final CollectorContextMap collectorContextMa
 
         if (Objects.nonNull(site)) {
             collectorPayloadBean.put(SITE_NAME, site.getHostname());
+            collectorPayloadBean.put(SITE_ID, site.getIdentifier());
         }
         return collectorPayloadBean;
     }

diff --git a/...egration/src/test/java/com/dotcms/analytics/track/collectors/PageDetailCollectorTest.java b/...egration/src/test/java/com/dotcms/analytics/track/collectors/PageDetailCollectorTest.java
@@ -103,6 +103,7 @@ public void testPageDetailCollector() throws DotDataException, UnknownHostExcept
         final Map<String, Object> expectedDataMap = Map.of(
                 Collector.EVENT_TYPE, EventType.PAGE_REQUEST.getType(),
                 Collector.SITE_NAME, testSite.getHostname(),
+                Collector.SITE_ID, testSite.getIdentifier(),
                 Collector.LANGUAGE, language.getIsoCode(),
                 Collector.URL, TEST_URL_MAP_DETAIL_PAGE_URL,
                 Collector.OBJECT, Map.of(

diff --git a/dotcms-integration/src/test/java/com/dotcms/analytics/track/collectors/Util.java b/dotcms-integration/src/test/java/com/dotcms/analytics/track/collectors/Util.java
@@ -37,6 +37,7 @@
 import com.dotmarketing.util.UtilMethods;
 import com.dotmarketing.util.WebKeys;
 import com.liferay.portal.model.User;
+import graphql.AssertException;
 import io.vavr.control.Try;
 
 import javax.servlet.http.HttpServletRequest;
@@ -325,6 +326,8 @@ public static void validateExpectedEntries(final Map<String, Object> expectedDat
                     assertEquals("Collected value must be equal to expected value for key: " + key,
                             expectedValue, collectedValue);
                 }
+            } else {
+                throw new AssertException("Expected key in the Collected value: " + key);
             }
         }
     }