[FIXED JENKINS-28585] Skips aggregation processing if no publisher su…

…pports aggregation.
jenkinsci · Jun 3, 2015 · ab0a504 · ab0a504
1 parent 0dc77cf
commit ab0a504
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/src/main/java/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher.java b/src/main/java/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher.java
@@ -207,26 +207,26 @@ public boolean perform(final AbstractBuild<?, ?> build, final Launcher launcher,
     }
 
     public ConditionalMatrixAggregator createAggregator(MatrixBuild build, Launcher launcher, BuildListener listener) {
-        if (isConfiguredAggregation()) {
-            // alerts if all publishers doesn't support aggregation
-            // even if configured for aggregation
-            boolean supportAggregation = false;
-
-            for (BuildStep publisher: getPublisherList()) {
-                if (publisher instanceof MatrixAggregatable) {
-                    supportAggregation = true;
-                    break;
-                }
+        boolean supportAggregation = false;
+
+        for (BuildStep publisher: getPublisherList()) {
+            if (publisher instanceof MatrixAggregatable) {
+                supportAggregation = true;
+                break;
             }
-
-            if (!supportAggregation) {
+        }
+
+        if (!supportAggregation) {
+            if (isConfiguredAggregation()) {
+                // alerts if all publishers doesn't support aggregation
+                // even if configured for aggregation
                 listener.getLogger().println(String.format(
                         "[%s] WARNING: Condition for Matrix Aggregation is configured for %s which does not support aggregation",
                         getDescriptor().getDisplayName(),
                         FlexiblePublisher.getBuildStepShortName(getPublisherList())
                 ));
-                return null;
             }
+            return null;
         }
         // First, decide whether the condition is satisfied
         // in the parent scope.