You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@continuum.apache.org by ct...@apache.org on 2009/03/11 01:32:55 UTC

svn commit: r752320 - /continuum/trunk/continuum-notifiers/continuum-notifier-api/src/main/java/org/apache/maven/continuum/notification/AbstractContinuumNotifier.java

Author: ctan
Date: Wed Mar 11 00:32:55 2009
New Revision: 752320

URL: http://svn.apache.org/viewvc?rev=752320&view=rev
Log:
[CONTINUUM-1823] checks if the scm root's previous state is the same as 
the current state before sending notification

Modified:
    continuum/trunk/continuum-notifiers/continuum-notifier-api/src/main/java/org/apache/maven/continuum/notification/AbstractContinuumNotifier.java

Modified: continuum/trunk/continuum-notifiers/continuum-notifier-api/src/main/java/org/apache/maven/continuum/notification/AbstractContinuumNotifier.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-notifiers/continuum-notifier-api/src/main/java/org/apache/maven/continuum/notification/AbstractContinuumNotifier.java?rev=752320&r1=752319&r2=752320&view=diff
==============================================================================
--- continuum/trunk/continuum-notifiers/continuum-notifier-api/src/main/java/org/apache/maven/continuum/notification/AbstractContinuumNotifier.java (original)
+++ continuum/trunk/continuum-notifiers/continuum-notifier-api/src/main/java/org/apache/maven/continuum/notification/AbstractContinuumNotifier.java Wed Mar 11 00:32:55 2009
@@ -285,7 +285,8 @@
             return true;
         }
 
-        if ( projectScmRoot.getState() == ContinuumProjectState.ERROR && projectNotifier.isSendOnScmFailure() )
+        if ( projectScmRoot.getState() == ContinuumProjectState.ERROR && projectNotifier.isSendOnScmFailure()
+             && projectScmRoot.getOldState() != projectScmRoot.getState() )
         {
             return true;
         }