From 38dc0349d99776a29b904c73d46c8d607a81d448 Mon Sep 17 00:00:00 2001
From: Juul Hobert <juul.hobert@infosupport.com>
Date: Fri, 9 Feb 2024 15:43:18 +0100
Subject: [PATCH] [MNG-7344] Recursive imported from tracking

---
 .../org/apache/maven/api/model/InputLocation.java   | 13 ++++++++++++-
 .../DefaultDependencyManagementImporter.java        |  6 ++++--
 2 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/api/maven-api-model/src/main/java/org/apache/maven/api/model/InputLocation.java b/api/maven-api-model/src/main/java/org/apache/maven/api/model/InputLocation.java
index a96a569bae2f..dc81d2da83a4 100644
--- a/api/maven-api-model/src/main/java/org/apache/maven/api/model/InputLocation.java
+++ b/api/maven-api-model/src/main/java/org/apache/maven/api/model/InputLocation.java
@@ -32,13 +32,14 @@ public class InputLocation implements Serializable, InputLocationTracker {
     private final int columnNumber;
     private final InputSource source;
     private final Map<Object, InputLocation> locations;
-    private final InputLocation importedFrom = null;
+    private final InputLocation importedFrom;
 
     public InputLocation(InputSource source) {
         this.lineNumber = -1;
         this.columnNumber = -1;
         this.source = source;
         this.locations = Collections.singletonMap(0, this);
+        this.importedFrom = null;
     }
 
     public InputLocation(int lineNumber, int columnNumber) {
@@ -55,6 +56,7 @@ public InputLocation(int lineNumber, int columnNumber, InputSource source, Objec
         this.source = source;
         this.locations =
                 selfLocationKey != null ? Collections.singletonMap(selfLocationKey, this) : Collections.emptyMap();
+        this.importedFrom = null;
     }
 
     public InputLocation(int lineNumber, int columnNumber, InputSource source, Map<Object, InputLocation> locations) {
@@ -62,6 +64,15 @@ public InputLocation(int lineNumber, int columnNumber, InputSource source, Map<O
         this.columnNumber = columnNumber;
         this.source = source;
         this.locations = ImmutableCollections.copy(locations);
+        this.importedFrom = null;
+    }
+
+    public InputLocation(InputLocation original, InputLocation importedFrom) {
+        this.lineNumber = original.lineNumber;
+        this.columnNumber = original.columnNumber;
+        this.source = original.source;
+        this.locations = original.locations;
+        this.importedFrom = importedFrom;
     }
 
     public int getLineNumber() {
diff --git a/maven-model-builder/src/main/java/org/apache/maven/model/composition/DefaultDependencyManagementImporter.java b/maven-model-builder/src/main/java/org/apache/maven/model/composition/DefaultDependencyManagementImporter.java
index 2a91073c3309..c560cfed1bce 100644
--- a/maven-model-builder/src/main/java/org/apache/maven/model/composition/DefaultDependencyManagementImporter.java
+++ b/maven-model-builder/src/main/java/org/apache/maven/model/composition/DefaultDependencyManagementImporter.java
@@ -76,7 +76,7 @@ public Model importManagement(
                                         + "to the dependencyManagement section of the POM."));
                     }
 
-                    if (request.isLocationTracking()) {
+                    if (present == null && request.isLocationTracking()) {
                         Dependency updatedDependency = updateWithImportedFrom(dependency, source);
                         dependencies.put(key, updatedDependency);
                     }
@@ -179,6 +179,8 @@ static Dependency updateWithImportedFrom(Dependency dependency, DependencyManage
         // We modify the input location that is used for the whole file.
         // This is likely correct because the POM hierarchy applies to the whole POM, not just one dependency.
         // TODO What to do now?!
-        return Dependency.newBuilder(dependency, true).importedFrom(bomLocation).build();
+        return Dependency.newBuilder(dependency, true)
+                .importedFrom(new InputLocation(bomLocation, dependency.getImportedFrom()))
+                .build();
     }
 }