Author: dennisl
Date: Sun Jul 16 02:05:55 2006
New Revision: 422415

URL: http://svn.apache.org/viewvc?rev=422415&view=rev
Log:
[MCHANGES-45] A mailto: link is always created for a due-to in the changes 
report, even though no due-to-email has been supplied

Modified:
    
maven/sandbox/plugins/maven-changes-plugin/src/main/java/org/apache/maven/changes/ChangesReportGenerator.java
    
maven/sandbox/plugins/maven-changes-plugin/src/site/changes/sample-changes.xml

Modified: 
maven/sandbox/plugins/maven-changes-plugin/src/main/java/org/apache/maven/changes/ChangesReportGenerator.java
URL: 
http://svn.apache.org/viewvc/maven/sandbox/plugins/maven-changes-plugin/src/main/java/org/apache/maven/changes/ChangesReportGenerator.java?rev=422415&r1=422414&r2=422415&view=diff
==============================================================================
--- 
maven/sandbox/plugins/maven-changes-plugin/src/main/java/org/apache/maven/changes/ChangesReportGenerator.java
 (original)
+++ 
maven/sandbox/plugins/maven-changes-plugin/src/main/java/org/apache/maven/changes/ChangesReportGenerator.java
 Sun Jul 16 02:05:55 2006
@@ -139,7 +139,15 @@
             {
                 sink.text( " " + bundle.getString( "report.changes.text.thanx" 
) + " " );
 
-                sinkLink( sink, action.getDueTo(), "mailto:"; + 
action.getDueToEmail() );
+                if ( action.getDueToEmail() != null )
+                {
+                    sinkLink( sink, action.getDueTo(), "mailto:"; + 
action.getDueToEmail() );
+                }
+                else {
+                    sink.text( action.getDueTo() );
+                }
+
+                sink.text( "." );
             }
 
             sink.tableCell_();

Modified: 
maven/sandbox/plugins/maven-changes-plugin/src/site/changes/sample-changes.xml
URL: 
http://svn.apache.org/viewvc/maven/sandbox/plugins/maven-changes-plugin/src/site/changes/sample-changes.xml?rev=422415&r1=422414&r2=422415&view=diff
==============================================================================
--- 
maven/sandbox/plugins/maven-changes-plugin/src/site/changes/sample-changes.xml 
(original)
+++ 
maven/sandbox/plugins/maven-changes-plugin/src/site/changes/sample-changes.xml 
Sun Jul 16 02:05:55 2006
@@ -5,7 +5,7 @@
   </properties>
   <body>
     <release version="1.1" date="2005-03-01" description="Subsequent release">
-      <action dev="jruiz" type="add">
+      <action dev="jruiz" type="add" due-to="Dennis Lundberg">
         Added additional documentation on how to configure the plugin.
       </action>
       <action dev="aramirez" type="fix" issue="MPJIRA-11">


Reply via email to