You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by de...@apache.org on 2007/07/05 22:24:39 UTC

svn commit: r553625 - in /maven/plugins/trunk/maven-changelog-plugin/src/main: java/org/apache/maven/plugin/changelog/ChangeLogHandler.java java/org/apache/maven/plugin/changelog/ChangeLogReport.java resources/scm-activity.properties

Author: dennisl
Date: Thu Jul  5 13:24:38 2007
New Revision: 553625

URL: http://svn.apache.org/viewvc?view=rev&rev=553625
Log:
[MCHANGELOG-63] Tag-based reports have new headers, SCM-comments with newlines get linebreaks in HTML, lay-out update on changelog-report
Submitted by: Roland Asmann

o Applied the first part of the patch for this issue, with modifications: creating more appropriate headings when using type=tags.

Modified:
    maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogHandler.java
    maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java
    maven/plugins/trunk/maven-changelog-plugin/src/main/resources/scm-activity.properties

Modified: maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogHandler.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogHandler.java?view=diff&rev=553625&r1=553624&r2=553625
==============================================================================
--- maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogHandler.java (original)
+++ maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogHandler.java Thu Jul  5 13:24:38 2007
@@ -21,6 +21,7 @@
 
 import org.apache.maven.scm.ChangeFile;
 import org.apache.maven.scm.ChangeSet;
+import org.apache.maven.scm.ScmTag;
 import org.apache.maven.scm.command.changelog.ChangeLogSet;
 import org.xml.sax.Attributes;
 import org.xml.sax.SAXException;
@@ -222,6 +223,8 @@
             }
 
             bufSet = new ChangeLogSet( bufEntries, startDate, endDate );
+            bufSet.setStartVersion( new ScmTag( attributes.getValue( "startTag" ) ) );
+            bufSet.setEndVersion( new ScmTag( attributes.getValue( "endTag" ) ) );
         }
     }
 }

Modified: maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java?view=diff&rev=553625&r1=553624&r2=553625
==============================================================================
--- maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java (original)
+++ maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java Thu Jul  5 13:24:38 2007
@@ -463,7 +463,8 @@
                 }
                 else
                 {
-                    result = provider.changeLog( repository, new ScmFileSet( basedir ), startTag, endTag );
+                    result = provider.changeLog( repository, new ScmFileSet( basedir ), new ScmRevision( startTag ),
+                                                 new ScmRevision( endTag ) );
 
                     checkResult( result );
 
@@ -516,7 +517,7 @@
         }
         catch ( MojoExecutionException e )
         {
-            throw new MavenReportException( "An error is occurred during changelog command : ", e );
+            throw new MavenReportException( "An error has occurred during changelog command : ", e );
         }
     }
 
@@ -867,7 +868,25 @@
 
         SimpleDateFormat headingDateFormater = new SimpleDateFormat( headingDateFormat );
 
-        if ( set.getStartDate() == null )
+        if ( "tag".equals( type ) )
+        {
+            if ( set.getStartVersion() == null || set.getStartVersion().getName() == null )
+            {
+                sink.text( bundle.getString( "report.SetTagCreation" ) );
+            }
+            else if ( set.getEndVersion() == null || set.getEndVersion().getName() == null )
+            {
+                sink.text( bundle.getString( "report.SetTagSince" ) );
+                sink.text( " '" + set.getStartVersion() + "'" );
+            }
+            else
+            {
+                sink.text( bundle.getString( "report.SetTagBetween" ) );
+                sink.text( " '" + set.getStartVersion() + "' " + bundle.getString( "report.And" ) + " '"
+                    + set.getEndVersion() + "'" );
+            }
+        }
+        else  if ( set.getStartDate() == null )
         {
             sink.text( bundle.getString( "report.SetRangeUnknown" ) );
         }

Modified: maven/plugins/trunk/maven-changelog-plugin/src/main/resources/scm-activity.properties
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changelog-plugin/src/main/resources/scm-activity.properties?view=diff&rev=553625&r1=553624&r2=553625
==============================================================================
--- maven/plugins/trunk/maven-changelog-plugin/src/main/resources/scm-activity.properties (original)
+++ maven/plugins/trunk/maven-changelog-plugin/src/main/resources/scm-activity.properties Thu Jul  5 13:24:38 2007
@@ -20,6 +20,9 @@
 report.SetRangeUnknown=Changes from an unknown range
 report.SetRangeSince=Changes since
 report.SetRangeBetween=Changes between
+report.SetTagCreation=Changes since project creation
+report.SetTagSince=Changes since tag
+report.SetTagBetween=Changes between tags
 report.To=to
 report.TotalCommits=Total commits