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/02 18:02:13 UTC

svn commit: r1054431 - /maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesMojo.java

Author: dennisl
Date: Sun Jan  2 17:02:12 2011
New Revision: 1054431

URL: http://svn.apache.org/viewvc?rev=1054431&view=rev
Log:
o Refactoring: Introduce a method.

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

Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesMojo.java?rev=1054431&r1=1054430&r2=1054431&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesMojo.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/changes/ChangesMojo.java Sun Jan  2 17:02:12 2011
@@ -293,22 +293,7 @@ public class ChangesMojo
         addIssueLinkTemplate( ChangesReportGenerator.DEFAULT_ISSUE_SYSTEM_KEY, issueLinkTemplate );
 
         // Show the current issueLinkTemplatePerSystem configuration
-        if ( getLog().isDebugEnabled() )
-        {
-            if ( issueLinkTemplatePerSystem == null )
-            {
-                getLog().debug( "No issueLinkTemplatePerSystem configuration was found" );
-            }
-            else
-            {
-                Iterator iterator = issueLinkTemplatePerSystem.entrySet().iterator();
-                while ( iterator.hasNext() )
-                {
-                    Map.Entry entry = (Map.Entry) iterator.next();
-                    getLog().debug( "issueLinkTemplatePerSystem[" + entry.getKey() + "] = " + entry.getValue() );
-                }
-            }
-        }
+        logIssueLinkTemplatePerSystem( issueLinkTemplatePerSystem );
 
         report.setIssueLinksPerSystem( issueLinkTemplatePerSystem );
         
@@ -349,6 +334,26 @@ public class ChangesMojo
         }
     }
 
+    private void logIssueLinkTemplatePerSystem( Map issueLinkTemplatePerSystem )
+    {
+        if ( getLog().isDebugEnabled() )
+        {
+            if ( issueLinkTemplatePerSystem == null )
+            {
+                getLog().debug( "No issueLinkTemplatePerSystem configuration was found" );
+            }
+            else
+            {
+                Iterator iterator = issueLinkTemplatePerSystem.entrySet().iterator();
+                while ( iterator.hasNext() )
+                {
+                    Map.Entry entry = (Map.Entry) iterator.next();
+                    getLog().debug( "issueLinkTemplatePerSystem[" + entry.getKey() + "] = " + entry.getValue() );
+                }
+            }
+        }
+    }
+
     public String getName( Locale locale )
     {
         return getBundle( locale ).getString( "report.changes.name" );