Merge PR #197 Fix #62 by excluding empty affected

files collection.
jenkinsci · Mar 2, 2017 · c419e9d · c419e9d
2 parents 71c8ecd + 0c6b4a7
commit c419e9d
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/main/java/jenkins/plugins/slack/ActiveNotifier.java b/src/main/java/jenkins/plugins/slack/ActiveNotifier.java
@@ -17,6 +17,7 @@
 import hudson.tasks.test.TestResult;
 import hudson.triggers.SCMTrigger;
 import hudson.util.LogTaskListener;
+import org.apache.commons.collections.CollectionUtils;
 import org.apache.commons.lang.StringUtils;
 import org.jenkinsci.plugins.displayurlapi.DisplayURLProvider;
 
@@ -138,7 +139,9 @@ String getChanges(AbstractBuild r, boolean includeCustomMessage) {
             Entry entry = (Entry) o;
             logger.info("Entry " + o);
             entries.add(entry);
-            files.addAll(entry.getAffectedFiles());
+            if (CollectionUtils.isNotEmpty(entry.getAffectedFiles())) {
+                files.addAll(entry.getAffectedFiles());
+            }
         }
         if (entries.isEmpty()) {
             logger.info("Empty change...");