Give explicit property definitions higher precedence than Maven's imp…

…licit ones. (openrewrite#4334)

* Exclude Maven built-in property from circular placeholder reference check.

* Revert "Exclude Maven built-in property from circular placeholder reference check."

This reverts commit 86ae0f1.

* Give explicit property definitions higher precedence than Maven's implicit ones.

* Reformat test

---------

Co-authored-by: hoanamzn <hoanamzn@amazon.com>
Co-authored-by: Tim te Beek <tim@moderne.io>

rewrite-maven/src/main/java/org/openrewrite/maven/tree/ResolvedPom.java

@@ -275,6 +275,10 @@ public String getPackaging() {
         if (property == null) {
             return null;
         }
+        String propVal = properties.get(property);
+        if (propVal != null) {
+            return propVal;
+        }
         switch (property) {
             case "groupId":
             case "project.groupId":
@@ -299,7 +303,7 @@ public String getPackaging() {
                 return requested.getParent() != null ? requested.getParent().getVersion() : null;
         }
 
-        return System.getProperty(property, properties.get(property));
+        return System.getProperty(property);
     }
 
     public @Nullable String getManagedVersion(@Nullable String groupId, String artifactId, @Nullable String type, @Nullable String classifier) {