Fix errors reported by Checkstyle.
git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1586409 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/main/java/org/apache/maven/plugin/changes/ChangesMojo.java b/src/main/java/org/apache/maven/plugin/changes/ChangesMojo.java
index 5e2a9d1..1b14d37 100644
--- a/src/main/java/org/apache/maven/plugin/changes/ChangesMojo.java
+++ b/src/main/java/org/apache/maven/plugin/changes/ChangesMojo.java
@@ -257,7 +257,10 @@
additionalProperties.put( "publishDate", simpleDateFormat.format( now ) );
ChangesXML changesXml = getChangesFromFile( xmlPath, project, additionalProperties );
- if ( changesXml == null ) return;
+ if ( changesXml == null )
+ {
+ return;
+ }
if ( aggregated )
{
@@ -278,7 +281,8 @@
final ChangesXML childXml = getChangesFromFile( changesFile, childProject, additionalProperties );
if ( childXml != null )
{
- releaseList = releaseUtils.mergeReleases( releaseList, childProject.getName(), childXml.getReleaseList() );
+ releaseList = releaseUtils.mergeReleases( releaseList, childProject.getName(),
+ childXml.getReleaseList() );
}
}
changesXml.setReleaseList( releaseList );
@@ -405,11 +409,12 @@
// so we get encoding from the file itself
xmlStreamReader = new XmlStreamReader( changesXml );
String encoding = xmlStreamReader.getEncoding();
- File resultFile = new File( filteredOutputDirectory, project.getGroupId() + "." + project.getArtifactId() + "-changes.xml" );
+ File resultFile = new File( filteredOutputDirectory,
+ project.getGroupId() + "." + project.getArtifactId() + "-changes.xml" );
final MavenFileFilterRequest mavenFileFilterRequest =
- new MavenFileFilterRequest( changesXml, resultFile, true, project, Collections.EMPTY_LIST, false,
- encoding, session, additionalProperties );
+ new MavenFileFilterRequest( changesXml, resultFile, true, project, Collections.EMPTY_LIST,
+ false, encoding, session, additionalProperties );
mavenFileFilter.copyFile( mavenFileFilterRequest );
changesXml = resultFile;
}
diff --git a/src/main/java/org/apache/maven/plugin/changes/ReleaseUtils.java b/src/main/java/org/apache/maven/plugin/changes/ReleaseUtils.java
index ebd12de..e9377ba 100644
--- a/src/main/java/org/apache/maven/plugin/changes/ReleaseUtils.java
+++ b/src/main/java/org/apache/maven/plugin/changes/ReleaseUtils.java
@@ -21,7 +21,6 @@
import java.util.ArrayList;
import java.util.Collections;
-import java.util.Iterator;
import java.util.List;
import org.apache.maven.plugin.MojoExecutionException;
diff --git a/src/main/java/org/apache/maven/plugin/issues/Issue.java b/src/main/java/org/apache/maven/plugin/issues/Issue.java
index 8ba42b4..443fa2c 100644
--- a/src/main/java/org/apache/maven/plugin/issues/Issue.java
+++ b/src/main/java/org/apache/maven/plugin/issues/Issue.java
@@ -252,7 +252,8 @@
this.version = version;
}
- public String toString() {
+ public String toString()
+ {
return this.getClass().getSimpleName()
+ "[id='" + this.getId() + "'"
+ ", title='" + this.getTitle() + "'"
diff --git a/src/main/java/org/apache/maven/plugin/issues/IssueManagementSystem.java b/src/main/java/org/apache/maven/plugin/issues/IssueManagementSystem.java
index 33ee4d7..a2232f2 100644
--- a/src/main/java/org/apache/maven/plugin/issues/IssueManagementSystem.java
+++ b/src/main/java/org/apache/maven/plugin/issues/IssueManagementSystem.java
@@ -40,14 +40,14 @@
*
* @return The map from keys used in poms and other config files to issue types.
*/
- public abstract Map<String, IssueType> getIssueTypeMap();
+ abstract Map<String, IssueType> getIssueTypeMap();
/**
* Get the name of the issue management system.
*
* @return The name of the IMS.
*/
- public abstract String getName();
+ abstract String getName();
/**
* Configure this issue management system.
@@ -55,7 +55,7 @@
* @param issueTypes The mapping of issue types used in this issue management system to the ones used in a changes.xml file
* @throws MojoExecutionException If the configuration fails
*/
- public abstract void applyConfiguration( Map<String, String> issueTypes )
+ abstract void applyConfiguration( Map<String, String> issueTypes )
throws MojoExecutionException;
}
\ No newline at end of file
diff --git a/src/main/java/org/apache/maven/plugin/jira/AdaptiveJiraDownloader.java b/src/main/java/org/apache/maven/plugin/jira/AdaptiveJiraDownloader.java
index 102088d..796bdf8 100644
--- a/src/main/java/org/apache/maven/plugin/jira/AdaptiveJiraDownloader.java
+++ b/src/main/java/org/apache/maven/plugin/jira/AdaptiveJiraDownloader.java
@@ -1,3 +1,5 @@
+package org.apache.maven.plugin.jira;
+
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
@@ -17,8 +19,6 @@
* under the License.
*/
-package org.apache.maven.plugin.jira;
-
import org.apache.maven.plugin.MojoExecutionException;
import org.apache.maven.plugin.issues.Issue;
diff --git a/src/main/java/org/apache/maven/plugin/jira/ClassicJiraDownloader.java b/src/main/java/org/apache/maven/plugin/jira/ClassicJiraDownloader.java
index 162290f..6c97f0e 100644
--- a/src/main/java/org/apache/maven/plugin/jira/ClassicJiraDownloader.java
+++ b/src/main/java/org/apache/maven/plugin/jira/ClassicJiraDownloader.java
@@ -34,7 +34,6 @@
import org.apache.commons.httpclient.params.HttpClientParams;
import org.apache.maven.plugin.MojoExecutionException;
import org.apache.maven.plugin.issues.Issue;
-import org.apache.maven.settings.Proxy;
import org.codehaus.plexus.util.IOUtil;
import org.codehaus.plexus.util.StringUtils;
diff --git a/src/main/java/org/apache/maven/plugin/jira/RestJiraDownloader.java b/src/main/java/org/apache/maven/plugin/jira/RestJiraDownloader.java
index 8afff99..6cb322e 100644
--- a/src/main/java/org/apache/maven/plugin/jira/RestJiraDownloader.java
+++ b/src/main/java/org/apache/maven/plugin/jira/RestJiraDownloader.java
@@ -1,3 +1,5 @@
+package org.apache.maven.plugin.jira;
+
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
@@ -17,8 +19,6 @@
* under the License.
*/
-package org.apache.maven.plugin.jira;
-
import com.fasterxml.jackson.core.JsonFactory;
import com.fasterxml.jackson.core.JsonGenerator;
import com.fasterxml.jackson.core.JsonParser;
@@ -204,8 +204,10 @@
private void resolveIds( WebClient client, String jiraProject )
throws IOException, MojoExecutionException, MojoFailureException
{
- resolveList( resolvedComponentIds, client, "components", component, "/rest/api/2/project/{key}/components", jiraProject );
- resolveList( resolvedFixVersionIds, client, "fixVersions", fixVersionIds, "/rest/api/2/project/{key}/versions", jiraProject );
+ resolveList( resolvedComponentIds, client, "components", component, "/rest/api/2/project/{key}/components",
+ jiraProject );
+ resolveList( resolvedFixVersionIds, client, "fixVersions", fixVersionIds, "/rest/api/2/project/{key}/versions",
+ jiraProject );
resolveList( resolvedStatusIds, client, "status", statusIds, "/rest/api/2/status" );
resolveList( resolvedResolutionIds, client, "resolution", resolutionIds, "/rest/api/2/resolution" );
resolveList( resolvedTypeIds, client, "type", typeIds, "/rest/api/2/issuetype" );
@@ -536,7 +538,8 @@
// if not one of the documented failures, assume that there's no rest in there in the first place.
throw new NoRest();
}
- throw new MojoExecutionException( String.format( "Authentication failure status %d.", authRes.getStatus() ) );
+ throw new MojoExecutionException( String.format( "Authentication failure status %d.",
+ authRes.getStatus() ) );
}
}
}
