continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From eveni...@apache.org
Subject svn commit: r354160 - in /maven/continuum/trunk/continuum-notifiers: continuum-notifier-irc/src/main/java/org/apache/maven/continuum/notification/irc/ continuum-notifier-msn/src/main/java/org/apache/maven/continuum/notification/msn/
Date Mon, 05 Dec 2005 20:31:04 GMT
Author: evenisse
Date: Mon Dec  5 12:30:56 2005
New Revision: 354160

URL: http://svn.apache.org/viewcvs?rev=354160&view=rev
Log: (empty)

Modified:
    maven/continuum/trunk/continuum-notifiers/continuum-notifier-irc/src/main/java/org/apache/maven/continuum/notification/irc/IrcContinuumNotifier.java
    maven/continuum/trunk/continuum-notifiers/continuum-notifier-msn/src/main/java/org/apache/maven/continuum/notification/msn/MsnContinuumNotifier.java

Modified: maven/continuum/trunk/continuum-notifiers/continuum-notifier-irc/src/main/java/org/apache/maven/continuum/notification/irc/IrcContinuumNotifier.java
URL: http://svn.apache.org/viewcvs/maven/continuum/trunk/continuum-notifiers/continuum-notifier-irc/src/main/java/org/apache/maven/continuum/notification/irc/IrcContinuumNotifier.java?rev=354160&r1=354159&r2=354160&view=diff
==============================================================================
--- maven/continuum/trunk/continuum-notifiers/continuum-notifier-irc/src/main/java/org/apache/maven/continuum/notification/irc/IrcContinuumNotifier.java
(original)
+++ maven/continuum/trunk/continuum-notifiers/continuum-notifier-irc/src/main/java/org/apache/maven/continuum/notification/irc/IrcContinuumNotifier.java
Mon Dec  5 12:30:56 2005
@@ -221,13 +221,16 @@
     {
         try
         {
-            project = store.getProjectWithBuilds( project.getId() );
+            // TODO: prefer to remove this and get them up front
+            if ( project.getId() > 0 )
+            {
+                project = store.getProjectWithBuilds( project.getId() );
+            }
         }
         catch ( ContinuumStoreException e )
         {
             throw new ContinuumException( "Unable to obtain project builds", e );
         }
-
         List builds = project.getBuildResults();
 
         if ( builds.size() < 2 )

Modified: maven/continuum/trunk/continuum-notifiers/continuum-notifier-msn/src/main/java/org/apache/maven/continuum/notification/msn/MsnContinuumNotifier.java
URL: http://svn.apache.org/viewcvs/maven/continuum/trunk/continuum-notifiers/continuum-notifier-msn/src/main/java/org/apache/maven/continuum/notification/msn/MsnContinuumNotifier.java?rev=354160&r1=354159&r2=354160&view=diff
==============================================================================
--- maven/continuum/trunk/continuum-notifiers/continuum-notifier-msn/src/main/java/org/apache/maven/continuum/notification/msn/MsnContinuumNotifier.java
(original)
+++ maven/continuum/trunk/continuum-notifiers/continuum-notifier-msn/src/main/java/org/apache/maven/continuum/notification/msn/MsnContinuumNotifier.java
Mon Dec  5 12:30:56 2005
@@ -107,7 +107,7 @@
 
         if ( source.equals( ContinuumNotificationDispatcher.MESSAGE_ID_BUILD_COMPLETE ) )
         {
-            sendMessage( project, build, recipients, configuration );
+            buildComplete( project, build, recipients, configuration );
         }
     }
 
@@ -149,7 +149,7 @@
         return message + " " + getReportUrl( project, build, configurationService );
     }
 
-    private void sendMessage( Project project, BuildResult build, Set recipients, Map configuration
)
+    private void buildComplete( Project project, BuildResult build, Set recipients, Map configuration
)
         throws NotificationException
     {
         String message;



Mime
View raw message