svn commit: r1380895 - /maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/ParameterQueryBuilder.java

[dennisl]

Author: dennisl
Date: Tue Sep  4 21:25:44 2012
New Revision: 1380895

URL: http://svn.apache.org/viewvc?rev=1380895&view=rev
Log:
[MCHANGES-252] [JIRA report] some configuration of the changes plugin are not 
passed to JiraDownloader 

- Move error logging to the correct place

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

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/ParameterQueryBuilder.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/ParameterQueryBuilder.java?rev=1380895&r1=1380894&r2=1380895&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/ParameterQueryBuilder.java
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/ParameterQueryBuilder.java
 Tue Sep  4 21:25:44 2012
@@ -292,12 +292,12 @@ public class ParameterQueryBuilder
                             + "' for sortColumnNames is not correct." );
                 }
             }
-        }
-        if ( validSortColumnNames == 0 )
-        {
-            // Error in the configuration
-            getLog().error(
-                "maven-changes-plugin: None of the configured sortColumnNames 
'" + sortColumnNames + "' are correct." );
+            if ( validSortColumnNames == 0 )
+            {
+                // Error in the configuration
+                getLog().error(
+                    "maven-changes-plugin: None of the configured 
sortColumnNames '" + sortColumnNames + "' are correct." );
+            }
         }
         return this;
     }