Author: dennisl Date: Sun Jul 8 09:36:26 2007 New Revision: 554384 URL: http://svn.apache.org/viewvc?view=rev&rev=554384 Log: o Improve debugging capabilities and error handling.
Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/announcement/AnnouncementMojo.java Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/announcement/AnnouncementMojo.java URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/announcement/AnnouncementMojo.java?view=diff&rev=554384&r1=554383&r2=554384 ============================================================================== --- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/announcement/AnnouncementMojo.java (original) +++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/announcement/AnnouncementMojo.java Sun Jul 8 09:36:26 2007 @@ -22,10 +22,12 @@ import java.io.File; import java.io.FileWriter; import java.io.Writer; +import java.util.Iterator; import java.util.List; import org.apache.maven.plugin.AbstractMojo; import org.apache.maven.plugin.MojoExecutionException; +import org.apache.maven.plugin.changes.Action; import org.apache.maven.plugin.changes.ChangesXML; import org.apache.maven.plugin.changes.Release; import org.apache.maven.project.MavenProject; @@ -349,7 +351,7 @@ /** * Get the latest release by matching the supplied releases - * with the version in the pom + * with the version from the pom. * * @param releases list of releases * @throws MojoExecutionException @@ -367,14 +369,25 @@ { pomVersion = pomVersion.substring( 0, pomVersion.length() - SNAPSHOT_SUFFIX.length() ); } + getLog().debug( "Found " + releases.size() + " releases."); for ( int i = 0; i < releases.size(); i++ ) { release = (Release) releases.get( i ); + if ( getLog().isDebugEnabled() ) + { + getLog().debug( "The release: " + release.getVersion() + + " has " + release.getAction().size() + " actions."); + } if ( release.getVersion() != null && release.getVersion().equals( pomVersion ) ) { isFound = true; + if ( getLog().isDebugEnabled() ) + { + getLog().debug( "Found the correct release: " + release.getVersion() ); + logRelease( release ); + } return release; } } @@ -387,6 +400,19 @@ return release; } + private void logRelease( Release release ) + { + Action action; + for ( Iterator iterator = release.getAction().iterator(); iterator.hasNext(); ) + { + action = (Action) iterator.next(); + getLog().debug( "o " + action.getType() ); + getLog().debug( " - " + action.getIssue() ); + getLog().debug( " - " + action.getAction() ); + getLog().debug( " - " + action.getDueTo() ); + } + } + /** * Create the velocity template * @@ -431,6 +457,10 @@ catch ( Exception e ) { + if ( e.getCause() != null ) + { + getLog().warn( e.getCause() ); + } throw new MojoExecutionException( e.toString(), e.getCause() ); } } @@ -473,7 +503,7 @@ } catch ( Exception e ) { - throw new MojoExecutionException( "Failed to download JIRA Announcement", e ); + throw new MojoExecutionException( "Failed to extract JIRA issues from the downloaded file", e ); } }