[MNG-7856] Maven Resolver Provider ctor and other minor code changes (#…

…1223) Maven Resolver Provider gets similar change as other Resolver components had in MRESOLVER-386. Also, some minor code updates like adding Override and other cosmetic stuff. --- https://issues.apache.org/jira/browse/MNG-7856
apache · Sep 5, 2023 · 1ac8be5 · 1ac8be5
1 parent c482de8
commit 1ac8be5
Show file tree

Hide file tree

Showing 16 changed files with 58 additions and 9 deletions.
diff --git a/.../src/main/java/org/apache/maven/repository/internal/ArtifactDescriptorReaderDelegate.java b/.../src/main/java/org/apache/maven/repository/internal/ArtifactDescriptorReaderDelegate.java
@@ -44,7 +44,7 @@
 
 /**
  * Populates Aether {@link ArtifactDescriptorResult} from Maven project {@link Model}.
- *
+ * <p/>
  * <strong>Note:</strong> This class is part of work in progress and can be changed or removed without notice.
  * @since 3.2.4
  */
@@ -96,7 +96,7 @@ private Dependency convert(org.apache.maven.model.Dependency dependency, Artifac
         }
 
         boolean system =
-                dependency.getSystemPath() != null && dependency.getSystemPath().length() > 0;
+                dependency.getSystemPath() != null && !dependency.getSystemPath().isEmpty();
 
         Map<String, String> props = null;
         if (system) {
@@ -136,7 +136,7 @@ private void setArtifactProperties(ArtifactDescriptorResult result, Model model)
         if (distMgmt != null) {
             downloadUrl = distMgmt.getDownloadUrl();
         }
-        if (downloadUrl != null && downloadUrl.length() > 0) {
+        if (downloadUrl != null && !downloadUrl.isEmpty()) {
             Artifact artifact = result.getArtifact();
             Map<String, String> props = new HashMap<>(artifact.getProperties());
             props.put(ArtifactProperties.DOWNLOAD_URL, downloadUrl);

diff --git a/...-provider/src/main/java/org/apache/maven/repository/internal/ArtifactDescriptorUtils.java b/...-provider/src/main/java/org/apache/maven/repository/internal/ArtifactDescriptorUtils.java
@@ -35,7 +35,7 @@ public class ArtifactDescriptorUtils {
     public static Artifact toPomArtifact(Artifact artifact) {
         Artifact pomArtifact = artifact;
 
-        if (pomArtifact.getClassifier().length() > 0 || !"pom".equals(pomArtifact.getExtension())) {
+        if (!pomArtifact.getClassifier().isEmpty() || !"pom".equals(pomArtifact.getExtension())) {
             pomArtifact =
                     new DefaultArtifact(artifact.getGroupId(), artifact.getArtifactId(), "pom", artifact.getVersion());
         }

diff --git a/...r/src/main/java/org/apache/maven/repository/internal/DefaultArtifactDescriptorReader.java b/...r/src/main/java/org/apache/maven/repository/internal/DefaultArtifactDescriptorReader.java
@@ -91,12 +91,13 @@ public class DefaultArtifactDescriptorReader implements ArtifactDescriptorReader
     private final ArtifactDescriptorReaderDelegate artifactDescriptorReaderDelegate =
             new ArtifactDescriptorReaderDelegate();
 
+    @Deprecated
     public DefaultArtifactDescriptorReader() {
         // enable no-arg constructor
     }
 
     @Inject
-    DefaultArtifactDescriptorReader(
+    public DefaultArtifactDescriptorReader(
             RemoteRepositoryManager remoteRepositoryManager,
             VersionResolver versionResolver,
             VersionRangeResolver versionRangeResolver,
@@ -113,6 +114,7 @@ public DefaultArtifactDescriptorReader() {
         setModelCacheFactory(modelCacheFactory);
     }
 
+    @Deprecated
     public void initService(ServiceLocator locator) {
         setRemoteRepositoryManager(locator.getService(RemoteRepositoryManager.class));
         setVersionResolver(locator.getService(VersionResolver.class));
@@ -165,6 +167,7 @@ public DefaultArtifactDescriptorReader setModelCacheFactory(ModelCacheFactory mo
         return this;
     }
 
+    @Override
     public ArtifactDescriptorResult readArtifactDescriptor(
             RepositorySystemSession session, ArtifactDescriptorRequest request) throws ArtifactDescriptorException {
         ArtifactDescriptorResult result = new ArtifactDescriptorResult(request);

diff --git a/...solver-provider/src/main/java/org/apache/maven/repository/internal/DefaultModelCache.java b/...solver-provider/src/main/java/org/apache/maven/repository/internal/DefaultModelCache.java
@@ -46,10 +46,12 @@ private DefaultModelCache(RepositorySystemSession session) {
         this.cache = session.getCache();
     }
 
+    @Override
     public Object get(String groupId, String artifactId, String version, String tag) {
         return cache.get(session, new Key(groupId, artifactId, version, tag));
     }
 
+    @Override
     public void put(String groupId, String artifactId, String version, String tag, Object data) {
         cache.put(session, new Key(groupId, artifactId, version, tag), data);
     }

diff --git a/...vider/src/main/java/org/apache/maven/repository/internal/DefaultVersionRangeResolver.java b/...vider/src/main/java/org/apache/maven/repository/internal/DefaultVersionRangeResolver.java
@@ -77,12 +77,13 @@ public class DefaultVersionRangeResolver implements VersionRangeResolver, Servic
 
     private RepositoryEventDispatcher repositoryEventDispatcher;
 
+    @Deprecated
     public DefaultVersionRangeResolver() {
         // enable default constructor
     }
 
     @Inject
-    DefaultVersionRangeResolver(
+    public DefaultVersionRangeResolver(
             MetadataResolver metadataResolver,
             SyncContextFactory syncContextFactory,
             RepositoryEventDispatcher repositoryEventDispatcher) {
@@ -91,6 +92,7 @@ public DefaultVersionRangeResolver() {
         setRepositoryEventDispatcher(repositoryEventDispatcher);
     }
 
+    @Deprecated
     public void initService(ServiceLocator locator) {
         setMetadataResolver(locator.getService(MetadataResolver.class));
         setSyncContextFactory(locator.getService(SyncContextFactory.class));
@@ -114,6 +116,7 @@ public DefaultVersionRangeResolver setRepositoryEventDispatcher(
         return this;
     }
 
+    @Override
     public VersionRangeResult resolveVersionRange(RepositorySystemSession session, VersionRangeRequest request)
             throws VersionRangeResolutionException {
         VersionRangeResult result = new VersionRangeResult(request);

diff --git a/...r-provider/src/main/java/org/apache/maven/repository/internal/DefaultVersionResolver.java b/...r-provider/src/main/java/org/apache/maven/repository/internal/DefaultVersionResolver.java
@@ -86,12 +86,13 @@ public class DefaultVersionResolver implements VersionResolver, Service {
 
     private RepositoryEventDispatcher repositoryEventDispatcher;
 
+    @Deprecated
     public DefaultVersionResolver() {
         // enable no-arg constructor
     }
 
     @Inject
-    DefaultVersionResolver(
+    public DefaultVersionResolver(
             MetadataResolver metadataResolver,
             SyncContextFactory syncContextFactory,
             RepositoryEventDispatcher repositoryEventDispatcher) {
@@ -100,6 +101,7 @@ public DefaultVersionResolver() {
         setRepositoryEventDispatcher(repositoryEventDispatcher);
     }
 
+    @Deprecated
     public void initService(ServiceLocator locator) {
         setMetadataResolver(locator.getService(MetadataResolver.class));
         setSyncContextFactory(locator.getService(SyncContextFactory.class));
@@ -123,6 +125,7 @@ public DefaultVersionResolver setRepositoryEventDispatcher(RepositoryEventDispat
     }
 
     @SuppressWarnings("checkstyle:methodlength")
+    @Override
     public VersionResult resolveVersion(RepositorySystemSession session, VersionRequest request)
             throws VersionResolutionException {
         RequestTrace trace = RequestTrace.newChild(request.getTrace(), request);

diff --git a/...er-provider/src/main/java/org/apache/maven/repository/internal/LocalSnapshotMetadata.java b/...er-provider/src/main/java/org/apache/maven/repository/internal/LocalSnapshotMetadata.java
@@ -73,6 +73,7 @@ public void bind(Artifact artifact) {
         artifacts.add(artifact);
     }
 
+    @Override
     public MavenMetadata setFile(File file) {
         return new LocalSnapshotMetadata(metadata, file, legacyFormat, timestamp);
     }
@@ -123,18 +124,22 @@ private String getKey(String classifier, String extension) {
         return classifier + ':' + extension;
     }
 
+    @Override
     public String getGroupId() {
         return metadata.getGroupId();
     }
 
+    @Override
     public String getArtifactId() {
         return metadata.getArtifactId();
     }
 
+    @Override
     public String getVersion() {
         return metadata.getVersion();
     }
 
+    @Override
     public Nature getNature() {
         return Nature.SNAPSHOT;
     }

diff --git a/...er/src/main/java/org/apache/maven/repository/internal/LocalSnapshotMetadataGenerator.java b/...er/src/main/java/org/apache/maven/repository/internal/LocalSnapshotMetadataGenerator.java
@@ -52,6 +52,7 @@ class LocalSnapshotMetadataGenerator implements MetadataGenerator {
         snapshots = new LinkedHashMap<>();
     }
 
+    @Override
     public Collection<? extends Metadata> prepare(Collection<? extends Artifact> artifacts) {
         for (Artifact artifact : artifacts) {
             if (artifact.isSnapshot()) {
@@ -68,10 +69,12 @@ public Collection<? extends Metadata> prepare(Collection<? extends Artifact> art
         return Collections.emptyList();
     }
 
+    @Override
     public Artifact transformArtifact(Artifact artifact) {
         return artifact;
     }
 
+    @Override
     public Collection<? extends Metadata> finish(Collection<? extends Artifact> artifacts) {
         return snapshots.values();
     }

diff --git a/...n-resolver-provider/src/main/java/org/apache/maven/repository/internal/MavenMetadata.java b/...n-resolver-provider/src/main/java/org/apache/maven/repository/internal/MavenMetadata.java
@@ -57,14 +57,17 @@ protected MavenMetadata(Metadata metadata, File file, Date timestamp) {
         this.timestamp = timestamp;
     }
 
+    @Override
     public String getType() {
         return MAVEN_METADATA_XML;
     }
 
+    @Override
     public File getFile() {
         return file;
     }
 
+    @Override
     public void merge(File existing, File result) throws RepositoryException {
         Metadata recessive = read(existing);
 
@@ -75,6 +78,7 @@ public void merge(File existing, File result) throws RepositoryException {
         merged = true;
     }
 
+    @Override
     public boolean isMerged() {
         return merged;
     }
@@ -104,6 +108,7 @@ private void write(File metadataFile, Metadata metadata) throws RepositoryExcept
         }
     }
 
+    @Override
     public Map<String, String> getProperties() {
         return Collections.emptyMap();
     }

diff --git a/...lver-provider/src/main/java/org/apache/maven/repository/internal/MavenResolverModule.java b/...lver-provider/src/main/java/org/apache/maven/repository/internal/MavenResolverModule.java
@@ -39,6 +39,7 @@
 /**
  * MavenResolverModule
  */
+@Deprecated
 public final class MavenResolverModule extends AbstractModule {
 
     @Override

diff --git a/...er-provider/src/main/java/org/apache/maven/repository/internal/MavenSnapshotMetadata.java b/...er-provider/src/main/java/org/apache/maven/repository/internal/MavenSnapshotMetadata.java
@@ -69,18 +69,22 @@ protected String getKey(String classifier, String extension) {
         return classifier + ':' + extension;
     }
 
+    @Override
     public String getGroupId() {
         return metadata.getGroupId();
     }
 
+    @Override
     public String getArtifactId() {
         return metadata.getArtifactId();
     }
 
+    @Override
     public String getVersion() {
         return metadata.getVersion();
     }
 
+    @Override
     public Nature getNature() {
         return Nature.SNAPSHOT;
     }

diff --git a/...solver-provider/src/main/java/org/apache/maven/repository/internal/RelocatedArtifact.java b/...solver-provider/src/main/java/org/apache/maven/repository/internal/RelocatedArtifact.java
@@ -49,6 +49,7 @@ public final class RelocatedArtifact extends AbstractArtifact {
         this.message = (message != null && message.length() > 0) ? message : null;
     }
 
+    @Override
     public String getGroupId() {
         if (groupId != null) {
             return groupId;
@@ -57,6 +58,7 @@ public String getGroupId() {
         }
     }
 
+    @Override
     public String getArtifactId() {
         if (artifactId != null) {
             return artifactId;
@@ -65,6 +67,7 @@ public String getArtifactId() {
         }
     }
 
+    @Override
     public String getVersion() {
         if (version != null) {
             return version;
@@ -101,22 +104,27 @@ public Artifact setProperties(Map<String, String> properties) {
         return new RelocatedArtifact(artifact.setProperties(properties), groupId, artifactId, version, message);
     }
 
+    @Override
     public String getClassifier() {
         return artifact.getClassifier();
     }
 
+    @Override
     public String getExtension() {
         return artifact.getExtension();
     }
 
+    @Override
     public File getFile() {
         return artifact.getFile();
     }
 
+    @Override
     public String getProperty(String key, String defaultValue) {
         return artifact.getProperty(key, defaultValue);
     }
 
+    @Override
     public Map<String, String> getProperties() {
         return artifact.getProperties();
     }

diff --git a/...r-provider/src/main/java/org/apache/maven/repository/internal/RemoteSnapshotMetadata.java b/...r-provider/src/main/java/org/apache/maven/repository/internal/RemoteSnapshotMetadata.java
@@ -52,6 +52,7 @@ private RemoteSnapshotMetadata(Metadata metadata, File file, boolean legacyForma
         super(metadata, file, legacyFormat, timestamp);
     }
 
+    @Override
     public MavenMetadata setFile(File file) {
         return new RemoteSnapshotMetadata(metadata, file, legacyFormat, timestamp);
     }

diff --git a/...r/src/main/java/org/apache/maven/repository/internal/RemoteSnapshotMetadataGenerator.java b/...r/src/main/java/org/apache/maven/repository/internal/RemoteSnapshotMetadataGenerator.java
@@ -65,6 +65,7 @@ class RemoteSnapshotMetadataGenerator implements MetadataGenerator {
         }
     }
 
+    @Override
     public Collection<? extends Metadata> prepare(Collection<? extends Artifact> artifacts) {
         for (Artifact artifact : artifacts) {
             if (artifact.isSnapshot()) {
@@ -81,6 +82,7 @@ public Collection<? extends Metadata> prepare(Collection<? extends Artifact> art
         return snapshots.values();
     }
 
+    @Override
     public Artifact transformArtifact(Artifact artifact) {
         if (artifact.isSnapshot() && artifact.getVersion().equals(artifact.getBaseVersion())) {
             Object key = RemoteSnapshotMetadata.getKey(artifact);
@@ -93,6 +95,7 @@ public Artifact transformArtifact(Artifact artifact) {
         return artifact;
     }
 
+    @Override
     public Collection<? extends Metadata> finish(Collection<? extends Artifact> artifacts) {
         return Collections.emptyList();
     }

diff --git a/...esolver-provider/src/main/java/org/apache/maven/repository/internal/VersionsMetadata.java b/...esolver-provider/src/main/java/org/apache/maven/repository/internal/VersionsMetadata.java
@@ -93,22 +93,27 @@ public static Object getKey(Artifact artifact) {
         return artifact.getGroupId() + ':' + artifact.getArtifactId();
     }
 
+    @Override
     public MavenMetadata setFile(File file) {
         return new VersionsMetadata(artifact, file, timestamp);
     }
 
+    @Override
     public String getGroupId() {
         return artifact.getGroupId();
     }
 
+    @Override
     public String getArtifactId() {
         return artifact.getArtifactId();
     }
 
+    @Override
     public String getVersion() {
         return "";
     }
 
+    @Override
     public Nature getNature() {
         return artifact.isSnapshot() ? Nature.RELEASE_OR_SNAPSHOT : Nature.RELEASE;
     }