svn commit: r1763092 - /maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java

[gboue]

Author: gboue
Date: Sun Oct  2 18:42:02 2016
New Revision: 1763092

URL: http://svn.apache.org/viewvc?rev=1763092&view=rev
Log:
[MCHANGES-375] Add stop between issue text and " Fixes issue-number"

When there is a description of the issue and it doesn't end with a period, this 
change adds a period so that the final text is easier to read. This closes #94.

Modified:
    
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java?rev=1763092&r1=1763091&r2=1763092&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesReportGenerator.java
 Sun Oct  2 18:42:02 2016
@@ -249,18 +249,24 @@ public class ChangesReportGenerator
 
         sink.tableCell();
 
+        String actionDescription = action.getAction();
+
         if ( escapeHTML )
         {
-            sink.text( action.getAction() );
+            sink.text( actionDescription );
         }
         else
         {
-            sink.rawText( action.getAction() );
+            sink.rawText( actionDescription );
         }
 
         // no null check needed classes from modello return a new ArrayList
         if ( StringUtils.isNotEmpty( action.getIssue() ) || ( 
!action.getFixedIssues().isEmpty() ) )
         {
+            if ( StringUtils.isNotBlank( actionDescription ) && 
!actionDescription.endsWith( "." ) )
+            {
+                sink.text( "." );
+            }
             sink.text( " " + bundle.getString( "report.changes.text.fixes" ) + 
" " );
 
             // Try to get the issue management system specified in the 
changes.xml file