svn commit: r1570990 - /maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java

[michaelo]

Author: michaelo
Date: Sun Feb 23 09:42:44 2014
New Revision: 1570990

URL: http://svn.apache.org/r1570990
Log:
[MCHECKSTYLE-209] Correct grammar, make icons and text consistent in the report

- While producing concrete file line violations, security level was not 
translated

Modified:
    
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java

Modified: 
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java?rev=1570990&r1=1570989&r2=1570990&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java
 (original)
+++ 
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReportGenerator.java
 Sun Feb 23 09:42:44 2014
@@ -195,7 +195,7 @@ public class CheckstyleReportGenerator
         {
             iconWarning();
         }
-        else if ( SeverityLevel.ERROR.getName().equalsIgnoreCase( level ) )
+         if ( SeverityLevel.ERROR.getName().equalsIgnoreCase( level ) )
         {
             iconError();
         }
@@ -463,7 +463,20 @@ public class CheckstyleReportGenerator
         configSeverity = getConfigAttribute( checkerConfig, 
parentConfigurations, "severity", "error" );
         iconSeverity( configSeverity );
         sink.nonBreakingSpace();
-        sink.text( StringUtils.capitalise( configSeverity ) );
+
+        if ( SeverityLevel.INFO.getName().equalsIgnoreCase( configSeverity ) )
+        {
+            sink.text( bundle.getString( "report.checkstyle.info" ) );
+        }
+        else if ( SeverityLevel.WARNING.getName().equalsIgnoreCase( 
configSeverity ) )
+        {
+            sink.text( bundle.getString( "report.checkstyle.warning" ) );
+        }
+        else if ( SeverityLevel.ERROR.getName().equalsIgnoreCase( 
configSeverity ) )
+        {
+            sink.text( bundle.getString( "report.checkstyle.error" ) );
+        }
+
         sink.tableCell_();
 
         sink.tableRow_();