Fix JENKINS-8642

Changed defaults for the Failing, Still Failing, Unstable and Still Unstable triggers.
jenkinsci · Feb 2, 2013 · e46a540 · e46a540
1 parent e8d34ec
commit e46a540
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 8 deletions.
diff --git a/src/main/java/hudson/plugins/emailext/ExtendedEmailPublisher.java b/src/main/java/hudson/plugins/emailext/ExtendedEmailPublisher.java
@@ -737,7 +737,7 @@ public static final class DescriptorImpl
     }
 
 	public MatrixAggregator createAggregator(MatrixBuild matrixbuild,
-			Launcher launcher, BuildListener buildlistener) {				
+			Launcher launcher, BuildListener buildlistener) {
 		return new MatrixAggregator(matrixbuild, launcher, buildlistener) {
 			@Override
             public boolean endBuild() throws InterruptedException, IOException {
@@ -753,14 +753,13 @@ public boolean endBuild() throws InterruptedException, IOException {
 
 			@Override		 
 			public boolean startBuild() throws InterruptedException,IOException {
-				LOGGER.log(Level.FINER,"end build of " + this.build.getDisplayName());            					
+				LOGGER.log(Level.FINER,"end build of " + this.build.getDisplayName());
 				// Will be run by parent so we check if needed to be executed by parent 
                 if (getMatrixTriggerMode().forParent) {
                     return ExtendedEmailPublisher.this._perform(this.build, this.listener, true);
                 }
                 return true;
             }
-
         };
-	}
+    }
 }
diff --git a/src/main/java/hudson/plugins/emailext/plugins/trigger/FailureTrigger.java b/src/main/java/hudson/plugins/emailext/plugins/trigger/FailureTrigger.java
@@ -55,6 +55,6 @@ public boolean getDefaultSendToDevs() {
 
     @Override
     public boolean getDefaultSendToList() {
-        return true;
+        return false;
     }
 }
diff --git a/src/main/java/hudson/plugins/emailext/plugins/trigger/StillFailingTrigger.java b/src/main/java/hudson/plugins/emailext/plugins/trigger/StillFailingTrigger.java
@@ -60,6 +60,6 @@ public boolean getDefaultSendToDevs() {
 
     @Override
     public boolean getDefaultSendToList() {
-        return false;
+        return true;
     }
 }
diff --git a/src/main/java/hudson/plugins/emailext/plugins/trigger/StillUnstableTrigger.java b/src/main/java/hudson/plugins/emailext/plugins/trigger/StillUnstableTrigger.java
@@ -59,6 +59,6 @@ public boolean getDefaultSendToDevs() {
 
     @Override
     public boolean getDefaultSendToList() {
-        return false;
+        return true;
     }
 }
diff --git a/src/main/java/hudson/plugins/emailext/plugins/trigger/UnstableTrigger.java b/src/main/java/hudson/plugins/emailext/plugins/trigger/UnstableTrigger.java
@@ -52,6 +52,6 @@ public boolean getDefaultSendToDevs() {
 
     @Override
     public boolean getDefaultSendToList() {
-        return true;
+        return false;
     }
 }