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