You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2011/06/12 00:14:13 UTC
svn commit: r1134792 -
/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java
Author: hboutemy
Date: Sat Jun 11 22:14:12 2011
New Revision: 1134792
URL: http://svn.apache.org/viewvc?rev=1134792&view=rev
Log:
fixed little formatting issue
Modified:
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java
Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java?rev=1134792&r1=1134791&r2=1134792&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java Sat Jun 11 22:14:12 2011
@@ -146,7 +146,7 @@ public abstract class AbstractJiraDownlo
if ( statusIds != null )
{
String[] stats = statusIds.split( "," );
- for ( String stat : stats )
+ for ( String stat : stats )
{
stat = stat.trim();
String statusParam = statusMap.get( stats );
@@ -154,15 +154,19 @@ public abstract class AbstractJiraDownlo
if ( statusParam != null )
{
localFilter.append( "&statusIds=" ).append( statusParam );
- } else {
+ }
+ else
+ {
// if it's numeric we can handle it too.
- try {
+ try
+ {
Integer.parseInt( stat );
localFilter.append( "&statusIds=" ).append( stat );
- } catch (NumberFormatException nfe) {
+ }
+ catch ( NumberFormatException nfe )
+ {
getLog().error( "maven-changes-plugin: invalid statusId " + stat );
}
-
}
}
}
@@ -421,10 +425,12 @@ public abstract class AbstractJiraDownlo
}
catch ( Exception e )
{
- if ( project.getIssueManagement() != null)
+ if ( project.getIssueManagement() != null )
{
getLog().error( "Error accessing " + project.getIssueManagement().getUrl(), e );
- } else {
+ }
+ else
+ {
getLog().error( "Error accessing mock project issues", e );
}
}
@@ -595,7 +601,7 @@ public abstract class AbstractJiraDownlo
getLog().error( "Invalid JIRA URL: " + jiraUrl + ". " + e.getMessage() );
}
String jiraHost = null;
- if( url != null )
+ if ( url != null )
{
jiraHost = url.getHost();
}
@@ -738,7 +744,8 @@ public abstract class AbstractJiraDownlo
}
}
- public List<Issue> getIssueList() throws MojoExecutionException
+ public List<Issue> getIssueList()
+ throws MojoExecutionException
{
if ( output.isFile() )
{