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 2011/01/03 22:38:22 UTC

svn commit: r1054776 - /maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracReportGenerator.java

Author: dennisl
Date: Mon Jan  3 21:38:22 2011
New Revision: 1054776

URL: http://svn.apache.org/viewvc?rev=1054776&view=rev
Log:
o Refactoring: Sort switch cases alphabetically.

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

Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracReportGenerator.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracReportGenerator.java?rev=1054776&r1=1054775&r2=1054776&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracReportGenerator.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracReportGenerator.java Mon Jan  3 21:38:22 2011
@@ -91,6 +91,10 @@ public class TracReportGenerator
         {
             switch ( columns[columnIndex] )
             {
+                case JiraMojo.COLUMN_ASSIGNEE:
+                    sinkHeader( sink, bundle.getString( "report.issues.label.by" ) );
+                    break;
+
                 case JiraMojo.COLUMN_CHANGED:
                     sinkHeader( sink, bundle.getString( "report.issues.label.changed" ) );
                     break;
@@ -103,16 +107,12 @@ public class TracReportGenerator
                     sinkHeader( sink, bundle.getString( "report.issues.label.created" ) );
                     break;
 
-                case JiraMojo.COLUMN_ID:
-                    sinkHeader( sink, bundle.getString( "report.issues.label.id" ) );
-                    break;
-
                 case JiraMojo.COLUMN_FIX_VERSION:
                     sinkHeader( sink, bundle.getString( "report.issues.label.fixVersion" ) );
                     break;
 
-                case JiraMojo.COLUMN_ASSIGNEE:
-                    sinkHeader( sink, bundle.getString( "report.issues.label.by" ) );
+                case JiraMojo.COLUMN_ID:
+                    sinkHeader( sink, bundle.getString( "report.issues.label.id" ) );
                     break;
 
                 case JiraMojo.COLUMN_PRIORITY:
@@ -167,6 +167,10 @@ public class TracReportGenerator
             {
                 switch ( columns[columnIndex] )
                 {
+                    case JiraMojo.COLUMN_ASSIGNEE:
+                        sinkCell( sink, issue.getAssignee() );
+                        break;
+
                     case JiraMojo.COLUMN_CHANGED:
                         sinkCell( sink, sdf.format( issue.getUpdated() ) );
                         break;
@@ -179,6 +183,10 @@ public class TracReportGenerator
                         sinkCell( sink, sdf.format( issue.getCreated() ) );
                         break;
 
+                    case JiraMojo.COLUMN_FIX_VERSION:
+                        sinkCell( sink, IssuesReportGenerator.printValues( issue.getFixVersions() ) );
+                        break;
+
                     case JiraMojo.COLUMN_ID:
                         sink.tableCell();
                         sink.link( issue.getLink() );
@@ -187,14 +195,6 @@ public class TracReportGenerator
                         sink.tableCell_();
                         break;
 
-                    case JiraMojo.COLUMN_FIX_VERSION:
-                        sinkCell( sink, IssuesReportGenerator.printValues( issue.getFixVersions() ) );
-                        break;
-
-                    case JiraMojo.COLUMN_ASSIGNEE:
-                        sinkCell( sink, issue.getAssignee() );
-                        break;
-
                     case JiraMojo.COLUMN_PRIORITY:
                         sinkCell( sink, issue.getPriority() );
                         break;