Merge pull request #45487 from aloubyansky/analytics-quarkus-version

Fix Quarkus platform BOM version info collection for analytics
quarkusio · Jan 10, 2025 · d379a5e · d379a5e
2 parents f040b40 + 3e7ebdf
commit d379a5e
Showing 1 changed file with 14 additions and 6 deletions.
diff --git a/...cts/bootstrap/app-model/src/main/java/io/quarkus/bootstrap/model/PlatformImportsImpl.java b/...cts/bootstrap/app-model/src/main/java/io/quarkus/bootstrap/model/PlatformImportsImpl.java
@@ -67,7 +67,7 @@ void addPlatformRelease(String propertyName, String propertyValue) {
         final String platformKey = propertyName.substring(PROPERTY_PREFIX.length(), platformKeyStreamSep);
         final String streamId = propertyName.substring(platformKeyStreamSep + 1, streamVersionSep);
         final String version = propertyName.substring(streamVersionSep + 1);
-        allPlatformInfo.computeIfAbsent(platformKey, k -> new PlatformInfo(k)).getOrCreateStream(streamId).addIfNotPresent(
+        allPlatformInfo.computeIfAbsent(platformKey, PlatformInfo::new).getOrCreateStream(streamId).addIfNotPresent(
                 version,
                 () -> {
                     final PlatformReleaseInfo ri = new PlatformReleaseInfo(platformKey, streamId, version, propertyValue);
@@ -81,10 +81,7 @@ public void addPlatformDescriptor(String groupId, String artifactId, String clas
                 artifactId.substring(0,
                         artifactId.length() - BootstrapConstants.PLATFORM_DESCRIPTOR_ARTIFACT_ID_SUFFIX.length()),
                 version);
-        platformImports.computeIfAbsent(bomCoords, c -> {
-            platformBoms.add(bomCoords);
-            return new PlatformImport();
-        }).descriptorFound = true;
+        platformImports.computeIfAbsent(bomCoords, this::newPlatformImport).descriptorFound = true;
     }
 
     public void addPlatformProperties(String groupId, String artifactId, String classifier, String type, String version,
@@ -93,7 +90,7 @@ public void addPlatformProperties(String groupId, String artifactId, String clas
                 artifactId.substring(0,
                         artifactId.length() - BootstrapConstants.PLATFORM_PROPERTIES_ARTIFACT_ID_SUFFIX.length()),
                 version);
-        platformImports.computeIfAbsent(bomCoords, c -> new PlatformImport());
+        platformImports.computeIfAbsent(bomCoords, this::newPlatformImport);
         importedPlatformBoms.computeIfAbsent(groupId, g -> new ArrayList<>());
         if (!importedPlatformBoms.get(groupId).contains(bomCoords)) {
             importedPlatformBoms.get(groupId).add(bomCoords);
@@ -117,6 +114,17 @@ public void addPlatformProperties(String groupId, String artifactId, String clas
         }
     }
 
+    /**
+     * This method is meant to be called when a new platform BOM import was detected.
+     *
+     * @param bom platform BOM coordinates
+     * @return new platform import instance
+     */
+    private PlatformImport newPlatformImport(ArtifactCoords bom) {
+        platformBoms.add(bom);
+        return new PlatformImport();
+    }
+
     public void setPlatformProperties(Map<String, String> platformProps) {
         this.collectedProps.putAll(platformProps);
     }