gnodet commented on code in PR #1774:
URL: https://github.com/apache/maven/pull/1774#discussion_r4292298602
##########
impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultProfileActivationContext.java:
##########
@@ -337,6 +340,36 @@ public String getModelProperty(String key) {
}
}
+ /**
+ * Gets a model property INCLUDING cascaded profile properties.
+ * This should ONLY be used for profile activation to enable cascading.
+ * Regular property lookups should use getModelProperty() which excludes
cascaded properties.
+ */
+ public String getModelPropertyForActivation(String key) {
Review Comment:
_Claude Code on behalf of Guillaume Nodet_
Fixed — `getModelPropertyForActivation` is now `private`. It is only called
internally by `interpolate()` within the same class. The activators use the SPI
methods (`getCascadedProfileProperty` / `getModelProperty`) directly.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
