svn commit: r1062507 - in /maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira: AbstractJiraDownloader.java JiraMojo.java

[de...@apache.org]

Author: dennisl
Date: Sun Jan 23 19:54:19 2011
New Revision: 1062507

URL: http://svn.apache.org/viewvc?rev=1062507&view=rev
Log:
[MCHANGES-217] wrong sorting order in jira report

o Document how the sort columns work for different versions of JIRA

Modified:
    maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java
    maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java

Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java?rev=1062507&r1=1062506&r2=1062507&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java Sun Jan 23 19:54:19 2011
@@ -221,7 +221,7 @@ public abstract class AbstractJiraDownlo
         if ( sortColumnNames != null )
         {
             String[] sortColumnNamesArray = sortColumnNames.split( "," );
-            // N.B. Add in reverse order (it's the way JIRA likes it!!)
+            // N.B. Add in reverse order (it's the way JIRA 3 likes it!!)
             for ( int i = sortColumnNamesArray.length - 1; i >= 0; i-- )
             {
                 String lowerColumnName = sortColumnNamesArray[i].trim().toLowerCase( Locale.ENGLISH );

Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java?rev=1062507&r1=1062506&r2=1062507&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java Sun Jan 23 19:54:19 2011
@@ -215,6 +215,13 @@ public class JiraMojo
      * <code>Resolution</code>, <code>Status</code>, <code>Summary</code>,
      * <code>Type</code>, <code>Updated</code> and <code>Version</code>.
      * </p>
+     * <p>
+     * <strong>Note:</strong> If you are using JIRA 4 you need to put your
+     * sort column names in the reverse order. The handling of this changed
+     * between JIRA 3 and JIRA 4. The current default value is suitable for
+     * JIRA 3. This may change in the future, so please configure your sort
+     * column names in an order that works for your own JIRA version. 
+     * </p>
      *
      * @parameter default-value="Priority DESC, Created DESC"
      * @since 2.0