diff --git a/src/it/projects/mdep-689-apply-filtering-go-offline-goal/pom.xml b/src/it/projects/mdep-689-apply-filtering-go-offline-goal/pom.xml index a41847d9e..0fee5f399 100644 --- a/src/it/projects/mdep-689-apply-filtering-go-offline-goal/pom.xml +++ b/src/it/projects/mdep-689-apply-filtering-go-offline-goal/pom.xml @@ -95,8 +95,8 @@ under the License. skip.this.groupid,skip.this.groupid.too skip-this-artifact,skip-this-artifact-too - + system ear diff --git a/src/main/java/org/apache/maven/plugins/dependency/fromConfiguration/ArtifactItem.java b/src/main/java/org/apache/maven/plugins/dependency/fromConfiguration/ArtifactItem.java index b98b4567c..ff093978b 100644 --- a/src/main/java/org/apache/maven/plugins/dependency/fromConfiguration/ArtifactItem.java +++ b/src/main/java/org/apache/maven/plugins/dependency/fromConfiguration/ArtifactItem.java @@ -36,37 +36,37 @@ */ public class ArtifactItem implements DependableCoordinate { /** - * Group ID of Artifact + * Group ID of artifact */ @Parameter(required = true) private String groupId; /** - * Name of Artifact + * Name of artifact */ @Parameter(required = true) private String artifactId; /** - * Version of Artifact + * Version of artifact */ @Parameter private String version = null; /** - * Type of Artifact (War,Jar,etc) + * Type of artifact (War, Jar, etc.) */ @Parameter(required = true) private String type = "jar"; /** - * Classifier for Artifact (tests,sources,etc) + * Classifier for artifact (tests, sources, etc.) */ @Parameter private String classifier; /** - * Location to use for this Artifact. Overrides default location. + * Location to use for this artifact. Overrides default location. */ @Parameter private File outputDirectory; @@ -78,7 +78,7 @@ public class ArtifactItem implements DependableCoordinate { private String destFileName; /** - * Force Overwrite..this is the one to set in pom + * Force Overwrite. This is the one to set in pom. */ private String overWrite; @@ -114,7 +114,7 @@ public class ArtifactItem implements DependableCoordinate { private FileMapper[] fileMappers; /** - * Default ctor. + * Default constructor. */ public ArtifactItem() { // default constructor @@ -140,7 +140,7 @@ private String filterEmptyString(String in) { } /** - * @return Returns the artifactId. + * @return returns the artifact ID */ @Override public String getArtifactId() { @@ -148,14 +148,14 @@ public String getArtifactId() { } /** - * @param theArtifact The artifactId to set. + * @param theArtifact the artifact ID to set */ public void setArtifactId(String theArtifact) { this.artifactId = filterEmptyString(theArtifact); } /** - * @return Returns the groupId. + * @return returns the group ID */ @Override public String getGroupId() { @@ -163,14 +163,14 @@ public String getGroupId() { } /** - * @param groupId The groupId to set. + * @param groupId the group ID to set */ public void setGroupId(String groupId) { this.groupId = filterEmptyString(groupId); } /** - * @return Returns the type. + * @return returns the type */ @Override public String getType() { @@ -178,14 +178,14 @@ public String getType() { } /** - * @param type The type to set. + * @param type the type to set */ public void setType(String type) { this.type = filterEmptyString(type); } /** - * @return Returns the version. + * @return returns the version */ @Override public String getVersion() { @@ -193,21 +193,21 @@ public String getVersion() { } /** - * @param version The version to set. + * @param version the version to set */ public void setVersion(String version) { this.version = filterEmptyString(version); } /** - * @return Returns the base version. + * @return teturns the base version */ public String getBaseVersion() { return ArtifactUtils.toSnapshotVersion(version); } /** - * @return Classifier. + * @return Classifier */ @Override public String getClassifier() { @@ -215,7 +215,7 @@ public String getClassifier() { } /** - * @param classifier Classifier. + * @param classifier classifier */ public void setClassifier(String classifier) { this.classifier = filterEmptyString(classifier); @@ -231,63 +231,63 @@ public String toString() { } /** - * @return Returns the location. + * @return returns the location */ public File getOutputDirectory() { return outputDirectory; } /** - * @param outputDirectory The outputDirectory to set. + * @param outputDirectory the outputDirectory to set */ public void setOutputDirectory(File outputDirectory) { this.outputDirectory = outputDirectory; } /** - * @return Returns the location. + * @return returns the location */ public String getDestFileName() { return destFileName; } /** - * @param destFileName The destFileName to set. + * @param destFileName the destination file name to set */ public void setDestFileName(String destFileName) { this.destFileName = filterEmptyString(destFileName); } /** - * @return Returns the needsProcessing. + * @return returns the needsProcessing */ public boolean isNeedsProcessing() { return this.needsProcessing; } /** - * @param needsProcessing The needsProcessing to set. + * @param needsProcessing the needsProcessing to set */ public void setNeedsProcessing(boolean needsProcessing) { this.needsProcessing = needsProcessing; } /** - * @return Returns the overWriteSnapshots. + * @return teturns the overWriteSnapshots */ public String getOverWrite() { return this.overWrite; } /** - * @param overWrite The overWrite to set. + * @param overWrite the overWrite to set */ public void setOverWrite(String overWrite) { this.overWrite = overWrite; } /** - * @return Returns the encoding. + * @return returns the encoding * @since 3.0 */ public String getEncoding() { @@ -295,7 +295,7 @@ public String getEncoding() { } /** - * @param encoding The encoding to set. + * @param encoding the encoding to set * @since 3.0 */ public void setEncoding(String encoding) { @@ -303,42 +303,42 @@ public void setEncoding(String encoding) { } /** - * @return Returns the artifact. + * @return returns the artifact */ public Artifact getArtifact() { return this.artifact; } /** - * @param artifact The artifact to set. + * @param artifact the artifact to set */ public void setArtifact(Artifact artifact) { this.artifact = artifact; } /** - * @return Returns a comma separated list of excluded items + * @return returns a comma separated list of excluded items */ public String getExcludes() { return DependencyUtil.cleanToBeTokenizedString(this.excludes); } /** - * @param excludes A comma separated list of items to exclude i.e. **\/*.xml, **\/*.properties + * @param excludes a comma separated list of items to exclude; for example, **\/*.xml, **\/*.properties */ public void setExcludes(String excludes) { this.excludes = excludes; } /** - * @return Returns a comma separated list of included items + * @return returns a comma separated list of items to include */ public String getIncludes() { return DependencyUtil.cleanToBeTokenizedString(this.includes); } /** - * @param includes A comma separated list of items to include i.e. **\/*.xml, **\/*.properties + * @param includes comma separated list of items to include; for example, **\/*.xml, **\/*.properties */ public void setIncludes(String includes) { this.includes = includes; @@ -346,7 +346,7 @@ public void setIncludes(String includes) { /** * @return {@link FileMapper}s to be used for rewriting each target path, or {@code null} if no rewriting shall - * happen. + * happen * * @since 3.1.2 */ @@ -356,7 +356,7 @@ public FileMapper[] getFileMappers() { /** * @param fileMappers {@link FileMapper}s to be used for rewriting each target path, or {@code null} if no - * rewriting shall happen. + * rewriting shall happen * * @since 3.1.2 */ diff --git a/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractDependencyFilterMojo.java b/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractDependencyFilterMojo.java index 9eaa182eb..3b274c5f5 100644 --- a/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractDependencyFilterMojo.java +++ b/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractDependencyFilterMojo.java @@ -478,7 +478,7 @@ private Set resolve(Set artifact resolverUtil.resolveArtifact(artifact, getProject().getRemoteProjectRepositories()); resolvedArtifacts.add(RepositoryUtils.toArtifact(resolveArtifact)); } catch (ArtifactResolutionException ex) { - // an error occurred during resolution, log it an continue + // an error occurred during resolution, log it and continue getLog().debug("error resolving: " + artifact, ex); if (stopOnFailure) { throw new MojoExecutionException("error resolving: " + artifact, ex); diff --git a/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractFromDependenciesMojo.java b/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractFromDependenciesMojo.java index 30516ee34..5f624e7a5 100644 --- a/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractFromDependenciesMojo.java +++ b/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractFromDependenciesMojo.java @@ -83,7 +83,7 @@ public abstract class AbstractFromDependenciesMojo extends AbstractDependencyFil /** * Place each type of file in a separate subdirectory. (example /outputDirectory/runtime /outputDirectory/provided - * etc) + * etc.) * * @since 2.2 */ @@ -91,7 +91,7 @@ public abstract class AbstractFromDependenciesMojo extends AbstractDependencyFil protected boolean useSubDirectoryPerScope; /** - * Place each type of file in a separate subdirectory. (example /outputDirectory/jars /outputDirectory/wars etc) + * Place each type of file in a separate subdirectory. (example /outputDirectory/jars /outputDirectory/wars etc.) * * @since 2.0-alpha-1 */ diff --git a/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/CopyDependenciesMojo.java b/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/CopyDependenciesMojo.java index 0d6eb7d73..8223d95ce 100644 --- a/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/CopyDependenciesMojo.java +++ b/src/main/java/org/apache/maven/plugins/dependency/fromDependencies/CopyDependenciesMojo.java @@ -222,11 +222,11 @@ private void installBaseSnapshot(Artifact artifact, ProjectBuildingRequest build * Copies the Artifact after building the destination file name if overridden. This method also checks if the * classifier is set and adds it to the destination file name if needed. * - * @param artifact representing the object to be copied. - * @param removeVersion specifies if the version should be removed from the file name when copying. - * @param prependGroupId specifies if the groupId should be prepend to the file while copying. - * @param theUseBaseVersion specifies if the baseVersion of the artifact should be used instead of the version. - * @throws MojoExecutionException with a message if an error occurs. + * @param artifact the object to be copied + * @param removeVersion specifies if the version should be removed from the file name when copying + * @param prependGroupId specifies if the group ID should be prefixed to the file while copying + * @param theUseBaseVersion specifies if the baseVersion of the artifact should be used instead of the version + * @throws MojoExecutionException with a message if an error occurs * @see #copyArtifact(Artifact, boolean, boolean, boolean, boolean) */ protected void copyArtifact( @@ -239,12 +239,12 @@ protected void copyArtifact( * Copies the Artifact after building the destination file name if overridden. This method also checks if the * classifier is set and adds it to the destination file name if needed. * - * @param artifact representing the object to be copied. - * @param removeVersion specifies if the version should be removed from the file name when copying. - * @param prependGroupId specifies if the groupId should be prepend to the file while copying. - * @param useBaseVersion specifies if the baseVersion of the artifact should be used instead of the version. - * @param removeClassifier specifies if the classifier should be removed from the file name when copying. - * @throws MojoExecutionException with a message if an error occurs. + * @param artifact the object to be copied + * @param removeVersion specifies if the version should be removed from the file name when copying + * @param prependGroupId specifies if the groupId should be prefixed to the file while copying + * @param useBaseVersion specifies if the baseVersion of the artifact should be used instead of the version + * @param removeClassifier specifies if the classifier should be removed from the file name when copying + * @throws MojoExecutionException with a message if an error occurs * @see CopyUtil#copyArtifactFile(Artifact, File) * @see DependencyUtil#getFormattedOutputDirectory(boolean, boolean, boolean, boolean, boolean, boolean, File, Artifact) */ diff --git a/src/main/java/org/apache/maven/plugins/dependency/tree/JsonDependencyNodeVisitor.java b/src/main/java/org/apache/maven/plugins/dependency/tree/JsonDependencyNodeVisitor.java index 015ce1b2d..333928212 100644 --- a/src/main/java/org/apache/maven/plugins/dependency/tree/JsonDependencyNodeVisitor.java +++ b/src/main/java/org/apache/maven/plugins/dependency/tree/JsonDependencyNodeVisitor.java @@ -84,7 +84,7 @@ private void writeNode(int indent, DependencyNode node, StringBuilder sb, Set