From 5b5d50e090226b3196e42c12881788a3d2d610c0 Mon Sep 17 00:00:00 2001
From: Gaetan Boismal <gaetan.boismal@forgerock.com>
Date: Thu, 30 Apr 2015 09:27:20 +0000
Subject: [PATCH] svn property check maven plugin improvement

---
 opendj-svn-property-check-maven-plugin/src/main/java/org/forgerock/maven/CheckSVNPropertyMojo.java |   14 +++++++++++---
 1 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/opendj-svn-property-check-maven-plugin/src/main/java/org/forgerock/maven/CheckSVNPropertyMojo.java b/opendj-svn-property-check-maven-plugin/src/main/java/org/forgerock/maven/CheckSVNPropertyMojo.java
index 8a3eff4..00509f6 100644
--- a/opendj-svn-property-check-maven-plugin/src/main/java/org/forgerock/maven/CheckSVNPropertyMojo.java
+++ b/opendj-svn-property-check-maven-plugin/src/main/java/org/forgerock/maven/CheckSVNPropertyMojo.java
@@ -94,9 +94,9 @@
         }
 
         if (!errorFilePaths.isEmpty()) {
-            getLog().warn(" Potential " + svnPropertyName + " updates needed " + "for the following files:");
-            for (String filename : errorFilePaths) {
-                getLog().warn("     " + filename);
+            logWithAppropriateLevel(" Potential " + svnPropertyName + " updates needed " + "for the following files:");
+            for (String fileName : errorFilePaths) {
+                logWithAppropriateLevel("     " + fileName);
             }
 
             if (!ignoreErrors) {
@@ -106,6 +106,14 @@
         }
     }
 
+    private void logWithAppropriateLevel(final String message) {
+        if (!ignoreErrors) {
+            getLog().error(message);
+        } else {
+            getLog().warn(message);
+        }
+    }
+
     /**
      * Examines the provided status item to determine whether the associated
      * file is acceptable.

--
Gitblit v1.10.0