diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java b/api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java index 7303e8e61c58..736503d25efc 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java @@ -117,7 +117,7 @@ default String key() { * Determines whether this artifact uses a snapshot version. * * @return {@code true} if the artifact is a snapshot, {@code false} otherwise - * @see org.apache.maven.api.Session#isVersionSnapshot(String) + * @see Session#isVersionSnapshot(String) */ boolean isSnapshot(); @@ -125,7 +125,7 @@ default String key() { * {@return coordinates with the same identifiers as this artifact} * This is a shortcut for {@code session.createArtifactCoordinates(artifact)}. * - * @see org.apache.maven.api.Session#createArtifactCoordinates(Artifact) + * @see Session#createArtifactCoordinates(Artifact) */ @Nonnull ArtifactCoordinates toCoordinates(); diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/Packaging.java b/api/maven-api-core/src/main/java/org/apache/maven/api/Packaging.java index 98ca09c87204..62419b85e907 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/Packaging.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/Packaging.java @@ -54,8 +54,8 @@ * } * * - * @see org.apache.maven.api.Session#requirePackaging(String) - * @see org.apache.maven.api.Project#getPackaging() + * @see Session#requirePackaging(String) + * @see Project#getPackaging() * @see org.apache.maven.api.model.Model#getPackaging() * @since 4.0.0 */ diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/Session.java b/api/maven-api-core/src/main/java/org/apache/maven/api/Session.java index 3f43210dc0f2..6d92bd864ed8 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/Session.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/Session.java @@ -693,7 +693,7 @@ Map> resolveDependencies( * * @param artifact the artifact for which to resolve the version * @return resolved version of the given artifact - * @throws org.apache.maven.api.services.VersionResolverException if the resolution failed + * @throws VersionResolverException if the resolution failed * * @see org.apache.maven.api.services.VersionResolver#resolve(Session, ArtifactCoordinates) (String) */ @@ -839,7 +839,7 @@ Optional resolveHighestVersion(@Nonnull ArtifactCoordinates artifact, L * @return the scope for the given identifier (never null) * @throws IllegalArgumentException if the given identifier is not a known scope * - * @see org.apache.maven.api.DependencyScope#forId(String) + * @see DependencyScope#forId(String) */ @Nonnull DependencyScope requireDependencyScope(@Nonnull String id); diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/Version.java b/api/maven-api-core/src/main/java/org/apache/maven/api/Version.java index cba76cac2849..c883682da4f2 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/Version.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/Version.java @@ -28,7 +28,7 @@ * * @since 4.0.0 * @see org.apache.maven.api.services.VersionParser#parseVersion(String) - * @see org.apache.maven.api.Session#parseVersion(String) + * @see Session#parseVersion(String) * @see VersionConstraint * @see VersionRange */ diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/VersionConstraint.java b/api/maven-api-core/src/main/java/org/apache/maven/api/VersionConstraint.java index 4d5d08204eef..1ad0ed9af57a 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/VersionConstraint.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/VersionConstraint.java @@ -32,7 +32,7 @@ * @see Version * @see VersionRange * @see org.apache.maven.api.services.VersionParser#parseVersionConstraint(String) - * @see org.apache.maven.api.Session#parseVersionConstraint(String) + * @see Session#parseVersionConstraint(String) * @since 4.0.0 */ @Experimental diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/VersionRange.java b/api/maven-api-core/src/main/java/org/apache/maven/api/VersionRange.java index ca58019dc02e..d3241e948d33 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/VersionRange.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/VersionRange.java @@ -31,7 +31,7 @@ * @see Version * @see VersionConstraint * @see org.apache.maven.api.services.VersionParser#parseVersionRange(String) - * @see org.apache.maven.api.Session#parseVersionRange(String) + * @see Session#parseVersionRange(String) * @since 4.0.0 */ @Experimental diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/services/DependencyResolverResult.java b/api/maven-api-core/src/main/java/org/apache/maven/api/services/DependencyResolverResult.java index 456cd6b505be..a2a100d4952a 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/services/DependencyResolverResult.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/services/DependencyResolverResult.java @@ -82,7 +82,7 @@ public interface DependencyResolverResult extends ResultDesign note * All types of path are determined together because they are sometime mutually exclusive. diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/services/ProblemCollector.java b/api/maven-api-core/src/main/java/org/apache/maven/api/services/ProblemCollector.java index 6e5a9d29f646..0c1b28f441f3 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/services/ProblemCollector.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/services/ProblemCollector.java @@ -46,7 +46,7 @@ public interface ProblemCollector

{ /** * Returns {@code true} if there is at least one problem collected with severity equal or more severe than - * {@link org.apache.maven.api.services.BuilderProblem.Severity#WARNING}. This check is logically equivalent + * {@link BuilderProblem.Severity#WARNING}. This check is logically equivalent * to "is there any problem reported?", given warning is the lowest severity. */ default boolean hasWarningProblems() { @@ -55,7 +55,7 @@ default boolean hasWarningProblems() { /** * Returns {@code true} if there is at least one problem collected with severity equal or more severe than - * {@link org.apache.maven.api.services.BuilderProblem.Severity#ERROR}. + * {@link BuilderProblem.Severity#ERROR}. */ default boolean hasErrorProblems() { return hasProblemsFor(BuilderProblem.Severity.ERROR); @@ -63,7 +63,7 @@ default boolean hasErrorProblems() { /** * Returns {@code true} if there is at least one problem collected with severity equal or more severe than - * {@link org.apache.maven.api.services.BuilderProblem.Severity#FATAL}. + * {@link BuilderProblem.Severity#FATAL}. */ default boolean hasFatalProblems() { return hasProblemsFor(BuilderProblem.Severity.FATAL); diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/services/ProjectManager.java b/api/maven-api-core/src/main/java/org/apache/maven/api/services/ProjectManager.java index 302768585d83..36aaf0f96e70 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/services/ProjectManager.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/services/ProjectManager.java @@ -54,7 +54,7 @@ * during parallel builds. * * @since 4.0.0 - * @see org.apache.maven.api.services.ProjectBuilder + * @see ProjectBuilder * @see Project */ @Experimental diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/services/SettingsBuilder.java b/api/maven-api-core/src/main/java/org/apache/maven/api/services/SettingsBuilder.java index 000ebb883780..3f6b94edcc18 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/services/SettingsBuilder.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/services/SettingsBuilder.java @@ -24,6 +24,7 @@ import org.apache.maven.api.Session; import org.apache.maven.api.annotations.Experimental; import org.apache.maven.api.annotations.Nonnull; +import org.apache.maven.api.settings.Profile; import org.apache.maven.api.settings.Settings; /** @@ -125,11 +126,11 @@ default ProblemCollector validate(@Nonnull Settings settings) { * Convert a model profile to a settings profile. */ @Nonnull - org.apache.maven.api.settings.Profile convert(@Nonnull org.apache.maven.api.model.Profile profile); + Profile convert(@Nonnull org.apache.maven.api.model.Profile profile); /** * Convert a settings profile to a model profile. */ @Nonnull - org.apache.maven.api.model.Profile convert(@Nonnull org.apache.maven.api.settings.Profile profile); + org.apache.maven.api.model.Profile convert(@Nonnull Profile profile); } diff --git a/api/maven-api-core/src/main/java/org/apache/maven/api/services/Source.java b/api/maven-api-core/src/main/java/org/apache/maven/api/services/Source.java index 2893faa4e248..ad3f66957601 100644 --- a/api/maven-api-core/src/main/java/org/apache/maven/api/services/Source.java +++ b/api/maven-api-core/src/main/java/org/apache/maven/api/services/Source.java @@ -45,7 +45,7 @@ * * * @since 4.0.0 - * @see org.apache.maven.api.services.ProjectBuilder#build(Session, Source) + * @see ProjectBuilder#build(Session, Source) */ @Experimental public interface Source { diff --git a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/LanguageProvider.java b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/LanguageProvider.java index 7a2f77068fe9..4094702248ca 100644 --- a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/LanguageProvider.java +++ b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/LanguageProvider.java @@ -40,8 +40,8 @@ * } * * - * @see org.apache.maven.api.Language - * @see org.apache.maven.api.spi.ExtensibleEnumProvider + * @see Language + * @see ExtensibleEnumProvider * @since 4.0.0 */ @Experimental diff --git a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/LifecycleProvider.java b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/LifecycleProvider.java index d37a327e5288..98c0ddec1482 100644 --- a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/LifecycleProvider.java +++ b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/LifecycleProvider.java @@ -48,8 +48,8 @@ * } * * - * @see org.apache.maven.api.Lifecycle - * @see org.apache.maven.api.spi.ExtensibleEnumProvider + * @see Lifecycle + * @see ExtensibleEnumProvider * @since 4.0.0 */ @Experimental diff --git a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/PackagingProvider.java b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/PackagingProvider.java index 2e2af56e8263..17812e5d4b70 100644 --- a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/PackagingProvider.java +++ b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/PackagingProvider.java @@ -43,8 +43,8 @@ * } * * - * @see org.apache.maven.api.Packaging - * @see org.apache.maven.api.spi.ExtensibleEnumProvider + * @see Packaging + * @see ExtensibleEnumProvider * @since 4.0.0 */ @Experimental diff --git a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/PathScopeProvider.java b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/PathScopeProvider.java index 7d618ae1108a..546a5276fc15 100644 --- a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/PathScopeProvider.java +++ b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/PathScopeProvider.java @@ -43,8 +43,8 @@ * } * * - * @see org.apache.maven.api.PathScope - * @see org.apache.maven.api.spi.ExtensibleEnumProvider + * @see PathScope + * @see ExtensibleEnumProvider * @since 4.0.0 */ @Experimental diff --git a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/ProjectScopeProvider.java b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/ProjectScopeProvider.java index bcd071e9be48..8c58f506737c 100644 --- a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/ProjectScopeProvider.java +++ b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/ProjectScopeProvider.java @@ -40,8 +40,8 @@ * } * * - * @see org.apache.maven.api.ProjectScope - * @see org.apache.maven.api.spi.ExtensibleEnumProvider + * @see ProjectScope + * @see ExtensibleEnumProvider * @since 4.0.0 */ @Experimental diff --git a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/TypeProvider.java b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/TypeProvider.java index 1aa5bf78d81b..691d612a1ef7 100644 --- a/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/TypeProvider.java +++ b/api/maven-api-spi/src/main/java/org/apache/maven/api/spi/TypeProvider.java @@ -45,8 +45,8 @@ * } * * - * @see org.apache.maven.api.Type - * @see org.apache.maven.api.spi.ExtensibleEnumProvider + * @see Type + * @see ExtensibleEnumProvider * @since 4.0.0 */ @Experimental diff --git a/compat/maven-builder-support/src/main/java/org/apache/maven/building/DefaultProblem.java b/compat/maven-builder-support/src/main/java/org/apache/maven/building/DefaultProblem.java index 19beeb576e3b..af1a2d447fa6 100644 --- a/compat/maven-builder-support/src/main/java/org/apache/maven/building/DefaultProblem.java +++ b/compat/maven-builder-support/src/main/java/org/apache/maven/building/DefaultProblem.java @@ -46,7 +46,7 @@ class DefaultProblem implements Problem { * * @param message The message describing the problem, may be {@code null}. * @param severity The severity level of the problem, may be {@code null} to default to - * {@link org.apache.maven.building.Problem.Severity#ERROR}. + * {@link Problem.Severity#ERROR}. * @param source A hint about the source of the problem like a file path, may be {@code null}. * @param lineNumber The one-based index of the line containing the problem or {@code -1} if unknown. * @param columnNumber The one-based index of the column containing the problem or {@code -1} if unknown. diff --git a/compat/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java b/compat/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java index d36824bc60d6..128dfa0f70a3 100644 --- a/compat/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java +++ b/compat/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java @@ -33,6 +33,7 @@ import java.nio.file.Files; import java.nio.file.Path; import java.util.ArrayList; +import java.util.Arrays; import java.util.Collections; import java.util.HashSet; import java.util.LinkedHashMap; @@ -954,7 +955,7 @@ private void encryption(CliRequest cliRequest) throws Exception { passwd = String.copyValueOf(password); // Sun/Oracle advises to empty the char array - java.util.Arrays.fill(password, ' '); + Arrays.fill(password, ' '); } } System.out.println(dispatcher.encrypt(passwd, null)); diff --git a/compat/maven-embedder/src/main/java/org/apache/maven/cli/event/ExecutionEventLogger.java b/compat/maven-embedder/src/main/java/org/apache/maven/cli/event/ExecutionEventLogger.java index 2aea29d6cb18..706ee0e402dd 100644 --- a/compat/maven-embedder/src/main/java/org/apache/maven/cli/event/ExecutionEventLogger.java +++ b/compat/maven-embedder/src/main/java/org/apache/maven/cli/event/ExecutionEventLogger.java @@ -32,6 +32,7 @@ import org.apache.maven.execution.ExecutionEvent; import org.apache.maven.execution.MavenExecutionResult; import org.apache.maven.execution.MavenSession; +import org.apache.maven.logging.api.LogLevelRecorder; import org.apache.maven.plugin.MojoExecution; import org.apache.maven.plugin.descriptor.MojoDescriptor; import org.apache.maven.project.MavenProject; @@ -146,7 +147,7 @@ public void sessionEnded(ExecutionEvent event) { ILoggerFactory iLoggerFactory = LoggerFactory.getILoggerFactory(); - if (iLoggerFactory instanceof org.apache.maven.logging.api.LogLevelRecorder recorder + if (iLoggerFactory instanceof LogLevelRecorder recorder && recorder.hasReachedMaxLevel()) { event.getSession() .getResult() diff --git a/compat/maven-embedder/src/main/java/org/apache/maven/cli/props/MavenPropertiesLoader.java b/compat/maven-embedder/src/main/java/org/apache/maven/cli/props/MavenPropertiesLoader.java index 7061445bc68b..806f82724799 100644 --- a/compat/maven-embedder/src/main/java/org/apache/maven/cli/props/MavenPropertiesLoader.java +++ b/compat/maven-embedder/src/main/java/org/apache/maven/cli/props/MavenPropertiesLoader.java @@ -23,6 +23,7 @@ import java.nio.file.Path; import java.util.Enumeration; import java.util.Map; +import java.util.Properties; import java.util.StringTokenizer; import java.util.function.UnaryOperator; @@ -37,7 +38,7 @@ public class MavenPropertiesLoader { "maven.override."; // prefix that marks that system property should override defaults. public static void loadProperties( - java.util.Properties properties, Path path, UnaryOperator callback, boolean escape) + Properties properties, Path path, UnaryOperator callback, boolean escape) throws IOException { MavenProperties sp = new MavenProperties(false); if (Files.exists(path)) { diff --git a/compat/maven-embedder/src/main/java/org/fusesource/jansi/Ansi.java b/compat/maven-embedder/src/main/java/org/fusesource/jansi/Ansi.java index f861da0098b0..8208feb3d088 100644 --- a/compat/maven-embedder/src/main/java/org/fusesource/jansi/Ansi.java +++ b/compat/maven-embedder/src/main/java/org/fusesource/jansi/Ansi.java @@ -20,6 +20,8 @@ import java.util.ArrayList; +import org.apache.maven.jline.MessageUtils; + /** * Provides a fluent API for generating * ANSI escape sequences. @@ -161,7 +163,7 @@ public interface Consumer { } public static boolean isEnabled() { - return org.apache.maven.jline.MessageUtils.isColorEnabled() && org.jline.jansi.Ansi.isEnabled(); + return MessageUtils.isColorEnabled() && org.jline.jansi.Ansi.isEnabled(); } public static void setEnabled(final boolean flag) { diff --git a/compat/maven-embedder/src/test/java/org/apache/maven/cli/props/MavenPropertiesTest.java b/compat/maven-embedder/src/test/java/org/apache/maven/cli/props/MavenPropertiesTest.java index 07aa1e8ba664..843bb018f75f 100644 --- a/compat/maven-embedder/src/test/java/org/apache/maven/cli/props/MavenPropertiesTest.java +++ b/compat/maven-embedder/src/test/java/org/apache/maven/cli/props/MavenPropertiesTest.java @@ -26,6 +26,7 @@ import java.io.StringWriter; import java.util.List; import java.util.Map; +import java.util.Properties; import org.apache.maven.impl.model.DefaultInterpolator; import org.junit.jupiter.api.BeforeEach; @@ -101,7 +102,7 @@ public void testSpaces() throws Exception { + "dblbackslash=\\\\\n" + " \n"; - java.util.Properties props1 = new java.util.Properties(); + Properties props1 = new Properties(); props1.load(new StringReader(config)); MavenProperties props2 = new MavenProperties(); @@ -168,7 +169,7 @@ public void testConfigInterpolation() throws IOException { String config = "a=$\\\\\\\\{var}\n" + "ab=${a}b\n" + "abc=${ab}c"; Map expected = Map.of("a", "$\\{var}", "ab", "$\\{var}b", "abc", "$\\{var}bc"); - java.util.Properties props1 = new java.util.Properties(); + Properties props1 = new Properties(); props1.load(new StringReader(config)); new DefaultInterpolator().performSubstitution((Map) props1, null, true); assertEquals(expected, props1); @@ -327,7 +328,7 @@ public void testMultiValueEscaping() throws IOException { pw.println(" cantaloupe, watermelon, \\"); pw.println(" kiwi, mango"); - java.util.Properties p = new java.util.Properties(); + Properties p = new Properties(); p.load(new StringReader(sw.toString())); Object o24 = p.getProperty("fruits"); assertEquals("apple, banana, pear, cantaloupe, watermelon, kiwi, mango", o24); diff --git a/compat/maven-model-builder/src/main/java/org/apache/maven/model/resolution/ModelResolver.java b/compat/maven-model-builder/src/main/java/org/apache/maven/model/resolution/ModelResolver.java index 9f76ebb2e3ca..f15165d26744 100644 --- a/compat/maven-model-builder/src/main/java/org/apache/maven/model/resolution/ModelResolver.java +++ b/compat/maven-model-builder/src/main/java/org/apache/maven/model/resolution/ModelResolver.java @@ -47,7 +47,7 @@ public interface ModelResolver { /** * Tries to resolve the POM for the specified parent coordinates possibly updating {@code parent}. *

- * Unlike the {@link #resolveModel(java.lang.String, java.lang.String, java.lang.String)} method, this method + * Unlike the {@link #resolveModel(String, String, String)} method, this method * supports version ranges and updates the given {@code parent} instance to match the returned {@code ModelSource}. * If {@code parent} declares a version range, the version corresponding to the returned {@code ModelSource} will * be set on the given {@code parent}. @@ -67,7 +67,7 @@ public interface ModelResolver { /** * Tries to resolve the POM for the specified dependency coordinates possibly updating {@code dependency}. *

- * Unlike the {@link #resolveModel(java.lang.String, java.lang.String, java.lang.String)} method, this method + * Unlike the {@link #resolveModel(String, String, String)} method, this method * supports version ranges and updates the given {@code dependency} instance to match the returned * {@code ModelSource}. If {@code dependency} declares a version range, the version corresponding to the returned * {@code ModelSource} will be set on the given {@code dependency}. diff --git a/compat/maven-model-builder/src/test/java/org/apache/maven/model/interpolation/AbstractModelInterpolatorTest.java b/compat/maven-model-builder/src/test/java/org/apache/maven/model/interpolation/AbstractModelInterpolatorTest.java index 6d0fa698f193..667aed6ffa13 100644 --- a/compat/maven-model-builder/src/test/java/org/apache/maven/model/interpolation/AbstractModelInterpolatorTest.java +++ b/compat/maven-model-builder/src/test/java/org/apache/maven/model/interpolation/AbstractModelInterpolatorTest.java @@ -30,6 +30,10 @@ import java.util.Properties; import java.util.TimeZone; +import org.apache.maven.api.model.Build; +import org.apache.maven.api.model.Organization; +import org.apache.maven.api.model.Repository; +import org.apache.maven.api.model.Scm; import org.apache.maven.model.Dependency; import org.apache.maven.model.Model; import org.apache.maven.model.Resource; @@ -136,7 +140,7 @@ public void testDefaultBuildTimestampFormatWithLocalTimeZoneMidnightRollover() { @Test public void testShouldNotThrowExceptionOnReferenceToNonExistentValue() throws Exception { Model model = new Model(org.apache.maven.api.model.Model.newBuilder() - .scm(org.apache.maven.api.model.Scm.newBuilder() + .scm(Scm.newBuilder() .connection("${test}/somepath") .build()) .build()); @@ -153,7 +157,7 @@ public void testShouldNotThrowExceptionOnReferenceToNonExistentValue() throws Ex @Test public void testShouldThrowExceptionOnRecursiveScmConnectionReference() throws Exception { var model = new Model(org.apache.maven.api.model.Model.newBuilder() - .scm(org.apache.maven.api.model.Scm.newBuilder() + .scm(Scm.newBuilder() .connection("${project.scm.connection}/somepath") .build()) .build()); @@ -170,7 +174,7 @@ public void testShouldNotThrowExceptionOnReferenceToValueContainingNakedExpressi Map props = new HashMap<>(); props.put("test", "test"); Model model = new Model(org.apache.maven.api.model.Model.newBuilder() - .scm(org.apache.maven.api.model.Scm.newBuilder() + .scm(Scm.newBuilder() .connection("${test}/somepath") .build()) .properties(props) @@ -192,7 +196,7 @@ public void shouldInterpolateOrganizationNameCorrectly() throws Exception { Model model = new Model(org.apache.maven.api.model.Model.newBuilder() .name("${project.organization.name} Tools") - .organization(org.apache.maven.api.model.Organization.newBuilder() + .organization(Organization.newBuilder() .name(orgName) .build()) .build()); @@ -265,7 +269,7 @@ public void testBasedir() throws Exception { Model model = new Model(org.apache.maven.api.model.Model.newBuilder() .version("3.8.1") .artifactId("foo") - .repositories(Collections.singletonList(org.apache.maven.api.model.Repository.newBuilder() + .repositories(Collections.singletonList(Repository.newBuilder() .url("file://localhost/${basedir}/temp-repo") .build())) .build()); @@ -285,7 +289,7 @@ public void testBaseUri() throws Exception { Model model = new Model(org.apache.maven.api.model.Model.newBuilder() .version("3.8.1") .artifactId("foo") - .repositories(Collections.singletonList(org.apache.maven.api.model.Repository.newBuilder() + .repositories(Collections.singletonList(Repository.newBuilder() .url("${project.baseUri}/temp-repo") .build())) .build()); @@ -359,7 +363,7 @@ public void expressionThatEvaluatesToNullReturnsTheLiteralString() throws Except @Test public void shouldInterpolateSourceDirectoryReferencedFromResourceDirectoryCorrectly() throws Exception { Model model = new Model(org.apache.maven.api.model.Model.newBuilder() - .build(org.apache.maven.api.model.Build.newBuilder() + .build(Build.newBuilder() .sourceDirectory("correct") .resources(List.of(org.apache.maven.api.model.Resource.newBuilder() .directory("${project.build.sourceDirectory}") diff --git a/compat/maven-model/src/main/java/org/apache/maven/model/InputLocation.java b/compat/maven-model/src/main/java/org/apache/maven/model/InputLocation.java index 82bfc9162b8b..6641ae8bfbba 100644 --- a/compat/maven-model/src/main/java/org/apache/maven/model/InputLocation.java +++ b/compat/maven-model/src/main/java/org/apache/maven/model/InputLocation.java @@ -18,6 +18,11 @@ */ package org.apache.maven.model; +import java.io.Serializable; +import java.util.Collection; +import java.util.Iterator; +import java.util.LinkedHashMap; +import java.util.Map; import java.util.stream.Collectors; /** @@ -26,7 +31,7 @@ * @version $Revision$ $Date$ */ @SuppressWarnings("all") -public final class InputLocation implements java.io.Serializable, Cloneable, InputLocationTracker { +public final class InputLocation implements Serializable, Cloneable, InputLocationTracker { // --------------------------/ // - Class/Member Variables -/ @@ -52,7 +57,7 @@ public final class InputLocation implements java.io.Serializable, Cloneable, Inp /** * Field locations. */ - private java.util.Map locations; + private Map locations; /** * Field location. @@ -106,7 +111,7 @@ public InputLocation clone() { InputLocation copy = (InputLocation) super.clone(); if (copy.locations != null) { - copy.locations = new java.util.LinkedHashMap(copy.locations); + copy.locations = new LinkedHashMap(copy.locations); } return copy; @@ -162,7 +167,7 @@ public InputLocation getLocation(Object key) { * * @return Map */ - public java.util.Map getLocations() { + public Map getLocations() { return locations; } // -- java.util.Map getLocations() @@ -198,7 +203,7 @@ public void setLocation(Object key, InputLocation location) { public void setOtherLocation(Object key, InputLocation location) { if (location != null) { if (this.locations == null) { - this.locations = new java.util.LinkedHashMap<>(); + this.locations = new LinkedHashMap<>(); } this.locations.put(key, location); } @@ -260,15 +265,15 @@ public static InputLocation merge(InputLocation target, InputLocation source, bo InputLocation result = new InputLocation(target.getLineNumber(), target.getColumnNumber(), target.getSource()); - java.util.Map locations; - java.util.Map sourceLocations = source.getLocations(); - java.util.Map targetLocations = target.getLocations(); + Map locations; + Map sourceLocations = source.getLocations(); + Map targetLocations = target.getLocations(); if (sourceLocations == null) { locations = targetLocations; } else if (targetLocations == null) { locations = sourceLocations; } else { - locations = new java.util.LinkedHashMap(); + locations = new LinkedHashMap(); locations.putAll(sourceDominant ? targetLocations : sourceLocations); locations.putAll(sourceDominant ? sourceLocations : targetLocations); } @@ -286,7 +291,7 @@ public static InputLocation merge(InputLocation target, InputLocation source, bo * @return InputLocation */ public static InputLocation merge( - InputLocation target, InputLocation source, java.util.Collection indices) { + InputLocation target, InputLocation source, Collection indices) { if (source == null) { return target; } else if (target == null) { @@ -295,16 +300,16 @@ public static InputLocation merge( InputLocation result = new InputLocation(target.getLineNumber(), target.getColumnNumber(), target.getSource()); - java.util.Map locations; - java.util.Map sourceLocations = source.getLocations(); - java.util.Map targetLocations = target.getLocations(); + Map locations; + Map sourceLocations = source.getLocations(); + Map targetLocations = target.getLocations(); if (sourceLocations == null) { locations = targetLocations; } else if (targetLocations == null) { locations = sourceLocations; } else { - locations = new java.util.LinkedHashMap<>(); - for (java.util.Iterator it = indices.iterator(); it.hasNext(); ) { + locations = new LinkedHashMap<>(); + for (Iterator it = indices.iterator(); it.hasNext(); ) { InputLocation location; Integer index = it.next(); if (index.intValue() < 0) { @@ -325,7 +330,7 @@ public static InputLocation merge( * * @param locations */ - public void setLocations(java.util.Map locations) { + public void setLocations(Map locations) { this.locations = locations; } // -- void setLocations( java.util.Map ) diff --git a/compat/maven-model/src/main/java/org/apache/maven/model/InputSource.java b/compat/maven-model/src/main/java/org/apache/maven/model/InputSource.java index 1bd81e925ee0..59135ef8cccd 100644 --- a/compat/maven-model/src/main/java/org/apache/maven/model/InputSource.java +++ b/compat/maven-model/src/main/java/org/apache/maven/model/InputSource.java @@ -18,13 +18,15 @@ */ package org.apache.maven.model; +import java.io.Serializable; + /** * Class InputSource. * * @version $Revision$ $Date$ */ @SuppressWarnings("all") -public class InputSource implements java.io.Serializable, Cloneable { +public class InputSource implements Serializable, Cloneable { // --------------------------/ // - Class/Member Variables -/ diff --git a/compat/maven-model/src/main/java/org/apache/maven/model/merge/ModelMerger.java b/compat/maven-model/src/main/java/org/apache/maven/model/merge/ModelMerger.java index ad90dac4b559..f5ae78bdae6e 100644 --- a/compat/maven-model/src/main/java/org/apache/maven/model/merge/ModelMerger.java +++ b/compat/maven-model/src/main/java/org/apache/maven/model/merge/ModelMerger.java @@ -19,6 +19,7 @@ package org.apache.maven.model.merge; import java.io.ObjectStreamException; +import java.io.Serializable; import java.util.AbstractList; import java.util.ArrayList; import java.util.Collection; @@ -2394,7 +2395,7 @@ private static List merge(List tgt, List src, KeyComputer comput * Merging list * @param */ - private static class MergingList extends AbstractList implements java.io.Serializable { + private static class MergingList extends AbstractList implements Serializable { private final KeyComputer keyComputer; private Map map; diff --git a/compat/maven-model/src/test/java/org/apache/maven/model/v4/Xpp3DomPerfTest.java b/compat/maven-model/src/test/java/org/apache/maven/model/v4/Xpp3DomPerfTest.java index 090292cffa35..55fe7796adfe 100644 --- a/compat/maven-model/src/test/java/org/apache/maven/model/v4/Xpp3DomPerfTest.java +++ b/compat/maven-model/src/test/java/org/apache/maven/model/v4/Xpp3DomPerfTest.java @@ -86,7 +86,7 @@ public int readWithStax(AdditionState state) throws IOException, XMLStreamExcept *

main.

* * @param args a {@link String} object. - * @throws org.openjdk.jmh.runner.RunnerException if any. + * @throws RunnerException if any. */ public static void main(String... args) throws RunnerException { Options opts = new OptionsBuilder().forks(1).build(); diff --git a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/AbstractMojo.java b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/AbstractMojo.java index f9ea24d94ee6..c81d3c1b00ea 100644 --- a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/AbstractMojo.java +++ b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/AbstractMojo.java @@ -163,7 +163,7 @@ public void setLog(Log log) { * gets injected by the Plexus container after the mojo has been constructed. Therefore, simply call this * method directly whenever you need the logger, it is fast enough and needs no caching. * - * @see org.apache.maven.plugin.Mojo#getLog() + * @see Mojo#getLog() * @deprecated Use SLF4J directly */ @Deprecated diff --git a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Execution.java b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Execution.java index 7ef46425eb9c..bdd195c05424 100644 --- a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Execution.java +++ b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Execution.java @@ -18,13 +18,17 @@ */ package org.apache.maven.plugin.lifecycle; +import java.io.Serializable; +import java.util.ArrayList; +import java.util.List; + /** * A set of goals to execute. * * @version $Revision$ $Date$ */ @SuppressWarnings("all") -public class Execution implements java.io.Serializable { +public class Execution implements Serializable { // --------------------------/ // - Class/Member Variables -/ @@ -38,7 +42,7 @@ public class Execution implements java.io.Serializable { /** * Field goals. */ - private java.util.List goals; + private List goals; // -----------/ // - Methods -/ @@ -67,9 +71,9 @@ public Object getConfiguration() { * * @return List */ - public java.util.List getGoals() { + public List getGoals() { if (this.goals == null) { - this.goals = new java.util.ArrayList(); + this.goals = new ArrayList(); } return this.goals; @@ -98,7 +102,7 @@ public void setConfiguration(Object configuration) { * * @param goals a goals object. */ - public void setGoals(java.util.List goals) { + public void setGoals(List goals) { this.goals = goals; } // -- void setGoals( java.util.List ) } diff --git a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Lifecycle.java b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Lifecycle.java index df6c225729bf..b6b463969422 100644 --- a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Lifecycle.java +++ b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Lifecycle.java @@ -18,6 +18,10 @@ */ package org.apache.maven.plugin.lifecycle; +import java.io.Serializable; +import java.util.ArrayList; +import java.util.List; + /** * * A custom lifecycle mapping definition. @@ -26,7 +30,7 @@ * @version $Revision$ $Date$ */ @SuppressWarnings("all") -public class Lifecycle implements java.io.Serializable { +public class Lifecycle implements Serializable { // --------------------------/ // - Class/Member Variables -/ @@ -41,7 +45,7 @@ public class Lifecycle implements java.io.Serializable { /** * Field phases. */ - private java.util.List phases; + private List phases; // -----------/ // - Methods -/ @@ -71,9 +75,9 @@ public String getId() { * * @return List */ - public java.util.List getPhases() { + public List getPhases() { if (this.phases == null) { - this.phases = new java.util.ArrayList(); + this.phases = new ArrayList(); } return this.phases; @@ -103,7 +107,7 @@ public void setId(String id) { * * @param phases a phases object. */ - public void setPhases(java.util.List phases) { + public void setPhases(List phases) { this.phases = phases; } // -- void setPhases( java.util.List ) } diff --git a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/LifecycleConfiguration.java b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/LifecycleConfiguration.java index bde244cf110d..2f9ff944e843 100644 --- a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/LifecycleConfiguration.java +++ b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/LifecycleConfiguration.java @@ -18,13 +18,17 @@ */ package org.apache.maven.plugin.lifecycle; +import java.io.Serializable; +import java.util.ArrayList; +import java.util.List; + /** * Root element of the lifecycle.xml file. * * @version $Revision$ $Date$ */ @SuppressWarnings("all") -public class LifecycleConfiguration implements java.io.Serializable { +public class LifecycleConfiguration implements Serializable { // --------------------------/ // - Class/Member Variables -/ @@ -33,7 +37,7 @@ public class LifecycleConfiguration implements java.io.Serializable { /** * Field lifecycles. */ - private java.util.List lifecycles; + private List lifecycles; /** * Field modelEncoding. @@ -58,9 +62,9 @@ public void addLifecycle(Lifecycle lifecycle) { * * @return List */ - public java.util.List getLifecycles() { + public List getLifecycles() { if (this.lifecycles == null) { - this.lifecycles = new java.util.ArrayList(); + this.lifecycles = new ArrayList(); } return this.lifecycles; @@ -89,7 +93,7 @@ public void removeLifecycle(Lifecycle lifecycle) { * * @param lifecycles a lifecycles object. */ - public void setLifecycles(java.util.List lifecycles) { + public void setLifecycles(List lifecycles) { this.lifecycles = lifecycles; } // -- void setLifecycles( java.util.List ) diff --git a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Phase.java b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Phase.java index e5731a1936cb..f8e3141a9930 100644 --- a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Phase.java +++ b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/lifecycle/Phase.java @@ -18,13 +18,17 @@ */ package org.apache.maven.plugin.lifecycle; +import java.io.Serializable; +import java.util.ArrayList; +import java.util.List; + /** * A phase mapping definition. * * @version $Revision$ $Date$ */ @SuppressWarnings("all") -public class Phase implements java.io.Serializable { +public class Phase implements Serializable { // --------------------------/ // - Class/Member Variables -/ @@ -38,7 +42,7 @@ public class Phase implements java.io.Serializable { /** * Field executions. */ - private java.util.List executions; + private List executions; /** * Configuration to pass to all goals run in this phase. @@ -72,9 +76,9 @@ public Object getConfiguration() { * * @return List */ - public java.util.List getExecutions() { + public List getExecutions() { if (this.executions == null) { - this.executions = new java.util.ArrayList(); + this.executions = new ArrayList(); } return this.executions; @@ -113,7 +117,7 @@ public void setConfiguration(Object configuration) { * * @param executions a executions object. */ - public void setExecutions(java.util.List executions) { + public void setExecutions(List executions) { this.executions = executions; } // -- void setExecutions( java.util.List ) diff --git a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/logging/SystemStreamLog.java b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/logging/SystemStreamLog.java index bff8bf3295c4..02cd7fb9e3c9 100644 --- a/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/logging/SystemStreamLog.java +++ b/compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/logging/SystemStreamLog.java @@ -30,77 +30,77 @@ @Deprecated public class SystemStreamLog implements Log { /** - * @see org.apache.maven.plugin.logging.Log#debug(java.lang.CharSequence) + * @see Log#debug(CharSequence) */ public void debug(CharSequence content) { print("debug", content); } /** - * @see org.apache.maven.plugin.logging.Log#debug(java.lang.CharSequence, java.lang.Throwable) + * @see Log#debug(CharSequence, Throwable) */ public void debug(CharSequence content, Throwable error) { print("debug", content, error); } /** - * @see org.apache.maven.plugin.logging.Log#debug(java.lang.Throwable) + * @see Log#debug(Throwable) */ public void debug(Throwable error) { print("debug", error); } /** - * @see org.apache.maven.plugin.logging.Log#info(java.lang.CharSequence) + * @see Log#info(CharSequence) */ public void info(CharSequence content) { print("info", content); } /** - * @see org.apache.maven.plugin.logging.Log#info(java.lang.CharSequence, java.lang.Throwable) + * @see Log#info(CharSequence, Throwable) */ public void info(CharSequence content, Throwable error) { print("info", content, error); } /** - * @see org.apache.maven.plugin.logging.Log#info(java.lang.Throwable) + * @see Log#info(Throwable) */ public void info(Throwable error) { print("info", error); } /** - * @see org.apache.maven.plugin.logging.Log#warn(java.lang.CharSequence) + * @see Log#warn(CharSequence) */ public void warn(CharSequence content) { print("warn", content); } /** - * @see org.apache.maven.plugin.logging.Log#warn(java.lang.CharSequence, java.lang.Throwable) + * @see Log#warn(CharSequence, Throwable) */ public void warn(CharSequence content, Throwable error) { print("warn", content, error); } /** - * @see org.apache.maven.plugin.logging.Log#warn(java.lang.Throwable) + * @see Log#warn(Throwable) */ public void warn(Throwable error) { print("warn", error); } /** - * @see org.apache.maven.plugin.logging.Log#error(java.lang.CharSequence) + * @see Log#error(CharSequence) */ public void error(CharSequence content) { System.err.println("[error] " + content.toString()); } /** - * @see org.apache.maven.plugin.logging.Log#error(java.lang.CharSequence, java.lang.Throwable) + * @see Log#error(CharSequence, Throwable) */ public void error(CharSequence content, Throwable error) { StringWriter sWriter = new StringWriter(); @@ -112,7 +112,7 @@ public void error(CharSequence content, Throwable error) { } /** - * @see org.apache.maven.plugin.logging.Log#error(java.lang.Throwable) + * @see Log#error(Throwable) */ public void error(Throwable error) { StringWriter sWriter = new StringWriter(); @@ -124,7 +124,7 @@ public void error(Throwable error) { } /** - * @see org.apache.maven.plugin.logging.Log#isDebugEnabled() + * @see Log#isDebugEnabled() */ public boolean isDebugEnabled() { // TODO Not sure how best to set these for this implementation... @@ -132,21 +132,21 @@ public boolean isDebugEnabled() { } /** - * @see org.apache.maven.plugin.logging.Log#isInfoEnabled() + * @see Log#isInfoEnabled() */ public boolean isInfoEnabled() { return true; } /** - * @see org.apache.maven.plugin.logging.Log#isWarnEnabled() + * @see Log#isWarnEnabled() */ public boolean isWarnEnabled() { return true; } /** - * @see org.apache.maven.plugin.logging.Log#isErrorEnabled() + * @see Log#isErrorEnabled() */ public boolean isErrorEnabled() { return true; diff --git a/compat/maven-repository-metadata/src/main/java/org/apache/maven/artifact/repository/metadata/io/xpp3/MetadataXpp3Writer.java b/compat/maven-repository-metadata/src/main/java/org/apache/maven/artifact/repository/metadata/io/xpp3/MetadataXpp3Writer.java index fbed96f67931..28339e485a63 100644 --- a/compat/maven-repository-metadata/src/main/java/org/apache/maven/artifact/repository/metadata/io/xpp3/MetadataXpp3Writer.java +++ b/compat/maven-repository-metadata/src/main/java/org/apache/maven/artifact/repository/metadata/io/xpp3/MetadataXpp3Writer.java @@ -58,9 +58,9 @@ public void setFileComment(String fileComment) { * * @param writer a writer object * @param metadata a Metadata object - * @throws java.io.IOException java.io.IOException if any + * @throws IOException java.io.IOException if any */ - public void write(Writer writer, Metadata metadata) throws java.io.IOException { + public void write(Writer writer, Metadata metadata) throws IOException { try { delegate.write(writer, metadata.getDelegate()); } catch (XMLStreamException e) { @@ -73,9 +73,9 @@ public void write(Writer writer, Metadata metadata) throws java.io.IOException { * * @param stream a stream object * @param metadata a Metadata object - * @throws java.io.IOException java.io.IOException if any + * @throws IOException java.io.IOException if any */ - public void write(OutputStream stream, Metadata metadata) throws java.io.IOException { + public void write(OutputStream stream, Metadata metadata) throws IOException { try { delegate.write(stream, metadata.getDelegate()); } catch (XMLStreamException e) { diff --git a/compat/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultModelResolver.java b/compat/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultModelResolver.java index ef2e32469d55..87082a6a145e 100644 --- a/compat/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultModelResolver.java +++ b/compat/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultModelResolver.java @@ -109,7 +109,7 @@ private DefaultModelResolver(DefaultModelResolver original) { } @Override - public void addRepository(org.apache.maven.model.Repository repository) throws InvalidRepositoryException { + public void addRepository(Repository repository) throws InvalidRepositoryException { addRepository(repository, false); } diff --git a/compat/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/scopes/Maven4ScopeManagerConfiguration.java b/compat/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/scopes/Maven4ScopeManagerConfiguration.java index a38833076355..a1416e6844ce 100644 --- a/compat/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/scopes/Maven4ScopeManagerConfiguration.java +++ b/compat/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/scopes/Maven4ScopeManagerConfiguration.java @@ -32,6 +32,7 @@ import org.eclipse.aether.impl.scope.InternalScopeManager; import org.eclipse.aether.impl.scope.ScopeManagerConfiguration; import org.eclipse.aether.internal.impl.scope.ScopeManagerDump; +import org.eclipse.aether.scope.ResolutionScope; import static org.eclipse.aether.impl.scope.BuildScopeQuery.all; import static org.eclipse.aether.impl.scope.BuildScopeQuery.byBuildPath; @@ -127,7 +128,7 @@ public Collection buildDependencyScope MavenArtifactProperties.LOCAL_PATH)); // == sanity check - if (result.size() != org.apache.maven.api.DependencyScope.values().length - 1) { // sans "undefined" + if (result.size() != DependencyScope.values().length - 1) { // sans "undefined" throw new IllegalStateException("Maven4 API dependency scope mismatch"); } @@ -135,7 +136,7 @@ public Collection buildDependencyScope } @Override - public Collection buildResolutionScopes( + public Collection buildResolutionScopes( InternalScopeManager internalScopeManager) { Collection allDependencyScopes = internalScopeManager.getDependencyScopeUniverse(); @@ -145,7 +146,7 @@ public Collection buildResolutionScope .getDependencyScope(DependencyScope.SYSTEM.id()) .orElse(null); - ArrayList result = new ArrayList<>(); + ArrayList result = new ArrayList<>(); result.add(internalScopeManager.createResolutionScope( RS_NONE, InternalScopeManager.Mode.REMOVE, diff --git a/compat/maven-toolchain-builder/src/test/java/org/apache/maven/toolchain/building/DefaultToolchainsBuilderTest.java b/compat/maven-toolchain-builder/src/test/java/org/apache/maven/toolchain/building/DefaultToolchainsBuilderTest.java index 8e3fb5d467ea..270d455f470b 100644 --- a/compat/maven-toolchain-builder/src/test/java/org/apache/maven/toolchain/building/DefaultToolchainsBuilderTest.java +++ b/compat/maven-toolchain-builder/src/test/java/org/apache/maven/toolchain/building/DefaultToolchainsBuilderTest.java @@ -256,7 +256,7 @@ void testEnvironmentVariablesAreInterpolated() throws Exception { .get(0) .getProvides() .get("key")); - org.codehaus.plexus.util.xml.Xpp3Dom toolchainConfiguration = (org.codehaus.plexus.util.xml.Xpp3Dom) + Xpp3Dom toolchainConfiguration = (Xpp3Dom) result.getEffectiveToolchains().getToolchains().get(0).getConfiguration(); assertEquals( interpolatedValue, toolchainConfiguration.getChild("jdkHome").getValue()); diff --git a/impl/maven-cli/src/main/java/org/apache/maven/cling/event/ExecutionEventLogger.java b/impl/maven-cli/src/main/java/org/apache/maven/cling/event/ExecutionEventLogger.java index 3d506a605060..4c3250354f7b 100644 --- a/impl/maven-cli/src/main/java/org/apache/maven/cling/event/ExecutionEventLogger.java +++ b/impl/maven-cli/src/main/java/org/apache/maven/cling/event/ExecutionEventLogger.java @@ -38,6 +38,7 @@ import org.apache.maven.execution.ExecutionEvent; import org.apache.maven.execution.MavenExecutionResult; import org.apache.maven.execution.MavenSession; +import org.apache.maven.logging.api.LogLevelRecorder; import org.apache.maven.plugin.MojoExecution; import org.apache.maven.plugin.descriptor.MojoDescriptor; import org.apache.maven.project.MavenProject; @@ -151,7 +152,7 @@ public void sessionEnded(ExecutionEvent event) { ILoggerFactory iLoggerFactory = LoggerFactory.getILoggerFactory(); - if (iLoggerFactory instanceof org.apache.maven.logging.api.LogLevelRecorder recorder + if (iLoggerFactory instanceof LogLevelRecorder recorder && recorder.hasReachedMaxLevel()) { event.getSession() .getResult() diff --git a/impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/mvn/CommonsCliMavenOptions.java b/impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/mvn/CommonsCliMavenOptions.java index 2632d71e48c2..82b004d44b1c 100644 --- a/impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/mvn/CommonsCliMavenOptions.java +++ b/impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/mvn/CommonsCliMavenOptions.java @@ -26,6 +26,7 @@ import org.apache.commons.cli.CommandLine; import org.apache.commons.cli.Option; +import org.apache.commons.cli.Options; import org.apache.commons.cli.ParseException; import org.apache.maven.api.cli.mvn.MavenOptions; import org.apache.maven.api.services.Interpolator; @@ -284,7 +285,7 @@ protected static class CLIManager extends CommonsCliOptions.CLIManager { public static final String AT_FILE = "af"; @Override - protected void prepareOptions(org.apache.commons.cli.Options options) { + protected void prepareOptions(Options options) { super.prepareOptions(options); options.addOption(Option.builder(ALTERNATE_POM_FILE) .longOpt("file") diff --git a/impl/maven-cli/src/main/java/org/apache/maven/cling/props/MavenPropertiesLoader.java b/impl/maven-cli/src/main/java/org/apache/maven/cling/props/MavenPropertiesLoader.java index 469b22938a33..fcfb734522b5 100644 --- a/impl/maven-cli/src/main/java/org/apache/maven/cling/props/MavenPropertiesLoader.java +++ b/impl/maven-cli/src/main/java/org/apache/maven/cling/props/MavenPropertiesLoader.java @@ -23,6 +23,7 @@ import java.nio.file.Path; import java.util.Enumeration; import java.util.Map; +import java.util.Properties; import java.util.StringTokenizer; import java.util.function.UnaryOperator; @@ -36,7 +37,7 @@ public class MavenPropertiesLoader { "maven.override."; // prefix that marks that system property should override defaults. public static void loadProperties( - java.util.Properties properties, Path path, UnaryOperator callback, boolean escape) + Properties properties, Path path, UnaryOperator callback, boolean escape) throws IOException { MavenProperties sp = new MavenProperties(false); if (Files.exists(path)) { diff --git a/impl/maven-core/src/main/java/org/apache/maven/bridge/MavenRepositorySystem.java b/impl/maven-core/src/main/java/org/apache/maven/bridge/MavenRepositorySystem.java index 1617b19c3714..2ba3c5523d11 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/bridge/MavenRepositorySystem.java +++ b/impl/maven-core/src/main/java/org/apache/maven/bridge/MavenRepositorySystem.java @@ -54,11 +54,14 @@ import org.apache.maven.execution.MavenExecutionRequest; import org.apache.maven.model.Dependency; import org.apache.maven.model.Plugin; +import org.apache.maven.model.Repository; +import org.apache.maven.model.RepositoryPolicy; import org.apache.maven.repository.Proxy; import org.apache.maven.settings.Mirror; import org.eclipse.aether.RepositorySystemSession; import org.eclipse.aether.repository.AuthenticationContext; import org.eclipse.aether.repository.AuthenticationSelector; +import org.eclipse.aether.repository.MirrorSelector; import org.eclipse.aether.repository.ProxySelector; import org.eclipse.aether.repository.RemoteRepository; @@ -173,7 +176,7 @@ public void injectMirror(List repositories, List mir private Mirror getMirror(RepositorySystemSession session, ArtifactRepository repository) { if (session != null) { - org.eclipse.aether.repository.MirrorSelector selector = session.getMirrorSelector(); + MirrorSelector selector = session.getMirrorSelector(); if (selector != null) { RemoteRepository repo = selector.getMirror(RepositoryUtils.toRepo(repository)); if (repo != null) { @@ -296,9 +299,9 @@ private ArtifactRepositoryLayout getLayout(String id) { // Taken from LegacyRepositorySystem // - public static org.apache.maven.model.Repository fromSettingsRepository( + public static Repository fromSettingsRepository( org.apache.maven.settings.Repository settingsRepository) { - org.apache.maven.model.Repository modelRepository = new org.apache.maven.model.Repository(); + Repository modelRepository = new Repository(); modelRepository.setId(settingsRepository.getId()); modelRepository.setLayout(settingsRepository.getLayout()); modelRepository.setName(settingsRepository.getName()); @@ -308,9 +311,9 @@ public static org.apache.maven.model.Repository fromSettingsRepository( return modelRepository; } - public static org.apache.maven.model.RepositoryPolicy fromSettingsRepositoryPolicy( + public static RepositoryPolicy fromSettingsRepositoryPolicy( org.apache.maven.settings.RepositoryPolicy settingsRepositoryPolicy) { - org.apache.maven.model.RepositoryPolicy modelRepositoryPolicy = new org.apache.maven.model.RepositoryPolicy(); + RepositoryPolicy modelRepositoryPolicy = new RepositoryPolicy(); if (settingsRepositoryPolicy != null) { modelRepositoryPolicy.setEnabled(settingsRepositoryPolicy.isEnabled()); modelRepositoryPolicy.setUpdatePolicy(settingsRepositoryPolicy.getUpdatePolicy()); @@ -324,7 +327,7 @@ public static ArtifactRepository buildArtifactRepository(org.apache.maven.settin return buildArtifactRepository(fromSettingsRepository(repo)); } - public static ArtifactRepository buildArtifactRepository(org.apache.maven.model.Repository repo) + public static ArtifactRepository buildArtifactRepository(Repository repo) throws InvalidRepositoryException { if (repo != null) { String id = repo.getId(); @@ -352,7 +355,7 @@ public static ArtifactRepository buildArtifactRepository(org.apache.maven.model. } public static ArtifactRepositoryPolicy buildArtifactRepositoryPolicy( - org.apache.maven.model.RepositoryPolicy policy) { + RepositoryPolicy policy) { boolean enabled = true; String updatePolicy = null; diff --git a/impl/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequestPopulator.java b/impl/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequestPopulator.java index 5f217daa50ff..8785d9d2fa3e 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequestPopulator.java +++ b/impl/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequestPopulator.java @@ -33,6 +33,7 @@ import org.apache.maven.artifact.repository.ArtifactRepository; import org.apache.maven.bridge.MavenRepositorySystem; import org.apache.maven.settings.Mirror; +import org.apache.maven.settings.Profile; import org.apache.maven.settings.Proxy; import org.apache.maven.settings.Repository; import org.apache.maven.settings.Server; @@ -208,7 +209,7 @@ public MavenExecutionRequest populateFromSettings(MavenExecutionRequest request, request.setActiveProfiles(settings.getActiveProfiles()); - for (org.apache.maven.settings.Profile rawProfile : settings.getProfiles()) { + for (Profile rawProfile : settings.getProfiles()) { request.addProfile(SettingsUtils.convertFromSettingsProfile(rawProfile)); if (settings.getActiveProfiles().contains(rawProfile.getId())) { diff --git a/impl/maven-core/src/main/java/org/apache/maven/execution/scope/internal/MojoExecutionScopeModule.java b/impl/maven-core/src/main/java/org/apache/maven/execution/scope/internal/MojoExecutionScopeModule.java index a477ec868cbb..6fb93152564b 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/execution/scope/internal/MojoExecutionScopeModule.java +++ b/impl/maven-core/src/main/java/org/apache/maven/execution/scope/internal/MojoExecutionScopeModule.java @@ -19,6 +19,7 @@ package org.apache.maven.execution.scope.internal; import com.google.inject.AbstractModule; +import org.apache.maven.api.Project; import org.apache.maven.api.plugin.Log; import org.apache.maven.execution.scope.MojoExecutionScoped; import org.apache.maven.plugin.MojoExecution; @@ -48,8 +49,8 @@ protected void configure() { bind(Log.class) .toProvider(MojoExecutionScope.seededKeyProvider(Log.class)) .in(scope); - bind(org.apache.maven.api.Project.class) - .toProvider(MojoExecutionScope.seededKeyProvider(org.apache.maven.api.Project.class)) + bind(Project.class) + .toProvider(MojoExecutionScope.seededKeyProvider(Project.class)) .in(scope); bind(org.apache.maven.api.MojoExecution.class) .toProvider(MojoExecutionScope.seededKeyProvider(org.apache.maven.api.MojoExecution.class)) diff --git a/impl/maven-core/src/main/java/org/apache/maven/internal/aether/LegacyRepositorySystemSessionExtender.java b/impl/maven-core/src/main/java/org/apache/maven/internal/aether/LegacyRepositorySystemSessionExtender.java index ee7129246d64..6114e3455564 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/internal/aether/LegacyRepositorySystemSessionExtender.java +++ b/impl/maven-core/src/main/java/org/apache/maven/internal/aether/LegacyRepositorySystemSessionExtender.java @@ -29,6 +29,7 @@ import org.apache.maven.artifact.repository.Authentication; import org.apache.maven.bridge.MavenRepositorySystem; import org.apache.maven.execution.MavenExecutionRequest; +import org.apache.maven.repository.Proxy; import org.apache.maven.settings.Mirror; import org.eclipse.aether.repository.AuthenticationContext; import org.eclipse.aether.repository.AuthenticationSelector; @@ -99,12 +100,12 @@ private void injectProxy(ProxySelector selector, List reposi } } - private org.apache.maven.repository.Proxy getProxy(ProxySelector selector, ArtifactRepository repository) { + private Proxy getProxy(ProxySelector selector, ArtifactRepository repository) { if (selector != null) { RemoteRepository repo = RepositoryUtils.toRepo(repository); org.eclipse.aether.repository.Proxy proxy = selector.getProxy(repo); if (proxy != null) { - org.apache.maven.repository.Proxy p = new org.apache.maven.repository.Proxy(); + Proxy p = new Proxy(); p.setHost(proxy.getHost()); p.setProtocol(proxy.getType()); p.setPort(proxy.getPort()); diff --git a/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultLifecycleRegistry.java b/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultLifecycleRegistry.java index 60a6c0b32a0b..996035d1e5ef 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultLifecycleRegistry.java +++ b/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultLifecycleRegistry.java @@ -148,7 +148,7 @@ public List computePhases(Lifecycle lifecycle) { } private static void addPhase( - Graph graph, Graph.Vertex before, Graph.Vertex after, org.apache.maven.api.Lifecycle.Phase phase) { + Graph graph, Graph.Vertex before, Graph.Vertex after, Lifecycle.Phase phase) { Graph.Vertex ep0 = graph.addVertex(BEFORE + phase.name()); Graph.Vertex ep1 = graph.addVertex("$$" + phase.name()); Graph.Vertex ep2 = graph.addVertex(phase.name()); diff --git a/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultProject.java b/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultProject.java index 5c53a246a5c5..4ebfdbbb188f 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultProject.java +++ b/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultProject.java @@ -38,6 +38,7 @@ import org.apache.maven.api.VersionConstraint; import org.apache.maven.api.annotations.Nonnull; import org.apache.maven.api.annotations.Nullable; +import org.apache.maven.api.model.Dependency; import org.apache.maven.api.model.DependencyManagement; import org.apache.maven.api.model.Model; import org.apache.maven.api.model.Profile; @@ -45,6 +46,7 @@ import org.apache.maven.impl.MappedList; import org.apache.maven.project.MavenProject; import org.apache.maven.project.artifact.ProjectArtifact; +import org.eclipse.aether.artifact.Artifact; import org.eclipse.aether.util.artifact.ArtifactIdUtils; public class DefaultProject implements Project { @@ -94,8 +96,8 @@ public String getVersion() { @Nonnull @Override public List getArtifacts() { - org.eclipse.aether.artifact.Artifact pomArtifact = RepositoryUtils.toArtifact(new ProjectArtifact(project)); - org.eclipse.aether.artifact.Artifact projectArtifact = RepositoryUtils.toArtifact(project.getArtifact()); + Artifact pomArtifact = RepositoryUtils.toArtifact(new ProjectArtifact(project)); + Artifact projectArtifact = RepositoryUtils.toArtifact(project.getArtifact()); ArrayList result = new ArrayList<>(2); result.add(session.getArtifact(ProducedArtifact.class, pomArtifact)); @@ -200,7 +202,7 @@ public List getEffectiveActiveProfiles() { } @Nonnull - private DependencyCoordinates toDependency(org.apache.maven.api.model.Dependency dependency) { + private DependencyCoordinates toDependency(Dependency dependency) { return new DependencyCoordinates() { @Override public String getGroupId() { diff --git a/impl/maven-core/src/main/java/org/apache/maven/internal/impl/InternalMavenSession.java b/impl/maven-core/src/main/java/org/apache/maven/internal/impl/InternalMavenSession.java index dbc679d903e2..218d8985c475 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/internal/impl/InternalMavenSession.java +++ b/impl/maven-core/src/main/java/org/apache/maven/internal/impl/InternalMavenSession.java @@ -24,8 +24,11 @@ import org.apache.maven.api.RemoteRepository; import org.apache.maven.api.Session; import org.apache.maven.api.annotations.Nullable; +import org.apache.maven.artifact.repository.ArtifactRepository; import org.apache.maven.execution.MavenSession; import org.apache.maven.impl.InternalSession; +import org.apache.maven.project.MavenProject; +import org.eclipse.aether.RepositorySystemSession; import static org.apache.maven.internal.impl.CoreUtils.cast; @@ -35,22 +38,22 @@ static InternalMavenSession from(Session session) { return cast(InternalMavenSession.class, session, "session should be an " + InternalMavenSession.class); } - static InternalMavenSession from(org.eclipse.aether.RepositorySystemSession session) { + static InternalMavenSession from(RepositorySystemSession session) { return cast(InternalMavenSession.class, session.getData().get(InternalSession.class), "session"); } - List getProjects(List projects); + List getProjects(List projects); /** * May return null if the input project is null or is not part of the reactor. */ @Nullable - Project getProject(org.apache.maven.project.MavenProject project); + Project getProject(MavenProject project); - List toArtifactRepositories( + List toArtifactRepositories( List repositories); - org.apache.maven.artifact.repository.ArtifactRepository toArtifactRepository(RemoteRepository repository); + ArtifactRepository toArtifactRepository(RemoteRepository repository); MavenSession getMavenSession(); } diff --git a/impl/maven-core/src/main/java/org/apache/maven/internal/transformation/impl/TransformedArtifact.java b/impl/maven-core/src/main/java/org/apache/maven/internal/transformation/impl/TransformedArtifact.java index e928e6101896..4fe0be9890a1 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/internal/transformation/impl/TransformedArtifact.java +++ b/impl/maven-core/src/main/java/org/apache/maven/internal/transformation/impl/TransformedArtifact.java @@ -32,6 +32,7 @@ import java.util.function.Supplier; import org.apache.maven.api.services.ModelBuilderException; +import org.apache.maven.artifact.Artifact; import org.apache.maven.artifact.DefaultArtifact; import org.apache.maven.internal.transformation.TransformationFailedException; import org.apache.maven.project.MavenProject; @@ -58,7 +59,7 @@ class TransformedArtifact extends DefaultArtifact { MavenProject project, Path target, RepositorySystemSession session, - org.apache.maven.artifact.Artifact source, + Artifact source, Supplier sourcePathProvider, String classifier, String extension) { diff --git a/impl/maven-core/src/main/java/org/apache/maven/lifecycle/Lifecycle.java b/impl/maven-core/src/main/java/org/apache/maven/lifecycle/Lifecycle.java index 91d392e280a6..e33575d331a1 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/lifecycle/Lifecycle.java +++ b/impl/maven-core/src/main/java/org/apache/maven/lifecycle/Lifecycle.java @@ -24,6 +24,7 @@ import java.util.Map; import java.util.stream.Collectors; +import org.apache.maven.api.services.LifecycleRegistry; import org.apache.maven.lifecycle.mapping.LifecyclePhase; /** @@ -39,7 +40,7 @@ public Lifecycle(String id, List phases, Map def } public Lifecycle( - org.apache.maven.api.services.LifecycleRegistry registry, org.apache.maven.api.Lifecycle lifecycle) { + LifecycleRegistry registry, org.apache.maven.api.Lifecycle lifecycle) { this.lifecycle = lifecycle; this.id = lifecycle.id(); this.phases = registry.computePhases(lifecycle); diff --git a/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java b/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java index e35bd8df1ca3..bca43559851a 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java +++ b/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java @@ -59,6 +59,7 @@ import org.apache.maven.plugin.prefix.NoPluginFoundForPrefixException; import org.apache.maven.plugin.version.PluginVersionResolutionException; import org.apache.maven.project.MavenProject; +import org.codehaus.plexus.util.xml.Xpp3Dom; /** * NOTE: This class is not part of any public api and can be changed or deleted without prior notice. @@ -511,7 +512,7 @@ private void injectLifecycleOverlay( if (phaseConfiguration != null) { for (MojoExecution forkedExecution : forkedExecutions) { - org.codehaus.plexus.util.xml.Xpp3Dom config = forkedExecution.getConfiguration(); + Xpp3Dom config = forkedExecution.getConfiguration(); if (config != null) { XmlNode forkedConfiguration = config.getDom(); diff --git a/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoDescriptorCreator.java b/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoDescriptorCreator.java index 009e3712cdc9..eb4ceb28ed0d 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoDescriptorCreator.java +++ b/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoDescriptorCreator.java @@ -49,6 +49,7 @@ import org.apache.maven.plugin.version.PluginVersionResolver; import org.apache.maven.project.MavenProject; import org.codehaus.plexus.configuration.PlexusConfiguration; +import org.codehaus.plexus.util.xml.Xpp3Dom; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -106,7 +107,7 @@ public static XmlNode convert(org.apache.maven.api.plugin.descriptor.MojoDescrip return XmlNode.newInstance("configuration", children); } - public static org.codehaus.plexus.util.xml.Xpp3Dom convert(MojoDescriptor mojoDescriptor) { + public static Xpp3Dom convert(MojoDescriptor mojoDescriptor) { PlexusConfiguration c = mojoDescriptor.getMojoConfiguration(); List children = new ArrayList<>(); @@ -128,7 +129,7 @@ public static org.codehaus.plexus.util.xml.Xpp3Dom convert(MojoDescriptor mojoDe } XmlNode dom = XmlNode.newInstance("configuration", children); - return new org.codehaus.plexus.util.xml.Xpp3Dom(dom); + return new Xpp3Dom(dom); } // org.apache.maven.plugins:maven-remote-resources-plugin:1.0:process@executionId diff --git a/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/concurrent/PluginLifecycle.java b/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/concurrent/PluginLifecycle.java index 81a319b7aea6..7150ff87d8d4 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/concurrent/PluginLifecycle.java +++ b/impl/maven-core/src/main/java/org/apache/maven/lifecycle/internal/concurrent/PluginLifecycle.java @@ -26,6 +26,7 @@ import org.apache.maven.api.Lifecycle; import org.apache.maven.api.model.Plugin; +import org.apache.maven.api.model.PluginExecution; import org.apache.maven.plugin.descriptor.PluginDescriptor; class PluginLifecycle implements Lifecycle { @@ -61,7 +62,7 @@ public List plugins() { .version(pluginDescriptor.getVersion()) .configuration(phase.getConfiguration()) .executions(phase.getExecutions().stream() - .map(exec -> org.apache.maven.api.model.PluginExecution.newBuilder() + .map(exec -> PluginExecution.newBuilder() .goals(exec.getGoals()) .configuration(exec.getConfiguration()) .build()) diff --git a/impl/maven-core/src/main/java/org/apache/maven/plugin/MojoExecution.java b/impl/maven-core/src/main/java/org/apache/maven/plugin/MojoExecution.java index fd8835a699ee..86fba1b18f68 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/plugin/MojoExecution.java +++ b/impl/maven-core/src/main/java/org/apache/maven/plugin/MojoExecution.java @@ -25,6 +25,7 @@ import org.apache.maven.api.xml.XmlNode; import org.apache.maven.model.Plugin; import org.apache.maven.plugin.descriptor.MojoDescriptor; +import org.codehaus.plexus.util.xml.Xpp3Dom; /** * MojoExecution @@ -39,7 +40,7 @@ public class MojoExecution { private MojoDescriptor mojoDescriptor; - private org.codehaus.plexus.util.xml.Xpp3Dom configuration; + private Xpp3Dom configuration; /** * Describes the source of an execution. @@ -96,7 +97,7 @@ public MojoExecution(MojoDescriptor mojoDescriptor, String executionId) { this.configuration = null; } - public MojoExecution(MojoDescriptor mojoDescriptor, org.codehaus.plexus.util.xml.Xpp3Dom configuration) { + public MojoExecution(MojoDescriptor mojoDescriptor, Xpp3Dom configuration) { this.mojoDescriptor = mojoDescriptor; this.configuration = configuration; this.executionId = null; @@ -104,7 +105,7 @@ public MojoExecution(MojoDescriptor mojoDescriptor, org.codehaus.plexus.util.xml public MojoExecution(MojoDescriptor mojoDescriptor, XmlNode configuration) { this.mojoDescriptor = mojoDescriptor; - this.configuration = new org.codehaus.plexus.util.xml.Xpp3Dom(configuration); + this.configuration = new Xpp3Dom(configuration); this.executionId = null; } @@ -133,16 +134,16 @@ public MojoDescriptor getMojoDescriptor() { return mojoDescriptor; } - public org.codehaus.plexus.util.xml.Xpp3Dom getConfiguration() { + public Xpp3Dom getConfiguration() { return configuration; } - public void setConfiguration(org.codehaus.plexus.util.xml.Xpp3Dom configuration) { + public void setConfiguration(Xpp3Dom configuration) { this.configuration = configuration; } public void setConfiguration(XmlNode configuration) { - this.configuration = configuration != null ? new org.codehaus.plexus.util.xml.Xpp3Dom(configuration) : null; + this.configuration = configuration != null ? new Xpp3Dom(configuration) : null; } public String identify() { diff --git a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java index b1b0da0b369d..30a3a2822f91 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java +++ b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java @@ -48,6 +48,7 @@ import org.apache.maven.api.PathType; import org.apache.maven.api.Project; import org.apache.maven.api.Session; +import org.apache.maven.api.plugin.Log; import org.apache.maven.api.plugin.descriptor.Resolution; import org.apache.maven.api.services.DependencyResolver; import org.apache.maven.api.services.DependencyResolverResult; @@ -550,7 +551,7 @@ private T loadV4Mojo( Project project = sessionV4.getProject(session.getCurrentProject()); org.apache.maven.api.MojoExecution execution = new DefaultMojoExecution(sessionV4, mojoExecution); - org.apache.maven.api.plugin.Log log = new DefaultLog( + Log log = new DefaultLog( LoggerFactory.getLogger(mojoExecution.getMojoDescriptor().getFullGoalName())); try { Injector injector = Injector.create(); @@ -560,7 +561,7 @@ private T loadV4Mojo( injector.bindInstance(Session.class, sessionV4); injector.bindInstance(Project.class, project); injector.bindInstance(org.apache.maven.api.MojoExecution.class, execution); - injector.bindInstance(org.apache.maven.api.plugin.Log.class, log); + injector.bindInstance(Log.class, log); mojo = mojoInterface.cast(injector.getInstance( Key.of(mojoDescriptor.getImplementationClass(), mojoDescriptor.getRoleHint()))); diff --git a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DeprecatedCoreExpressionValidator.java b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DeprecatedCoreExpressionValidator.java index faaac5cf1c53..da8677c9ad33 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DeprecatedCoreExpressionValidator.java +++ b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DeprecatedCoreExpressionValidator.java @@ -25,6 +25,7 @@ import java.util.HashMap; import java.util.Objects; +import org.apache.maven.artifact.repository.ArtifactRepository; import org.apache.maven.execution.MavenSession; import org.apache.maven.plugin.PluginValidationManager; import org.apache.maven.plugin.descriptor.MojoDescriptor; @@ -83,7 +84,7 @@ protected void doValidate( private boolean isDeprecated(Parameter parameter) { return Objects.equals( - org.apache.maven.artifact.repository.ArtifactRepository.class.getName(), parameter.getType()) + ArtifactRepository.class.getName(), parameter.getType()) && DEPRECATED_CORE_PARAMETERS.containsKey(parameter.getDefaultValue()); } } diff --git a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/Maven3CompatDependenciesValidator.java b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/Maven3CompatDependenciesValidator.java index 0cf8fee6ad35..37f3377ae322 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/Maven3CompatDependenciesValidator.java +++ b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/Maven3CompatDependenciesValidator.java @@ -26,6 +26,7 @@ import org.apache.maven.plugin.PluginValidationManager; import org.eclipse.aether.RepositorySystemSession; import org.eclipse.aether.artifact.Artifact; +import org.eclipse.aether.graph.Dependency; import org.eclipse.aether.resolution.ArtifactDescriptorResult; /** @@ -47,7 +48,7 @@ protected void doValidate( RepositorySystemSession session, Artifact pluginArtifact, ArtifactDescriptorResult artifactDescriptorResult) { - for (org.eclipse.aether.graph.Dependency dependency : artifactDescriptorResult.getDependencies()) { + for (Dependency dependency : artifactDescriptorResult.getDependencies()) { if ("org.apache.maven".equals(dependency.getArtifact().getGroupId()) && "maven-compat".equals(dependency.getArtifact().getArtifactId()) && !DependencyScope.TEST.is(dependency.getScope())) { diff --git a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/MavenScopeDependenciesValidator.java b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/MavenScopeDependenciesValidator.java index 460a91c6ec18..acdc75eee1c3 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/MavenScopeDependenciesValidator.java +++ b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/MavenScopeDependenciesValidator.java @@ -29,6 +29,7 @@ import org.apache.maven.plugin.PluginValidationManager; import org.eclipse.aether.RepositorySystemSession; import org.eclipse.aether.artifact.Artifact; +import org.eclipse.aether.graph.Dependency; import org.eclipse.aether.resolution.ArtifactDescriptorResult; /** @@ -52,7 +53,7 @@ protected void doValidate( ArtifactDescriptorResult artifactDescriptorResult) { Set mavenArtifacts = artifactDescriptorResult.getDependencies().stream() .filter(d -> !DependencyScope.PROVIDED.is(d.getScope()) && !DependencyScope.TEST.is(d.getScope())) - .map(org.eclipse.aether.graph.Dependency::getArtifact) + .map(Dependency::getArtifact) .filter(a -> "org.apache.maven".equals(a.getGroupId())) .filter(a -> !DefaultPluginValidationManager.EXPECTED_PROVIDED_SCOPE_EXCLUSIONS_GA.contains( a.getGroupId() + ":" + a.getArtifactId())) diff --git a/impl/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectHelper.java b/impl/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectHelper.java index b5ee23b4e877..733fa4eb0983 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectHelper.java +++ b/impl/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectHelper.java @@ -92,7 +92,7 @@ public void attachArtifact(MavenProject project, File artifactFile, String artif /** * Add an attached artifact or replace the file for an existing artifact. * - * @see MavenProject#addAttachedArtifact(org.apache.maven.artifact.Artifact) + * @see MavenProject#addAttachedArtifact(Artifact) * @param project project reference. * @param artifact artifact to add or replace. */ diff --git a/impl/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuilder.java b/impl/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuilder.java index a14dfaabe158..4170dacc4269 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuilder.java +++ b/impl/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuilder.java @@ -51,6 +51,7 @@ import org.apache.maven.api.LocalRepository; import org.apache.maven.api.ProjectScope; import org.apache.maven.api.SessionData; +import org.apache.maven.api.WorkspaceRepository; import org.apache.maven.api.annotations.Nonnull; import org.apache.maven.api.annotations.Nullable; import org.apache.maven.api.model.Build; @@ -449,7 +450,7 @@ ProjectBuildingResult build(boolean parent, Artifact artifact, boolean allowStub ArtifactResolverResult.ResultItem resItem = res.getResult(coordinates); pomArtifact = InternalMavenSession.from(session).toArtifact(resItem.getArtifact()); - localProject = resItem.getRepository() instanceof org.apache.maven.api.WorkspaceRepository; + localProject = resItem.getRepository() instanceof WorkspaceRepository; } catch (ArtifactResolverException e) { if (e.getResult().getResults().values().iterator().next().isMissing() && allowStubModel) { return build(parent, null, createStubModelSource(artifact)); diff --git a/impl/maven-core/src/main/java/org/apache/maven/project/MavenProjectHelper.java b/impl/maven-core/src/main/java/org/apache/maven/project/MavenProjectHelper.java index 4e75473210b2..c6578c2bba23 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/project/MavenProjectHelper.java +++ b/impl/maven-core/src/main/java/org/apache/maven/project/MavenProjectHelper.java @@ -28,7 +28,7 @@ public interface MavenProjectHelper { String ROLE = MavenProjectHelper.class.getName(); /** - * See {@link #attachArtifact(MavenProject, String, String, java.io.File)}, but with type set to null. + * See {@link #attachArtifact(MavenProject, String, String, File)}, but with type set to null. * @param project project reference. * @param artifactFile artifact file. * @param artifactClassifier artifact classifier. @@ -36,7 +36,7 @@ public interface MavenProjectHelper { void attachArtifact(MavenProject project, File artifactFile, String artifactClassifier); /** - * * See {@link #attachArtifact(MavenProject, String, String, java.io.File)}, but with classifier set to null. + * * See {@link #attachArtifact(MavenProject, String, String, File)}, but with classifier set to null. * @param project project reference. * @param artifactType artifact type. * @param artifactFile artifact file. diff --git a/impl/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingRequest.java b/impl/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingRequest.java index c2f8fb3bf340..1e62e7d4c80b 100644 --- a/impl/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingRequest.java +++ b/impl/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingRequest.java @@ -123,7 +123,7 @@ public interface ProjectBuildingRequest { List getInactiveProfileIds(); /** - * Add a {@link org.apache.maven.model.Profile} that has come from an external source. This may be from a custom + * Add a {@link Profile} that has come from an external source. This may be from a custom * configuration like the MavenCLI settings.xml file, or from a custom dialog in an IDE integration like M2Eclipse. * * @param profile diff --git a/impl/maven-core/src/test/java/org/apache/maven/di/DiTest.java b/impl/maven-core/src/test/java/org/apache/maven/di/DiTest.java index 2b3a0a861cb3..72f8c16b14d1 100644 --- a/impl/maven-core/src/test/java/org/apache/maven/di/DiTest.java +++ b/impl/maven-core/src/test/java/org/apache/maven/di/DiTest.java @@ -30,6 +30,7 @@ import com.google.inject.Binding; import com.google.inject.Injector; import com.google.inject.TypeLiteral; +import org.apache.maven.api.di.Inject; import org.apache.maven.api.model.Model; import org.apache.maven.api.services.Source; import org.apache.maven.api.spi.ModelParser; @@ -38,6 +39,7 @@ import org.codehaus.plexus.DefaultContainerConfiguration; import org.codehaus.plexus.DefaultPlexusContainer; import org.codehaus.plexus.PlexusContainer; +import org.eclipse.sisu.Priority; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Nested; import org.junit.jupiter.api.Test; @@ -102,13 +104,13 @@ void testDI() throws Exception { } static class DiInjected { - @org.apache.maven.api.di.Inject + @Inject ModelParser parser; - @org.apache.maven.api.di.Inject + @Inject List parsers; - @org.apache.maven.api.di.Inject + @Inject Map parsersMap; } @@ -174,13 +176,13 @@ void testDI() throws Exception { } static class DiInjected { - @org.apache.maven.api.di.Inject + @Inject ModelParser parser; - @org.apache.maven.api.di.Inject + @Inject List parsers; - @org.apache.maven.api.di.Inject + @Inject Map parsersMap; } @@ -246,13 +248,13 @@ void testDI() throws Exception { } static class DiInjected { - @org.apache.maven.api.di.Inject + @Inject ModelParser parser; - @org.apache.maven.api.di.Inject + @Inject List parsers; - @org.apache.maven.api.di.Inject + @Inject Map parsersMap; } @@ -325,18 +327,18 @@ void testDI() throws Exception { } static class DiInjected { - @org.apache.maven.api.di.Inject + @Inject ModelParser parser; - @org.apache.maven.api.di.Inject + @Inject List parsers; - @org.apache.maven.api.di.Inject + @Inject Map parsersMap; } - @javax.inject.Named("sisu") - @org.eclipse.sisu.Priority(100) + @Named("sisu") + @Priority(100) static class TestModelParserSisu implements ModelParser { @Override public Optional locate(Path dir) { diff --git a/impl/maven-core/src/test/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactoryTest.java b/impl/maven-core/src/test/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactoryTest.java index 8b80dc0efd60..5ba8c72d5644 100644 --- a/impl/maven-core/src/test/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactoryTest.java +++ b/impl/maven-core/src/test/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactoryTest.java @@ -40,6 +40,7 @@ import org.codehaus.plexus.testing.PlexusTest; import org.codehaus.plexus.util.xml.Xpp3Dom; import org.eclipse.aether.ConfigurationProperties; +import org.eclipse.aether.RepositorySystem; import org.eclipse.aether.collection.VersionFilter; import org.eclipse.aether.repository.RepositoryPolicy; import org.eclipse.aether.util.graph.version.ChainedVersionFilter; @@ -69,7 +70,7 @@ public class DefaultRepositorySystemSessionFactoryTest { protected EventSpyDispatcher eventSpyDispatcher; @Inject - protected org.eclipse.aether.RepositorySystem aetherRepositorySystem; + protected RepositorySystem aetherRepositorySystem; @Inject protected RuntimeInformation information; diff --git a/impl/maven-core/src/test/java/org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4Test.java b/impl/maven-core/src/test/java/org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4Test.java index 4b25cf559bcc..43509d141cd2 100644 --- a/impl/maven-core/src/test/java/org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4Test.java +++ b/impl/maven-core/src/test/java/org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4Test.java @@ -33,6 +33,7 @@ import org.apache.maven.AbstractCoreMavenComponentTestCase; import org.apache.maven.RepositoryUtils; import org.apache.maven.api.Artifact; +import org.apache.maven.api.Dependency; import org.apache.maven.api.MojoExecution; import org.apache.maven.api.Session; import org.apache.maven.artifact.DefaultArtifact; @@ -132,7 +133,7 @@ public void testPluginArtifactMapExpressionReference() throws Exception { MojoExecution exec = newMojoExecution(session); @SuppressWarnings("unchecked") - Map depResults = (Map) + Map depResults = (Map) new PluginParameterExpressionEvaluatorV4(session, null, exec) .evaluate("${mojo.plugin.dependenciesMap}"); diff --git a/impl/maven-core/src/test/java/org/apache/maven/project/ProjectBuilderTest.java b/impl/maven-core/src/test/java/org/apache/maven/project/ProjectBuilderTest.java index 541ac8063c60..627406e05e3d 100644 --- a/impl/maven-core/src/test/java/org/apache/maven/project/ProjectBuilderTest.java +++ b/impl/maven-core/src/test/java/org/apache/maven/project/ProjectBuilderTest.java @@ -82,7 +82,7 @@ void testBuildFromModelSource() throws Exception { ProjectBuildingRequest configuration = new DefaultProjectBuildingRequest(); configuration.setRepositorySession(mavenSession.getRepositorySession()); ProjectBuildingResult result = getContainer() - .lookup(org.apache.maven.project.ProjectBuilder.class) + .lookup(ProjectBuilder.class) .build(pomFile, configuration); assertNotNull(result.getProject().getParentFile()); @@ -96,7 +96,7 @@ void testVersionlessManagedDependency() throws Exception { configuration.setRepositorySession(mavenSession.getRepositorySession()); ProjectBuildingException e = assertThrows(ProjectBuildingException.class, () -> getContainer() - .lookup(org.apache.maven.project.ProjectBuilder.class) + .lookup(ProjectBuilder.class) .build(pomFile, configuration)); assertThat( e.getResults(), @@ -116,12 +116,12 @@ void testResolveDependencies() throws Exception { // single project build entry point ProjectBuildingResult result = getContainer() - .lookup(org.apache.maven.project.ProjectBuilder.class) + .lookup(ProjectBuilder.class) .build(pomFile, configuration); assertEquals(1, result.getProject().getArtifacts().size()); // multi projects build entry point List results = getContainer() - .lookup(org.apache.maven.project.ProjectBuilder.class) + .lookup(ProjectBuilder.class) .build(Collections.singletonList(pomFile), false, configuration); assertEquals(1, results.size()); MavenProject mavenProject = results.get(0).getProject(); @@ -150,12 +150,12 @@ void testDontResolveDependencies() throws Exception { // single project build entry point ProjectBuildingResult result = getContainer() - .lookup(org.apache.maven.project.ProjectBuilder.class) + .lookup(ProjectBuilder.class) .build(pomFile, configuration); assertEquals(0, result.getProject().getArtifacts().size()); // multi projects build entry point List results = getContainer() - .lookup(org.apache.maven.project.ProjectBuilder.class) + .lookup(ProjectBuilder.class) .build(Collections.singletonList(pomFile), false, configuration); assertEquals(1, results.size()); MavenProject mavenProject = results.get(0).getProject(); @@ -173,8 +173,8 @@ void testReadModifiedPoms(@TempDir Path tempDir) throws Exception { mavenSession.getRequest().setRootDirectory(tempDir); ProjectBuildingRequest configuration = new DefaultProjectBuildingRequest(); configuration.setRepositorySession(mavenSession.getRepositorySession()); - org.apache.maven.project.ProjectBuilder projectBuilder = - getContainer().lookup(org.apache.maven.project.ProjectBuilder.class); + ProjectBuilder projectBuilder = + getContainer().lookup(ProjectBuilder.class); File child = new File(tempDir.toFile(), "child/pom.xml"); // build project once projectBuilder.build(child, configuration); @@ -198,8 +198,8 @@ void testReadErroneousMavenProjectContainsReference() throws Exception { ProjectBuildingRequest configuration = new DefaultProjectBuildingRequest(); configuration.setValidationLevel(ModelBuildingRequest.VALIDATION_LEVEL_MINIMAL); configuration.setRepositorySession(mavenSession.getRepositorySession()); - org.apache.maven.project.ProjectBuilder projectBuilder = - getContainer().lookup(org.apache.maven.project.ProjectBuilder.class); + ProjectBuilder projectBuilder = + getContainer().lookup(ProjectBuilder.class); // single project build entry point ProjectBuildingException ex1 = @@ -230,8 +230,8 @@ void testReadInvalidPom() throws Exception { ProjectBuildingRequest configuration = new DefaultProjectBuildingRequest(); configuration.setValidationLevel(ModelBuildingRequest.VALIDATION_LEVEL_STRICT); configuration.setRepositorySession(mavenSession.getRepositorySession()); - org.apache.maven.project.ProjectBuilder projectBuilder = - getContainer().lookup(org.apache.maven.project.ProjectBuilder.class); + ProjectBuilder projectBuilder = + getContainer().lookup(ProjectBuilder.class); // single project build entry point Exception ex = assertThrows(Exception.class, () -> projectBuilder.build(pomFile, configuration)); @@ -260,8 +260,8 @@ void testReadParentAndChildWithRegularVersionSetParentFile() throws Exception { ProjectBuildingRequest configuration = new DefaultProjectBuildingRequest(); configuration.setValidationLevel(ModelBuildingRequest.VALIDATION_LEVEL_MINIMAL); configuration.setRepositorySession(mavenSession.getRepositorySession()); - org.apache.maven.project.ProjectBuilder projectBuilder = - getContainer().lookup(org.apache.maven.project.ProjectBuilder.class); + ProjectBuilder projectBuilder = + getContainer().lookup(ProjectBuilder.class); // read poms separately boolean parentFileWasFoundOnChild = false; @@ -339,7 +339,7 @@ void testBuildFromModelSourceResolvesBasedir() throws Exception { ProjectBuildingRequest configuration = new DefaultProjectBuildingRequest(); configuration.setRepositorySession(mavenSession.getRepositorySession()); ProjectBuildingResult result = getContainer() - .lookup(org.apache.maven.project.ProjectBuilder.class) + .lookup(ProjectBuilder.class) .build(pomFile, configuration); assertEquals( diff --git a/impl/maven-core/src/test/java/org/apache/maven/project/harness/Xpp3DomPointerFactory.java b/impl/maven-core/src/test/java/org/apache/maven/project/harness/Xpp3DomPointerFactory.java index 20ef83c50441..1007bc27a133 100644 --- a/impl/maven-core/src/test/java/org/apache/maven/project/harness/Xpp3DomPointerFactory.java +++ b/impl/maven-core/src/test/java/org/apache/maven/project/harness/Xpp3DomPointerFactory.java @@ -24,6 +24,7 @@ import org.apache.commons.jxpath.ri.model.NodePointer; import org.apache.commons.jxpath.ri.model.NodePointerFactory; import org.apache.maven.api.xml.XmlNode; +import org.codehaus.plexus.util.xml.Xpp3Dom; /** * A node pointer factory for JXPath to support Xpp3Dom. @@ -36,7 +37,7 @@ public int getOrder() { } public NodePointer createNodePointer(QName name, Object object, Locale locale) { - if (object instanceof org.codehaus.plexus.util.xml.Xpp3Dom xpp3Dom) { + if (object instanceof Xpp3Dom xpp3Dom) { object = xpp3Dom.getDom(); } if (object instanceof XmlNode xmlNode) { @@ -46,7 +47,7 @@ public NodePointer createNodePointer(QName name, Object object, Locale locale) { } public NodePointer createNodePointer(NodePointer parent, QName name, Object object) { - if (object instanceof org.codehaus.plexus.util.xml.Xpp3Dom xpp3Dom) { + if (object instanceof Xpp3Dom xpp3Dom) { object = xpp3Dom.getDom(); } if (object instanceof XmlNode xmlNode) { diff --git a/impl/maven-di/src/main/java/org/apache/maven/di/impl/InjectorImpl.java b/impl/maven-di/src/main/java/org/apache/maven/di/impl/InjectorImpl.java index c3a069bca55e..8a7cd4e089f4 100644 --- a/impl/maven-di/src/main/java/org/apache/maven/di/impl/InjectorImpl.java +++ b/impl/maven-di/src/main/java/org/apache/maven/di/impl/InjectorImpl.java @@ -431,15 +431,15 @@ private void removeFromResolutionStack(Key key) { } private static class SingletonScope implements Scope { - Map, java.util.function.Supplier> cache = new ConcurrentHashMap<>(); + Map, Supplier> cache = new ConcurrentHashMap<>(); @Nonnull @SuppressWarnings("unchecked") @Override - public java.util.function.Supplier scope( - @Nonnull Key key, @Nonnull java.util.function.Supplier unscoped) { - return (java.util.function.Supplier) - cache.computeIfAbsent(key, k -> new java.util.function.Supplier() { + public Supplier scope( + @Nonnull Key key, @Nonnull Supplier unscoped) { + return (Supplier) + cache.computeIfAbsent(key, k -> new Supplier() { volatile T instance; @Override diff --git a/impl/maven-di/src/main/java/org/apache/maven/di/impl/ReflectionUtils.java b/impl/maven-di/src/main/java/org/apache/maven/di/impl/ReflectionUtils.java index 809e39e9552e..0863ab2539cc 100644 --- a/impl/maven-di/src/main/java/org/apache/maven/di/impl/ReflectionUtils.java +++ b/impl/maven-di/src/main/java/org/apache/maven/di/impl/ReflectionUtils.java @@ -45,6 +45,7 @@ import org.apache.maven.api.di.Named; import org.apache.maven.api.di.Priority; import org.apache.maven.api.di.Qualifier; +import org.apache.maven.api.di.Scope; import org.apache.maven.di.Key; import static java.util.stream.Collectors.toList; @@ -100,7 +101,7 @@ public final class ReflectionUtils { public static @Nullable Annotation scopeOf(AnnotatedElement annotatedElement) { Annotation scope = null; for (Annotation annotation : annotatedElement.getDeclaredAnnotations()) { - if (annotation.annotationType().isAnnotationPresent(org.apache.maven.api.di.Scope.class)) { + if (annotation.annotationType().isAnnotationPresent(Scope.class)) { if (scope != null) { throw new DIException("More than one scope annotation on " + annotatedElement); } diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/AbstractNode.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/AbstractNode.java index ad0dea065f5c..0a2635573bb1 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/AbstractNode.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/AbstractNode.java @@ -25,10 +25,11 @@ import org.apache.maven.api.Node; import org.apache.maven.api.NodeVisitor; +import org.eclipse.aether.graph.DependencyNode; public abstract class AbstractNode implements Node { - abstract org.eclipse.aether.graph.DependencyNode getDependencyNode(); + abstract DependencyNode getDependencyNode(); @Override public boolean accept(NodeVisitor visitor) { diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/AbstractSession.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/AbstractSession.java index 605a36b903c0..717934e12bc2 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/AbstractSession.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/AbstractSession.java @@ -100,6 +100,7 @@ import org.eclipse.aether.RepositorySystem; import org.eclipse.aether.RepositorySystemSession; import org.eclipse.aether.artifact.ArtifactType; +import org.eclipse.aether.graph.DependencyNode; import org.eclipse.aether.repository.ArtifactRepository; import org.eclipse.aether.transfer.TransferResource; @@ -114,7 +115,7 @@ public abstract class AbstractSession implements InternalSession { protected final Lookup lookup; private final Map, Service> services = new ConcurrentHashMap<>(); private final List listeners = new CopyOnWriteArrayList<>(); - private final Map allNodes = + private final Map allNodes = Collections.synchronizedMap(new WeakHashMap<>()); private final Map, Map> allArtifacts = new ConcurrentHashMap<>(); @@ -207,12 +208,12 @@ public org.apache.maven.api.Repository getRepository(ArtifactRepository reposito } @Override - public Node getNode(org.eclipse.aether.graph.DependencyNode node) { + public Node getNode(DependencyNode node) { return getNode(node, false); } @Override - public Node getNode(org.eclipse.aether.graph.DependencyNode node, boolean verbose) { + public Node getNode(DependencyNode node, boolean verbose) { return allNodes.computeIfAbsent(node, n -> new DefaultNode(this, n, verbose)); } diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultArtifactCoordinates.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultArtifactCoordinates.java index 27f299ecf34b..5b5e68cfa947 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultArtifactCoordinates.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultArtifactCoordinates.java @@ -23,6 +23,7 @@ import org.apache.maven.api.ArtifactCoordinates; import org.apache.maven.api.VersionConstraint; import org.apache.maven.api.annotations.Nonnull; +import org.eclipse.aether.artifact.Artifact; import static java.util.Objects.requireNonNull; @@ -31,15 +32,15 @@ */ public class DefaultArtifactCoordinates implements ArtifactCoordinates { private final @Nonnull InternalSession session; - private final @Nonnull org.eclipse.aether.artifact.Artifact coordinates; + private final @Nonnull Artifact coordinates; public DefaultArtifactCoordinates( - @Nonnull InternalSession session, @Nonnull org.eclipse.aether.artifact.Artifact coordinates) { + @Nonnull InternalSession session, @Nonnull Artifact coordinates) { this.session = requireNonNull(session, "session"); this.coordinates = requireNonNull(coordinates, "coordinates"); } - public org.eclipse.aether.artifact.Artifact getCoordinates() { + public Artifact getCoordinates() { return coordinates; } diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultArtifactCoordinatesFactory.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultArtifactCoordinatesFactory.java index d7b6da1174cf..af5ad80cc7d1 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultArtifactCoordinatesFactory.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultArtifactCoordinatesFactory.java @@ -25,6 +25,7 @@ import org.apache.maven.api.services.ArtifactCoordinatesFactory; import org.apache.maven.api.services.ArtifactCoordinatesFactoryRequest; import org.eclipse.aether.artifact.ArtifactType; +import org.eclipse.aether.artifact.DefaultArtifact; import static java.util.Objects.requireNonNull; @@ -37,7 +38,7 @@ public ArtifactCoordinates create(@Nonnull ArtifactCoordinatesFactoryRequest req InternalSession session = InternalSession.from(request.getSession()); if (request.getCoordinatesString() != null) { return new DefaultArtifactCoordinates( - session, new org.eclipse.aether.artifact.DefaultArtifact(request.getCoordinatesString())); + session, new DefaultArtifact(request.getCoordinatesString())); } else { ArtifactType type = null; if (request.getType() != null) { @@ -52,7 +53,7 @@ public ArtifactCoordinates create(@Nonnull ArtifactCoordinatesFactoryRequest req str != null && !str.isEmpty() ? request.getExtension() : type != null ? type.getExtension() : ""; return new DefaultArtifactCoordinates( session, - new org.eclipse.aether.artifact.DefaultArtifact( + new DefaultArtifact( request.getGroupId(), request.getArtifactId(), classifier, diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyCoordinates.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyCoordinates.java index 9128e122c986..3ed98c91f467 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyCoordinates.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyCoordinates.java @@ -25,10 +25,11 @@ import org.apache.maven.api.VersionConstraint; import org.apache.maven.api.annotations.Nonnull; import org.apache.maven.api.annotations.Nullable; +import org.eclipse.aether.graph.Dependency; public class DefaultDependencyCoordinates extends AetherDependencyWrapper implements DependencyCoordinates { public DefaultDependencyCoordinates( - @Nonnull InternalSession session, @Nonnull org.eclipse.aether.graph.Dependency dependency) { + @Nonnull InternalSession session, @Nonnull Dependency dependency) { super(session, dependency); } diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyCoordinatesFactory.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyCoordinatesFactory.java index cd52a06bfcd4..e84cca5d35e3 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyCoordinatesFactory.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyCoordinatesFactory.java @@ -26,6 +26,8 @@ import org.apache.maven.api.services.DependencyCoordinatesFactory; import org.apache.maven.api.services.DependencyCoordinatesFactoryRequest; import org.eclipse.aether.artifact.ArtifactType; +import org.eclipse.aether.artifact.DefaultArtifact; +import org.eclipse.aether.graph.Dependency; import static java.util.Objects.requireNonNull; import static org.apache.maven.impl.ImplUtils.map; @@ -47,16 +49,16 @@ public DependencyCoordinates create(@Nonnull DependencyCoordinatesFactoryRequest if (request.getCoordinatesString() != null) { return new DefaultDependencyCoordinates( session, - new org.eclipse.aether.graph.Dependency( - new org.eclipse.aether.artifact.DefaultArtifact(request.getCoordinatesString()), + new Dependency( + new DefaultArtifact(request.getCoordinatesString()), request.getScope(), request.isOptional(), map(request.getExclusions(), this::toExclusion))); } else { return new DefaultDependencyCoordinates( session, - new org.eclipse.aether.graph.Dependency( - new org.eclipse.aether.artifact.DefaultArtifact( + new Dependency( + new DefaultArtifact( request.getGroupId(), request.getArtifactId(), request.getClassifier(), diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyResolver.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyResolver.java index ac6ba73d5f7e..dab53e54a80c 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyResolver.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyResolver.java @@ -54,6 +54,7 @@ import org.eclipse.aether.collection.CollectRequest; import org.eclipse.aether.collection.CollectResult; import org.eclipse.aether.collection.DependencyCollectionException; +import org.eclipse.aether.graph.Dependency; import org.eclipse.aether.graph.DependencyFilter; import org.eclipse.aether.graph.DependencyNode; import org.eclipse.aether.scope.ResolutionScope; @@ -154,7 +155,7 @@ private static DependencyFilter getScopeDependencyFilter(PathScope scope) { Set scopes = scope.dependencyScopes().stream().map(DependencyScope::id).collect(Collectors.toSet()); return (n, p) -> { - org.eclipse.aether.graph.Dependency d = n.getDependency(); + Dependency d = n.getDependency(); return d == null || scopes.contains(d.getScope()); }; } diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDownloadedArtifact.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDownloadedArtifact.java index e5dc30e80918..d43905d6f96c 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDownloadedArtifact.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDownloadedArtifact.java @@ -22,6 +22,7 @@ import org.apache.maven.api.DownloadedArtifact; import org.apache.maven.api.annotations.Nonnull; +import org.eclipse.aether.artifact.Artifact; /** * A wrapper class around a maven resolver artifact. @@ -29,7 +30,7 @@ public class DefaultDownloadedArtifact extends DefaultArtifact implements DownloadedArtifact { public DefaultDownloadedArtifact( - @Nonnull InternalSession session, @Nonnull org.eclipse.aether.artifact.Artifact artifact) { + @Nonnull InternalSession session, @Nonnull Artifact artifact) { super(session, artifact); } diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultNode.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultNode.java index 3dfe43bdb89b..986ced246e9f 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultNode.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultNode.java @@ -35,11 +35,11 @@ public class DefaultNode extends AbstractNode { protected final @Nonnull InternalSession session; - protected final @Nonnull org.eclipse.aether.graph.DependencyNode node; + protected final @Nonnull DependencyNode node; protected final boolean verbose; public DefaultNode( - @Nonnull InternalSession session, @Nonnull org.eclipse.aether.graph.DependencyNode node, boolean verbose) { + @Nonnull InternalSession session, @Nonnull DependencyNode node, boolean verbose) { this.session = session; this.node = node; this.verbose = verbose; @@ -114,8 +114,8 @@ public String asString() { } List details = new ArrayList<>(); - org.eclipse.aether.graph.DependencyNode winner = - (org.eclipse.aether.graph.DependencyNode) node.getData().get(ConflictResolver.NODE_DATA_WINNER); + DependencyNode winner = + (DependencyNode) node.getData().get(ConflictResolver.NODE_DATA_WINNER); String winnerVersion = winner != null ? winner.getArtifact().getBaseVersion() : null; boolean included = (winnerVersion == null); diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultProducedArtifact.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultProducedArtifact.java index 42ab1d1e72ab..7120b670db5e 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultProducedArtifact.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultProducedArtifact.java @@ -20,6 +20,7 @@ import org.apache.maven.api.ProducedArtifact; import org.apache.maven.api.annotations.Nonnull; +import org.eclipse.aether.artifact.Artifact; /** * A wrapper class around a maven resolver artifact. @@ -27,7 +28,7 @@ public class DefaultProducedArtifact extends DefaultArtifact implements ProducedArtifact { public DefaultProducedArtifact( - @Nonnull InternalSession session, @Nonnull org.eclipse.aether.artifact.Artifact artifact) { + @Nonnull InternalSession session, @Nonnull Artifact artifact) { super(session, artifact); } } diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultRepositoryFactory.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultRepositoryFactory.java index 9cd9b6a0f4eb..52922ae1c4b9 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultRepositoryFactory.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultRepositoryFactory.java @@ -82,7 +82,7 @@ public List aggregate( .toList(); } - public static org.eclipse.aether.repository.RepositoryPolicy buildRepositoryPolicy( + public static RepositoryPolicy buildRepositoryPolicy( org.apache.maven.api.model.RepositoryPolicy policy) { boolean enabled = true; String updatePolicy = RepositoryPolicy.UPDATE_POLICY_DAILY; @@ -96,6 +96,6 @@ public static org.eclipse.aether.repository.RepositoryPolicy buildRepositoryPoli checksumPolicy = policy.getChecksumPolicy(); } } - return new org.eclipse.aether.repository.RepositoryPolicy(enabled, updatePolicy, checksumPolicy); + return new RepositoryPolicy(enabled, updatePolicy, checksumPolicy); } } diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultTransportProvider.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultTransportProvider.java index 1530f9d75a0d..94442d95bcbd 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultTransportProvider.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultTransportProvider.java @@ -37,7 +37,7 @@ @Named @Singleton public class DefaultTransportProvider implements TransportProvider { - private final org.eclipse.aether.spi.connector.transport.TransporterProvider transporterProvider; + private final TransporterProvider transporterProvider; @Inject public DefaultTransportProvider(TransporterProvider transporterProvider) { diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultVersionRangeResolver.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultVersionRangeResolver.java index df182d976a3a..ed7860b32cb1 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultVersionRangeResolver.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultVersionRangeResolver.java @@ -34,6 +34,8 @@ import org.apache.maven.api.services.VersionRangeResolverResult; import org.eclipse.aether.RepositorySystem; import org.eclipse.aether.repository.ArtifactRepository; +import org.eclipse.aether.repository.LocalRepository; +import org.eclipse.aether.repository.RemoteRepository; import org.eclipse.aether.resolution.VersionRangeRequest; import org.eclipse.aether.resolution.VersionRangeResolutionException; import org.eclipse.aether.resolution.VersionRangeResult; @@ -99,9 +101,9 @@ public List getVersions() { @Override public Optional getRepository(Version version) { ArtifactRepository repo = repos.get(version.toString()); - if (repo instanceof org.eclipse.aether.repository.LocalRepository localRepository) { + if (repo instanceof LocalRepository localRepository) { return Optional.of(new DefaultLocalRepository(localRepository)); - } else if (repo instanceof org.eclipse.aether.repository.RemoteRepository remoteRepository) { + } else if (repo instanceof RemoteRepository remoteRepository) { return Optional.of(new DefaultRemoteRepository(remoteRepository)); } else { return Optional.empty(); diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultVersionResolver.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultVersionResolver.java index c80a1d24ad1d..42b72678c839 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultVersionResolver.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultVersionResolver.java @@ -31,6 +31,8 @@ import org.apache.maven.api.services.VersionResolverRequest; import org.apache.maven.api.services.VersionResolverResult; import org.eclipse.aether.RepositorySystem; +import org.eclipse.aether.repository.LocalRepository; +import org.eclipse.aether.repository.RemoteRepository; import org.eclipse.aether.resolution.VersionRequest; import org.eclipse.aether.resolution.VersionResolutionException; import org.eclipse.aether.resolution.VersionResult; @@ -87,10 +89,10 @@ public Version getVersion() { @Override public Optional getRepository() { - if (res.getRepository() instanceof org.eclipse.aether.repository.LocalRepository localRepository) { + if (res.getRepository() instanceof LocalRepository localRepository) { return Optional.of(new DefaultLocalRepository(localRepository)); } else if (res.getRepository() - instanceof org.eclipse.aether.repository.RemoteRepository remoteRepository) { + instanceof RemoteRepository remoteRepository) { return Optional.of(new DefaultRemoteRepository(remoteRepository)); } else { return Optional.empty(); diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/InternalSession.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/InternalSession.java index e39836e2552d..8a8c4e8985c6 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/InternalSession.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/InternalSession.java @@ -39,6 +39,8 @@ import org.apache.maven.api.services.Result; import org.eclipse.aether.RepositorySystem; import org.eclipse.aether.RepositorySystemSession; +import org.eclipse.aether.graph.DependencyNode; +import org.eclipse.aether.repository.ArtifactRepository; import static org.apache.maven.impl.ImplUtils.cast; @@ -48,11 +50,11 @@ static InternalSession from(Session session) { return cast(InternalSession.class, session, "session should be an " + InternalSession.class); } - static InternalSession from(org.eclipse.aether.RepositorySystemSession session) { + static InternalSession from(RepositorySystemSession session) { return cast(InternalSession.class, session.getData().get(InternalSession.class), "session"); } - static void associate(org.eclipse.aether.RepositorySystemSession rsession, Session session) { + static void associate(RepositorySystemSession rsession, Session session) { if (!rsession.getData().set(InternalSession.class, null, from(session))) { throw new IllegalStateException("A maven session is already associated with the repository session"); } @@ -81,11 +83,11 @@ , REP extends Result> List requests( WorkspaceRepository getWorkspaceRepository(org.eclipse.aether.repository.WorkspaceRepository repository); - Repository getRepository(org.eclipse.aether.repository.ArtifactRepository repository); + Repository getRepository(ArtifactRepository repository); - Node getNode(org.eclipse.aether.graph.DependencyNode node); + Node getNode(DependencyNode node); - Node getNode(org.eclipse.aether.graph.DependencyNode node, boolean verbose); + Node getNode(DependencyNode node, boolean verbose); @Nonnull Artifact getArtifact(@Nonnull org.eclipse.aether.artifact.Artifact artifact); diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/PathSelector.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/PathSelector.java index 05401739341e..0026052042de 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/PathSelector.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/PathSelector.java @@ -59,7 +59,7 @@ * @author Benjamin Bentmann * @author Martin Desruisseaux * - * @see java.nio.file.FileSystem#getPathMatcher(String) + * @see FileSystem#getPathMatcher(String) */ public class PathSelector implements PathMatcher { /** diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/RequestTraceHelper.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/RequestTraceHelper.java index 44b54003f25d..59b0193b9870 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/RequestTraceHelper.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/RequestTraceHelper.java @@ -21,6 +21,7 @@ import java.util.stream.Collectors; import org.apache.maven.api.Session; +import org.apache.maven.api.model.Plugin; import org.apache.maven.api.services.Request; import org.eclipse.aether.RequestTrace; import org.eclipse.aether.collection.CollectRequest; @@ -127,7 +128,7 @@ public static String interpretTrace(boolean detailed, RequestTrace requestTrace) return "artifact descriptor request for " + request.getArtifact(); } else if (data instanceof ArtifactRequest request) { return "artifact request for " + request.getArtifact(); - } else if (data instanceof org.apache.maven.api.model.Plugin plugin) { + } else if (data instanceof Plugin plugin) { return "plugin " + plugin.getId(); } requestTrace = requestTrace.getParent(); diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/SettingsUtilsV4.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/SettingsUtilsV4.java index 8a5d1e81a2bb..7e9470c46634 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/SettingsUtilsV4.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/SettingsUtilsV4.java @@ -45,6 +45,7 @@ import org.apache.maven.api.model.ActivationFile; import org.apache.maven.api.model.InputLocation; +import org.apache.maven.api.model.InputSource; import org.apache.maven.api.settings.Activation; import org.apache.maven.api.settings.ActivationOS; import org.apache.maven.api.settings.ActivationProperty; @@ -128,7 +129,7 @@ public static Profile convertToSettingsProfile(org.apache.maven.api.model.Profil activation.os(os); } - org.apache.maven.api.model.ActivationFile modelFile = modelActivation.getFile(); + ActivationFile modelFile = modelActivation.getFile(); if (modelFile != null) { org.apache.maven.api.settings.ActivationFile file = @@ -356,9 +357,9 @@ private static RepositoryPolicy convertRepositoryPolicy(org.apache.maven.api.mod } private static org.apache.maven.api.settings.InputLocation toLocation( - org.apache.maven.api.model.InputLocation location) { + InputLocation location) { if (location != null) { - org.apache.maven.api.model.InputSource source = location.getSource(); + InputSource source = location.getSource(); Map locs = location.getLocations().entrySet().stream() .collect(Collectors.toMap(Map.Entry::getKey, e -> toLocation(e.getValue()))); return new org.apache.maven.api.settings.InputLocation( @@ -371,16 +372,16 @@ private static org.apache.maven.api.settings.InputLocation toLocation( } } - private static org.apache.maven.api.model.InputLocation toLocation( + private static InputLocation toLocation( org.apache.maven.api.settings.InputLocation location) { if (location != null) { org.apache.maven.api.settings.InputSource source = location.getSource(); Map locs = location.getLocations().entrySet().stream() .collect(Collectors.toMap(Map.Entry::getKey, e -> toLocation(e.getValue()))); - return new org.apache.maven.api.model.InputLocation( + return new InputLocation( location.getLineNumber(), location.getColumnNumber(), - source != null ? new org.apache.maven.api.model.InputSource("", source.getLocation()) : null, + source != null ? new InputSource("", source.getLocation()) : null, locs); } else { return null; diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/di/SessionScope.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/di/SessionScope.java index b47c5acde830..75d8072975e0 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/di/SessionScope.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/di/SessionScope.java @@ -22,6 +22,7 @@ import java.lang.reflect.InvocationHandler; import java.lang.reflect.InvocationTargetException; import java.lang.reflect.Method; +import java.lang.reflect.Proxy; import java.util.Collection; import java.util.List; import java.util.Map; @@ -104,7 +105,7 @@ protected T createProxy(Key key, Supplier unscoped) { InvocationHandler dispatcher = (proxy, method, args) -> dispatch(key, unscoped, method, args); Class superType = (Class) Types.getRawType(key.getType()); Class[] interfaces = getInterfaces(superType); - return (T) java.lang.reflect.Proxy.newProxyInstance(superType.getClassLoader(), interfaces, dispatcher); + return (T) Proxy.newProxyInstance(superType.getClassLoader(), interfaces, dispatcher); } protected Object dispatch(Key key, Supplier unscoped, Method method, Object[] args) throws Throwable { diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelBuilder.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelBuilder.java index 82ebd5a770f1..3a2c78f6a884 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelBuilder.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelBuilder.java @@ -1916,8 +1916,8 @@ private Model getSuperModel(String modelVersion) { return superPomProvider.getSuperPom(modelVersion); } - private static org.apache.maven.api.model.Dependency addExclusions( - org.apache.maven.api.model.Dependency candidate, List exclusions) { + private static Dependency addExclusions( + Dependency candidate, List exclusions) { return candidate.withExclusions(Stream.concat(candidate.getExclusions().stream(), exclusions.stream()) .toList()); } diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelProcessor.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelProcessor.java index bcd0a191f8c3..0f04f2c7543d 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelProcessor.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelProcessor.java @@ -33,6 +33,7 @@ import org.apache.maven.api.di.Named; import org.apache.maven.api.di.Singleton; import org.apache.maven.api.model.Model; +import org.apache.maven.api.services.Source; import org.apache.maven.api.services.model.ModelProcessor; import org.apache.maven.api.services.xml.ModelXmlFactory; import org.apache.maven.api.services.xml.XmlReaderRequest; @@ -83,7 +84,7 @@ public Path locateExistingPom(Path projectDirectory) { // while the ModelParser#locatePom needs to return an existing path! Path pom = modelParsers.stream() .map(m -> m.locate(projectDirectory) - .map(org.apache.maven.api.services.Source::getPath) + .map(Source::getPath) .orElse(null)) .filter(Objects::nonNull) .findFirst() diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/model/profile/ConditionProfileActivator.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/model/profile/ConditionProfileActivator.java index 98aa2c037639..c89f871ecf43 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/model/profile/ConditionProfileActivator.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/model/profile/ConditionProfileActivator.java @@ -18,6 +18,7 @@ */ package org.apache.maven.impl.model.profile; +import java.lang.reflect.Method; import java.util.HashMap; import java.util.Map; import java.util.function.UnaryOperator; @@ -116,7 +117,7 @@ public Map registerFunctions( ConditionFunctions conditionFunctions = new ConditionFunctions(context, versionParser); - for (java.lang.reflect.Method method : ConditionFunctions.class.getDeclaredMethods()) { + for (Method method : ConditionFunctions.class.getDeclaredMethods()) { String methodName = method.getName(); if (methodName.endsWith("_")) { methodName = methodName.substring(0, methodName.length() - 1); diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/resolver/scopes/Maven4ScopeManagerConfiguration.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/resolver/scopes/Maven4ScopeManagerConfiguration.java index 896b240053f6..d78aa19398d0 100644 --- a/impl/maven-impl/src/main/java/org/apache/maven/impl/resolver/scopes/Maven4ScopeManagerConfiguration.java +++ b/impl/maven-impl/src/main/java/org/apache/maven/impl/resolver/scopes/Maven4ScopeManagerConfiguration.java @@ -32,6 +32,7 @@ import org.eclipse.aether.impl.scope.InternalScopeManager; import org.eclipse.aether.impl.scope.ScopeManagerConfiguration; import org.eclipse.aether.internal.impl.scope.ScopeManagerDump; +import org.eclipse.aether.scope.ResolutionScope; import static org.eclipse.aether.impl.scope.BuildScopeQuery.all; import static org.eclipse.aether.impl.scope.BuildScopeQuery.byBuildPath; @@ -133,7 +134,7 @@ public Collection buildDependencyScope } @Override - public Collection buildResolutionScopes( + public Collection buildResolutionScopes( InternalScopeManager internalScopeManager) { Collection allDependencyScopes = internalScopeManager.getDependencyScopeUniverse(); @@ -143,7 +144,7 @@ public Collection buildResolutionScope .getDependencyScope(DependencyScope.SYSTEM.id()) .orElse(null); - ArrayList result = new ArrayList<>(); + ArrayList result = new ArrayList<>(); result.add(internalScopeManager.createResolutionScope( RS_NONE, InternalScopeManager.Mode.REMOVE, diff --git a/impl/maven-impl/src/test/java/org/apache/maven/impl/DefaultSettingsValidatorTest.java b/impl/maven-impl/src/test/java/org/apache/maven/impl/DefaultSettingsValidatorTest.java index f604fe6bb3af..e587e8bbddf9 100644 --- a/impl/maven-impl/src/test/java/org/apache/maven/impl/DefaultSettingsValidatorTest.java +++ b/impl/maven-impl/src/test/java/org/apache/maven/impl/DefaultSettingsValidatorTest.java @@ -49,7 +49,7 @@ void testValidate() { ProblemCollector problems = validator.validate(model); assertEquals(0, problems.totalProblemsReported()); - Repository repo = org.apache.maven.api.settings.Repository.newInstance(false); + Repository repo = Repository.newInstance(false); Settings model2 = Settings.newBuilder() .profiles(List.of(prof.withRepositories(List.of(repo)))) .build(); diff --git a/impl/maven-impl/src/test/java/org/apache/maven/impl/cache/AbstractRequestCacheTest.java b/impl/maven-impl/src/test/java/org/apache/maven/impl/cache/AbstractRequestCacheTest.java index 6683bee0de95..646cf394878c 100644 --- a/impl/maven-impl/src/test/java/org/apache/maven/impl/cache/AbstractRequestCacheTest.java +++ b/impl/maven-impl/src/test/java/org/apache/maven/impl/cache/AbstractRequestCacheTest.java @@ -18,8 +18,7 @@ */ package org.apache.maven.impl.cache; -import java.util.Arrays; -import java.util.List; +import java.util.*; import java.util.function.Function; import org.apache.maven.api.ProtoSession; @@ -199,12 +198,12 @@ public boolean equals(Object obj) { return false; } TestRequestImpl that = (TestRequestImpl) obj; - return java.util.Objects.equals(id, that.id); + return Objects.equals(id, that.id); } @Override public int hashCode() { - return java.util.Objects.hash(id); + return Objects.hash(id); } @Override @@ -229,7 +228,7 @@ public TestRequest getRequest() { } static class TestRequestCache extends AbstractRequestCache { - private final java.util.Map failures = new java.util.HashMap<>(); + private final Map failures = new HashMap<>(); void addFailure(TestRequest request, RuntimeException exception) { failures.put(request, exception); diff --git a/impl/maven-impl/src/test/java/org/apache/maven/impl/resolver/DefaultModelResolverTest.java b/impl/maven-impl/src/test/java/org/apache/maven/impl/resolver/DefaultModelResolverTest.java index e680267895ad..fc1d07785429 100644 --- a/impl/maven-impl/src/test/java/org/apache/maven/impl/resolver/DefaultModelResolverTest.java +++ b/impl/maven-impl/src/test/java/org/apache/maven/impl/resolver/DefaultModelResolverTest.java @@ -133,7 +133,7 @@ void testResolveParentSuccessfullyResolvesExistingParentUsingHighestVersion() th .version("(,2.0)") .build(); - AtomicReference modified = new AtomicReference<>(); + AtomicReference modified = new AtomicReference<>(); assertNotNull(this.newModelResolver().resolveModel(session, null, parent, modified)); assertEquals("1", modified.get().getVersion()); } @@ -204,7 +204,7 @@ void testResolveDependencySuccessfullyResolvesExistingDependencyUsingHighestVers .version("(,2.0)") .build(); - AtomicReference modified = new AtomicReference<>(); + AtomicReference modified = new AtomicReference<>(); assertNotNull(this.newModelResolver().resolveModel(session, null, dependency, modified)); assertEquals("1", modified.get().getVersion()); } diff --git a/impl/maven-logging/src/main/java/org/apache/maven/slf4j/SimpleLoggerConfiguration.java b/impl/maven-logging/src/main/java/org/apache/maven/slf4j/SimpleLoggerConfiguration.java index d07f66179fc4..5b2cf203707c 100644 --- a/impl/maven-logging/src/main/java/org/apache/maven/slf4j/SimpleLoggerConfiguration.java +++ b/impl/maven-logging/src/main/java/org/apache/maven/slf4j/SimpleLoggerConfiguration.java @@ -18,10 +18,7 @@ */ package org.apache.maven.slf4j; -import java.io.FileNotFoundException; -import java.io.FileOutputStream; -import java.io.InputStream; -import java.io.PrintStream; +import java.io.*; import java.time.format.DateTimeFormatter; import java.util.Properties; @@ -139,7 +136,7 @@ private void loadProperties() { properties.load(in); mavenPropsLoaded = true; } - } catch (java.io.IOException e) { + } catch (IOException e) { // ignored } @@ -160,7 +157,7 @@ private void loadProperties() { } } } - } catch (java.io.IOException e) { + } catch (IOException e) { // ignored } } diff --git a/impl/maven-xml/src/main/java/org/apache/maven/internal/xml/DefaultXmlService.java b/impl/maven-xml/src/main/java/org/apache/maven/internal/xml/DefaultXmlService.java index bb99243ccf9d..6de97167d7eb 100644 --- a/impl/maven-xml/src/main/java/org/apache/maven/internal/xml/DefaultXmlService.java +++ b/impl/maven-xml/src/main/java/org/apache/maven/internal/xml/DefaultXmlService.java @@ -39,6 +39,8 @@ import java.util.stream.Collectors; import java.util.stream.Stream; +import com.ctc.wstx.api.WstxOutputProperties; +import com.ctc.wstx.stax.WstxOutputFactory; import org.apache.maven.api.annotations.Nonnull; import org.apache.maven.api.annotations.Nullable; import org.apache.maven.api.xml.XmlNode; @@ -147,10 +149,10 @@ private XmlNode doBuild(XMLStreamReader parser, boolean trim, InputLocationBuild @Override public void doWrite(XmlNode node, Writer writer) throws IOException { try { - XMLOutputFactory factory = new com.ctc.wstx.stax.WstxOutputFactory(); + XMLOutputFactory factory = new WstxOutputFactory(); factory.setProperty(XMLOutputFactory.IS_REPAIRING_NAMESPACES, false); - factory.setProperty(com.ctc.wstx.api.WstxOutputProperties.P_USE_DOUBLE_QUOTES_IN_XML_DECL, true); - factory.setProperty(com.ctc.wstx.api.WstxOutputProperties.P_ADD_SPACE_AFTER_EMPTY_ELEM, true); + factory.setProperty(WstxOutputProperties.P_USE_DOUBLE_QUOTES_IN_XML_DECL, true); + factory.setProperty(WstxOutputProperties.P_ADD_SPACE_AFTER_EMPTY_ELEM, true); XMLStreamWriter serializer = new IndentingXMLStreamWriter(factory.createXMLStreamWriter(writer)); writeNode(serializer, node); serializer.close(); diff --git a/impl/maven-xml/src/test/java/org/apache/maven/internal/xml/XmlNodeImplTest.java b/impl/maven-xml/src/test/java/org/apache/maven/internal/xml/XmlNodeImplTest.java index af2fcd3dbab7..901e7877eb04 100644 --- a/impl/maven-xml/src/test/java/org/apache/maven/internal/xml/XmlNodeImplTest.java +++ b/impl/maven-xml/src/test/java/org/apache/maven/internal/xml/XmlNodeImplTest.java @@ -191,7 +191,7 @@ void testAppend() throws Exception { /** *

testCombineId.

* - * @throws java.lang.Exception if any. + * @throws Exception if any. */ @Test void testCombineId() throws Exception { @@ -237,7 +237,7 @@ void testCombineId() throws Exception { /** *

testCombineKeys.

* - * @throws java.lang.Exception if any. + * @throws Exception if any. */ @Test void testCombineKeys() throws Exception { @@ -548,7 +548,7 @@ void testShouldMergePluginConfigurationSubItemsWithMergeAttributeSet() throws XM /** *

testShouldNotChangeUponMergeWithItselfWhenFirstOrLastSubItemIsEmpty.

* - * @throws java.lang.Exception if any. + * @throws Exception if any. */ @Test void testShouldNotChangeUponMergeWithItselfWhenFirstOrLastSubItemIsEmpty() throws Exception { @@ -569,7 +569,7 @@ void testShouldNotChangeUponMergeWithItselfWhenFirstOrLastSubItemIsEmpty() throw /** *

testShouldCopyRecessiveChildrenNotPresentInTarget.

* - * @throws java.lang.Exception if any. + * @throws Exception if any. */ @Test void testShouldCopyRecessiveChildrenNotPresentInTarget() throws Exception { @@ -601,7 +601,7 @@ void testDupeChildren() throws IOException, XMLStreamException { /** *

testShouldRemoveEntireElementWithAttributesAndChildren.

* - * @throws java.lang.Exception if any. + * @throws Exception if any. */ @Test void testShouldRemoveEntireElementWithAttributesAndChildren() throws Exception { @@ -619,7 +619,7 @@ void testShouldRemoveEntireElementWithAttributesAndChildren() throws Exception { /** *

testShouldRemoveDoNotRemoveTagWhenSwappedInputDOMs.

* - * @throws java.lang.Exception if any. + * @throws Exception if any. */ @Test void testShouldRemoveDoNotRemoveTagWhenSwappedInputDOMs() throws Exception { diff --git a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng1751ForcedMetadataUpdateDuringDeploymentTest.java b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng1751ForcedMetadataUpdateDuringDeploymentTest.java index 06382a6e139b..6512f649bd99 100644 --- a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng1751ForcedMetadataUpdateDuringDeploymentTest.java +++ b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng1751ForcedMetadataUpdateDuringDeploymentTest.java @@ -21,6 +21,7 @@ import java.io.File; import java.nio.file.Files; import java.nio.file.StandardCopyOption; +import java.util.Locale; import java.util.Properties; import org.junit.jupiter.api.Test; @@ -76,7 +77,7 @@ public void testit() throws Exception { verifier.verifyErrorFreeLog(); Properties checksums = verifier.loadProperties("target/checksum.properties"); - String sha1 = checksums.getProperty("dep-0.1-SNAPSHOT.jar", "").toLowerCase(java.util.Locale.ENGLISH); + String sha1 = checksums.getProperty("dep-0.1-SNAPSHOT.jar", "").toLowerCase(Locale.ENGLISH); assertEquals(40, sha1.length(), sha1); assertNotEquals("fc081cd365b837dcb01eb9991f21c409b155ea5c", sha1); } diff --git a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4231SnapshotUpdatePolicyTest.java b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4231SnapshotUpdatePolicyTest.java index 7acb83c6422f..94a5459bacc7 100644 --- a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4231SnapshotUpdatePolicyTest.java +++ b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4231SnapshotUpdatePolicyTest.java @@ -19,6 +19,7 @@ package org.apache.maven.it; import java.io.File; +import java.util.Locale; import java.util.Map; import java.util.Properties; @@ -114,6 +115,6 @@ public void testitNever() throws Exception { } private void assertChecksum(String checksum, String jar, Properties checksums) { - assertEquals(checksum, checksums.getProperty(jar, "").toLowerCase(java.util.Locale.ENGLISH)); + assertEquals(checksum, checksums.getProperty(jar, "").toLowerCase(Locale.ENGLISH)); } } diff --git a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4281PreferLocalSnapshotTest.java b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4281PreferLocalSnapshotTest.java index 417dff1cdf1c..30c368a6a6b8 100644 --- a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4281PreferLocalSnapshotTest.java +++ b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4281PreferLocalSnapshotTest.java @@ -19,6 +19,7 @@ package org.apache.maven.it; import java.io.File; +import java.util.Locale; import java.util.Properties; import org.junit.jupiter.api.Test; @@ -72,6 +73,6 @@ public void testit() throws Exception { private void assertChecksum(String checksum, Properties checksums) { String actual = checksums.getProperty("dependency-0.1-SNAPSHOT.jar"); - assertEquals(checksum, actual.toLowerCase(java.util.Locale.ENGLISH)); + assertEquals(checksum, actual.toLowerCase(Locale.ENGLISH)); } } diff --git a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4326LocalSnapshotSuppressesRemoteCheckTest.java b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4326LocalSnapshotSuppressesRemoteCheckTest.java index 1ea9298444fb..bad933310ac3 100644 --- a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4326LocalSnapshotSuppressesRemoteCheckTest.java +++ b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4326LocalSnapshotSuppressesRemoteCheckTest.java @@ -24,10 +24,9 @@ import java.io.File; import java.io.IOException; import java.io.PrintWriter; -import java.util.Date; -import java.util.Deque; -import java.util.List; -import java.util.Map; +import java.text.DateFormat; +import java.text.SimpleDateFormat; +import java.util.*; import java.util.concurrent.ConcurrentLinkedDeque; import org.eclipse.jetty.server.Handler; @@ -88,8 +87,8 @@ public void handle( } if (uri.endsWith("dep/0.1-SNAPSHOT/maven-metadata.xml")) { - java.text.DateFormat fmt = new java.text.SimpleDateFormat("yyyyMMddHHmmss"); - fmt.setTimeZone(java.util.TimeZone.getTimeZone("UTC")); + DateFormat fmt = new SimpleDateFormat("yyyyMMddHHmmss"); + fmt.setTimeZone(TimeZone.getTimeZone("UTC")); String now = fmt.format(new Date(System.currentTimeMillis() + 3000)); response.setStatus(HttpServletResponse.SC_OK); diff --git a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4361ForceDependencySnapshotUpdateTest.java b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4361ForceDependencySnapshotUpdateTest.java index 2354f80613ad..4f9696ec7b3d 100644 --- a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4361ForceDependencySnapshotUpdateTest.java +++ b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4361ForceDependencySnapshotUpdateTest.java @@ -19,6 +19,7 @@ package org.apache.maven.it; import java.io.File; +import java.util.Locale; import java.util.Map; import java.util.Properties; @@ -81,6 +82,6 @@ public void testit() throws Exception { } private void assertChecksum(String checksum, String jar, Properties checksums) { - assertEquals(checksum, checksums.getProperty(jar, "").toLowerCase(java.util.Locale.ENGLISH)); + assertEquals(checksum, checksums.getProperty(jar, "").toLowerCase(Locale.ENGLISH)); } } diff --git a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4400RepositoryOrderTest.java b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4400RepositoryOrderTest.java index 56a86b2edb8d..035dfdfaa484 100644 --- a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4400RepositoryOrderTest.java +++ b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4400RepositoryOrderTest.java @@ -19,6 +19,7 @@ package org.apache.maven.it; import java.io.File; +import java.util.Locale; import java.util.Properties; import org.junit.jupiter.api.Test; @@ -83,6 +84,6 @@ public void testitPomRepos() throws Exception { } private void assertChecksum(String checksum, Properties checksums) { - assertEquals(checksum, checksums.getProperty("dep-0.1.jar").toLowerCase(java.util.Locale.ENGLISH)); + assertEquals(checksum, checksums.getProperty("dep-0.1.jar").toLowerCase(Locale.ENGLISH)); } } diff --git a/its/core-it-suite/src/test/resources/mng-3259/module4/src/main/java/mng/Module4Home.java b/its/core-it-suite/src/test/resources/mng-3259/module4/src/main/java/mng/Module4Home.java index 1dca6dba638f..eb8cc0ba03fb 100644 --- a/its/core-it-suite/src/test/resources/mng-3259/module4/src/main/java/mng/Module4Home.java +++ b/its/core-it-suite/src/test/resources/mng-3259/module4/src/main/java/mng/Module4Home.java @@ -18,6 +18,8 @@ */ package mng; +import java.rmi.RemoteException; + /** * Home interface for Module4. */ @@ -25,5 +27,5 @@ public interface Module4Home extends javax.ejb.EJBHome { public static final String COMP_NAME = "java:comp/env/ejb/Module4"; public static final String JNDI_NAME = "mng/Module4"; - public Module4 create() throws javax.ejb.CreateException, java.rmi.RemoteException; + public Module4 create() throws javax.ejb.CreateException, RemoteException; } diff --git a/its/core-it-suite/src/test/resources/mng-6506-package-annotation/plugin/src/main/java/testmojo/package-info.java b/its/core-it-suite/src/test/resources/mng-6506-package-annotation/plugin/src/main/java/testmojo/package-info.java index d0b9f39ae7c7..251b7e74a6bc 100644 --- a/its/core-it-suite/src/test/resources/mng-6506-package-annotation/plugin/src/main/java/testmojo/package-info.java +++ b/its/core-it-suite/src/test/resources/mng-6506-package-annotation/plugin/src/main/java/testmojo/package-info.java @@ -17,5 +17,5 @@ * under the License. */ -@testmojo.TestAnnotation +@TestAnnotation package testmojo; diff --git a/its/core-it-support/core-it-javaagent/src/main/java/org/apache/maven/coreits/javaagent/mng5669/Premain.java b/its/core-it-support/core-it-javaagent/src/main/java/org/apache/maven/coreits/javaagent/mng5669/Premain.java index 7f53da774f92..4473f0af2f34 100644 --- a/its/core-it-support/core-it-javaagent/src/main/java/org/apache/maven/coreits/javaagent/mng5669/Premain.java +++ b/its/core-it-support/core-it-javaagent/src/main/java/org/apache/maven/coreits/javaagent/mng5669/Premain.java @@ -62,7 +62,7 @@ public byte[] transform( } private static class DefaultModelReaderVisitor extends ClassVisitor { - DefaultModelReaderVisitor(int api, org.objectweb.asm.ClassVisitor cv) { + DefaultModelReaderVisitor(int api, ClassVisitor cv) { super(api, cv); } diff --git a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/AppendConfigMojo.java b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/AppendConfigMojo.java index 3535c9929886..87d131078c30 100644 --- a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/AppendConfigMojo.java +++ b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/AppendConfigMojo.java @@ -61,54 +61,54 @@ public class AppendConfigMojo extends AbstractMojo { private String defaultParam; /** - * A simple parameter of type {@link java.lang.String}. + * A simple parameter of type {@link String}. */ @Parameter(property = "config.stringParam") private String stringParam; /** - * A simple parameter of type {@link java.io.File}. + * A simple parameter of type {@link File}. */ @Parameter(property = "config.fileParam") private File fileParam; /** - * An array parameter of component type {@link java.lang.String}. + * An array parameter of component type {@link String}. * */ @Parameter private String[] stringParams; /** - * An array parameter of component type {@link java.io.File}. + * An array parameter of component type {@link File}. * */ @Parameter private File[] fileParams; /** - * A collection parameter of type {@link java.util.List}. + * A collection parameter of type {@link List}. * */ @Parameter private List listParam; /** - * A collection parameter of type {@link java.util.Set}. + * A collection parameter of type {@link Set}. * */ @Parameter private Set setParam; /** - * A collection parameter of type {@link java.util.Map}. + * A collection parameter of type {@link Map}. * */ @Parameter private Map mapParam; /** - * A collection parameter of type {@link java.util.Properties}. + * A collection parameter of type {@link Properties}. * */ @Parameter diff --git a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CliConfigMojo.java b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CliConfigMojo.java index cd2fc4217ef5..4c7790a968eb 100644 --- a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CliConfigMojo.java +++ b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CliConfigMojo.java @@ -52,37 +52,37 @@ public class CliConfigMojo extends AbstractMojo { private File propertiesFile; /** - * A simple parameter of type {@link java.lang.String}. + * A simple parameter of type {@link String}. */ @Parameter(property = "config.stringParam") private String stringParam; /** - * A simple parameter of type {@link java.io.File}. + * A simple parameter of type {@link File}. */ @Parameter(property = "config.fileParam") private File fileParam; /** - * An array parameter of component type {@link java.lang.String}. + * An array parameter of component type {@link String}. */ @Parameter(property = "config.stringParams") private String[] stringParams; /** - * An array parameter of component type {@link java.io.File}. + * An array parameter of component type {@link File}. */ @Parameter(property = "config.fileParams") private File[] fileParams; /** - * A collection parameter of type {@link java.util.List}. + * A collection parameter of type {@link List}. */ @Parameter(property = "config.listParam") private List listParam; /** - * A collection parameter of type {@link java.util.Set}. + * A collection parameter of type {@link Set}. */ @Parameter(property = "config.setParam") private Set setParam; diff --git a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/ConfigMojo.java b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/ConfigMojo.java index 3b3fc8a47ef1..7bacf26254e1 100644 --- a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/ConfigMojo.java +++ b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/ConfigMojo.java @@ -80,133 +80,133 @@ public class ConfigMojo extends AbstractMojo { private String aliasDefaultExpressionParam; /** - * A simple parameter of type {@link java.lang.Boolean}. + * A simple parameter of type {@link Boolean}. */ @Parameter(property = "config.booleanParam") private Boolean booleanParam; /** - * A simple parameter of type {@link java.lang.Boolean#TYPE}. + * A simple parameter of type {@link Boolean#TYPE}. */ @Parameter(property = "config.primitiveBooleanParam") private boolean primitiveBooleanParam; /** - * A simple parameter of type {@link java.lang.Byte}. + * A simple parameter of type {@link Byte}. */ @Parameter(property = "config.byteParam") private Byte byteParam; /** - * A simple parameter of type {@link java.lang.Short}. + * A simple parameter of type {@link Short}. */ @Parameter(property = "config.shortParam") private Short shortParam; /** - * A simple parameter of type {@link java.lang.Integer}. + * A simple parameter of type {@link Integer}. */ @Parameter(property = "config.integerParam") private Integer integerParam; /** - * A simple parameter of type {@link java.lang.Integer#TYPE}. + * A simple parameter of type {@link Integer#TYPE}. */ @Parameter(property = "config.primitiveIntegerParam") private int primitiveIntegerParam; /** - * A simple parameter of type {@link java.lang.Long}. + * A simple parameter of type {@link Long}. */ @Parameter(property = "config.longParam") private Long longParam; /** - * A simple parameter of type {@link java.lang.Float}. + * A simple parameter of type {@link Float}. */ @Parameter(property = "config.floatParam") private Float floatParam; /** - * A simple parameter of type {@link java.lang.Double}. + * A simple parameter of type {@link Double}. */ @Parameter(property = "config.doubleParam") private Double doubleParam; /** - * A simple parameter of type {@link java.lang.Character}. + * A simple parameter of type {@link Character}. */ @Parameter(property = "config.characterParam") private Character characterParam; /** - * A simple parameter of type {@link java.lang.String}. + * A simple parameter of type {@link String}. */ @Parameter(property = "config.stringParam") private String stringParam; /** - * A simple parameter of type {@link java.io.File}. + * A simple parameter of type {@link File}. */ @Parameter(property = "config.fileParam") private File fileParam; /** - * A simple parameter of type {@link java.util.Date}. + * A simple parameter of type {@link Date}. */ @Parameter(property = "config.dateParam") private Date dateParam; /** - * A simple parameter of type {@link java.net.URL}. + * A simple parameter of type {@link URL}. */ @Parameter(property = "config.urlParam") private URL urlParam; /** - * A simple parameter of type {@link java.net.URI} (requires Maven 3.x). + * A simple parameter of type {@link URI} (requires Maven 3.x). * */ @Parameter private URI uriParam; /** - * An array parameter of component type {@link java.lang.String}. + * An array parameter of component type {@link String}. * */ @Parameter private String[] stringParams; /** - * An array parameter of component type {@link java.io.File}. + * An array parameter of component type {@link File}. * */ @Parameter private File[] fileParams; /** - * A collection parameter of type {@link java.util.List}. + * A collection parameter of type {@link List}. * */ @Parameter private List listParam; /** - * A collection parameter of type {@link java.util.Set}. + * A collection parameter of type {@link Set}. * */ @Parameter private Set setParam; /** - * A collection parameter of type {@link java.util.Map}. + * A collection parameter of type {@link Map}. * */ @Parameter private Map mapParam; /** - * A collection parameter of type {@link java.util.Properties}. + * A collection parameter of type {@link Properties}. * */ @Parameter @@ -219,7 +219,7 @@ public class ConfigMojo extends AbstractMojo { private String[] aliasStringParams; /** - * A complex parameter of type {@link org.apache.maven.plugin.coreit.Bean}. + * A complex parameter of type {@link Bean}. * */ @Parameter diff --git a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CustomComponentConfigurator.java b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CustomComponentConfigurator.java index 299168546461..05fb7f53179f 100644 --- a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CustomComponentConfigurator.java +++ b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CustomComponentConfigurator.java @@ -22,6 +22,7 @@ import org.codehaus.plexus.component.annotations.Component; import org.codehaus.plexus.component.configurator.AbstractComponentConfigurator; import org.codehaus.plexus.component.configurator.ComponentConfigurationException; +import org.codehaus.plexus.component.configurator.ComponentConfigurator; import org.codehaus.plexus.component.configurator.ConfigurationListener; import org.codehaus.plexus.component.configurator.converters.composite.ObjectWithFieldsConverter; import org.codehaus.plexus.component.configurator.expression.ExpressionEvaluator; @@ -33,7 +34,7 @@ * * @author Benjamin Bentmann */ -@Component(role = org.codehaus.plexus.component.configurator.ComponentConfigurator.class, hint = "coreit") +@Component(role = ComponentConfigurator.class, hint = "coreit") public class CustomComponentConfigurator extends AbstractComponentConfigurator { @Override diff --git a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CustomConfigMojo.java b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CustomConfigMojo.java index f7f46d5b70d4..22abb2d29fbe 100644 --- a/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CustomConfigMojo.java +++ b/its/core-it-support/core-it-plugins/maven-it-plugin-configuration/src/main/java/org/apache/maven/plugin/coreit/CustomConfigMojo.java @@ -65,54 +65,54 @@ public class CustomConfigMojo extends AbstractMojo { private String defaultParam; /** - * A simple parameter of type {@link java.lang.String}. + * A simple parameter of type {@link String}. */ @Parameter(property = "config.stringParam") private String stringParam; /** - * A simple parameter of type {@link java.io.File}. + * A simple parameter of type {@link File}. */ @Parameter(property = "config.fileParam") private File fileParam; /** - * An array parameter of component type {@link java.lang.String}. + * An array parameter of component type {@link String}. * */ @Parameter private String[] stringParams; /** - * An array parameter of component type {@link java.io.File}. + * An array parameter of component type {@link File}. * */ @Parameter private File[] fileParams; /** - * A collection parameter of type {@link java.util.List}. + * A collection parameter of type {@link List}. * */ @Parameter private List listParam; /** - * A collection parameter of type {@link java.util.Set}. + * A collection parameter of type {@link Set}. * */ @Parameter private Set setParam; /** - * A collection parameter of type {@link java.util.Map}. + * A collection parameter of type {@link Map}. * */ @Parameter private Map mapParam; /** - * A collection parameter of type {@link java.util.Properties}. + * A collection parameter of type {@link Properties}. * */ @Parameter diff --git a/its/core-it-support/core-it-plugins/maven-it-plugin-plexus-lifecycle/src/main/java/org/apache/maven/its/plugins/plexuslifecycle/DefaultFakeComponent.java b/its/core-it-support/core-it-plugins/maven-it-plugin-plexus-lifecycle/src/main/java/org/apache/maven/its/plugins/plexuslifecycle/DefaultFakeComponent.java index 49a1803c5213..2d4dc03da331 100644 --- a/its/core-it-support/core-it-plugins/maven-it-plugin-plexus-lifecycle/src/main/java/org/apache/maven/its/plugins/plexuslifecycle/DefaultFakeComponent.java +++ b/its/core-it-support/core-it-plugins/maven-it-plugin-plexus-lifecycle/src/main/java/org/apache/maven/its/plugins/plexuslifecycle/DefaultFakeComponent.java @@ -29,7 +29,7 @@ /** * @author Olivier Lamy */ -@Component(role = org.apache.maven.its.plugins.plexuslifecycle.FakeComponent.class) +@Component(role = FakeComponent.class) public class DefaultFakeComponent implements FakeComponent, Contextualizable, Disposable, LogEnabled { private Logger logger; diff --git a/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/coreit/CoreItHttpWagon.java b/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/coreit/CoreItHttpWagon.java index a97aeb52f4dd..fdc49af0f578 100644 --- a/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/coreit/CoreItHttpWagon.java +++ b/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/coreit/CoreItHttpWagon.java @@ -33,6 +33,7 @@ import org.apache.maven.wagon.OutputData; import org.apache.maven.wagon.ResourceDoesNotExistException; import org.apache.maven.wagon.TransferFailedException; +import org.apache.maven.wagon.Wagon; import org.apache.maven.wagon.authentication.AuthenticationException; import org.apache.maven.wagon.authorization.AuthorizationException; import org.apache.maven.wagon.resource.Resource; @@ -41,7 +42,7 @@ /** * Shamelessly copied from ScpExternalWagon in this same project... */ -@Component(role = org.apache.maven.wagon.Wagon.class, hint = "http-coreit", instantiationStrategy = "per-lookup") +@Component(role = Wagon.class, hint = "http-coreit", instantiationStrategy = "per-lookup") public class CoreItHttpWagon extends AbstractWagon { @Override public void get(String resourceName, File destination) diff --git a/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/coreit/CoreItWagon.java b/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/coreit/CoreItWagon.java index 09c494457f80..c0fa6b78e8d6 100644 --- a/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/coreit/CoreItWagon.java +++ b/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/coreit/CoreItWagon.java @@ -33,6 +33,7 @@ import org.apache.maven.wagon.OutputData; import org.apache.maven.wagon.ResourceDoesNotExistException; import org.apache.maven.wagon.TransferFailedException; +import org.apache.maven.wagon.Wagon; import org.apache.maven.wagon.authentication.AuthenticationException; import org.apache.maven.wagon.authentication.AuthenticationInfo; import org.apache.maven.wagon.authorization.AuthorizationException; @@ -42,7 +43,7 @@ /** * Shamelessly copied from ScpExternalWagon in this same project... */ -@Component(role = org.apache.maven.wagon.Wagon.class, hint = "coreit", instantiationStrategy = "per-lookup") +@Component(role = Wagon.class, hint = "coreit", instantiationStrategy = "per-lookup") public class CoreItWagon extends AbstractWagon { @Override public void get(String resourceName, File destination) diff --git a/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/ssh/external/ScpExternalWagon.java b/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/ssh/external/ScpExternalWagon.java index 6c601dc9a46c..982a553c59df 100644 --- a/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/ssh/external/ScpExternalWagon.java +++ b/its/core-it-support/core-it-wagon/src/main/java/org/apache/maven/wagon/providers/ssh/external/ScpExternalWagon.java @@ -33,6 +33,7 @@ import org.apache.maven.wagon.OutputData; import org.apache.maven.wagon.ResourceDoesNotExistException; import org.apache.maven.wagon.TransferFailedException; +import org.apache.maven.wagon.Wagon; import org.apache.maven.wagon.authentication.AuthenticationException; import org.apache.maven.wagon.authorization.AuthorizationException; import org.apache.maven.wagon.resource.Resource; @@ -42,7 +43,7 @@ * NOTE: Plexus will only pick this correctly if the Class package and name are the same as that in core. This is * because the core component descriptor is read, but the class is read from the latter JAR. */ -@Component(role = org.apache.maven.wagon.Wagon.class, hint = "scpexe", instantiationStrategy = "per-lookup") +@Component(role = Wagon.class, hint = "scpexe", instantiationStrategy = "per-lookup") public class ScpExternalWagon extends AbstractWagon { @Override public void get(String resourceName, File destination) diff --git a/its/core-it-support/maven-it-helper/src/main/java/org/apache/maven/it/AbstractMavenIntegrationTestCase.java b/its/core-it-support/maven-it-helper/src/main/java/org/apache/maven/it/AbstractMavenIntegrationTestCase.java index 744dcfd8c6d7..971a98cc94fa 100644 --- a/its/core-it-support/maven-it-helper/src/main/java/org/apache/maven/it/AbstractMavenIntegrationTestCase.java +++ b/its/core-it-support/maven-it-helper/src/main/java/org/apache/maven/it/AbstractMavenIntegrationTestCase.java @@ -31,6 +31,7 @@ import org.apache.maven.artifact.versioning.DefaultArtifactVersion; import org.apache.maven.artifact.versioning.InvalidVersionSpecificationException; import org.apache.maven.artifact.versioning.VersionRange; +import org.junit.jupiter.api.Assertions; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.TestInfo; @@ -318,6 +319,6 @@ protected Verifier newVerifier(String basedir, String settings, boolean debug) t } public static void fail(String message) { - org.junit.jupiter.api.Assertions.fail(message); + Assertions.fail(message); } } diff --git a/src/mdo/java/WrapperProperties.java b/src/mdo/java/WrapperProperties.java index 8e1af9a71678..a0fe9b91c503 100644 --- a/src/mdo/java/WrapperProperties.java +++ b/src/mdo/java/WrapperProperties.java @@ -18,14 +18,7 @@ */ package ${package}; -import java.io.IOError; -import java.io.IOException; -import java.io.InputStream; -import java.io.OutputStream; -import java.io.PrintStream; -import java.io.PrintWriter; -import java.io.Reader; -import java.io.Writer; +import java.io.*; import java.util.AbstractSet; import java.util.ArrayList; import java.util.Collection; @@ -363,7 +356,7 @@ public void storeToXML(OutputStream os, String comment, String encoding) throws } - private Object writeReplace() throws java.io.ObjectStreamException { + private Object writeReplace() throws ObjectStreamException { Properties props = new Properties(); props.putAll(getter.get()); return props;