This is an automated email from the ASF dual-hosted git repository. michaelo pushed a commit to branch MNG-7658 in repository https://gitbox.apache.org/repos/asf/maven.git
commit b9678ce4e760cac9e156b2097aa50cad154f5e20 Author: Michael Osipov <[email protected]> AuthorDate: Sun Jan 1 19:52:02 2023 +0100 [MNG-7658] CI-friendly versions should only come/rely on user properties This closes #945 --- .../model/interpolation/DefaultModelVersionProcessor.java | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/maven-model-builder/src/main/java/org/apache/maven/model/interpolation/DefaultModelVersionProcessor.java b/maven-model-builder/src/main/java/org/apache/maven/model/interpolation/DefaultModelVersionProcessor.java index a80490e4a..a01c4e6d7 100644 --- a/maven-model-builder/src/main/java/org/apache/maven/model/interpolation/DefaultModelVersionProcessor.java +++ b/maven-model-builder/src/main/java/org/apache/maven/model/interpolation/DefaultModelVersionProcessor.java @@ -46,18 +46,16 @@ public class DefaultModelVersionProcessor implements ModelVersionProcessor { || SHA1_PROPERTY.equals(property); } - // TODO This class MUST test for user properties and THEN for system properties @Override public void overwriteModelProperties(Properties modelProperties, ModelBuildingRequest request) { - if (request.getSystemProperties().containsKey(REVISION_PROPERTY)) { - modelProperties.put(REVISION_PROPERTY, request.getSystemProperties().get(REVISION_PROPERTY)); + if (request.getUserProperties().containsKey(REVISION_PROPERTY)) { + modelProperties.put(REVISION_PROPERTY, request.getUserProperties().get(REVISION_PROPERTY)); } - if (request.getSystemProperties().containsKey(CHANGELIST_PROPERTY)) { - modelProperties.put( - CHANGELIST_PROPERTY, request.getSystemProperties().get(CHANGELIST_PROPERTY)); + if (request.getUserProperties().containsKey(CHANGELIST_PROPERTY)) { + modelProperties.put(CHANGELIST_PROPERTY, request.getUserProperties().get(CHANGELIST_PROPERTY)); } - if (request.getSystemProperties().containsKey(SHA1_PROPERTY)) { - modelProperties.put(SHA1_PROPERTY, request.getSystemProperties().get(SHA1_PROPERTY)); + if (request.getUserProperties().containsKey(SHA1_PROPERTY)) { + modelProperties.put(SHA1_PROPERTY, request.getUserProperties().get(SHA1_PROPERTY)); } } }
