You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ev...@apache.org on 2006/04/05 18:05:21 UTC
svn commit: r391660 - in
/maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum:
DefaultContinuum.java buildcontroller/DefaultBuildController.java
Author: evenisse
Date: Wed Apr 5 09:05:20 2006
New Revision: 391660
URL: http://svn.apache.org/viewcvs?rev=391660&view=rev
Log:
Define the project state at the start of the build so the sumary page have correct icons after a click on 'Build All'
Modified:
maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
Modified: maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
URL: http://svn.apache.org/viewcvs/maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java?rev=391660&r1=391659&r2=391660&view=diff
==============================================================================
--- maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java (original)
+++ maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java Wed Apr 5 09:05:20 2006
@@ -368,8 +368,6 @@
for ( Iterator i = projectsList.iterator(); i.hasNext(); )
{
- long start = System.currentTimeMillis();
-
Project project = (Project) i.next();
Integer buildDefId = (Integer) buildDefinitionsIds.get( new Integer( project.getId() ) );
@@ -381,8 +379,6 @@
}
buildProject( project, buildDefId.intValue(), trigger );
-
- getLogger().debug( "Add " + project.getName() + " in " + ( System.currentTimeMillis() - start ) + "ms" );
}
}
Modified: maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
URL: http://svn.apache.org/viewcvs/maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java?rev=391660&r1=391659&r2=391660&view=diff
==============================================================================
--- maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java (original)
+++ maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java Wed Apr 5 09:05:20 2006
@@ -97,7 +97,15 @@
{
project = store.getProject( projectId );
+ project.setOldState( project.getState() );
+
+ project.setState( ContinuumProjectState.BUILDING );
+
+ store.updateProject( project );
+
buildDefinition = store.getBuildDefinition( buildDefinitionId );
+
+ notifierDispatcher.buildStarted( project );
}
catch ( ContinuumStoreException ex )
{
@@ -130,14 +138,6 @@
try
{
- project.setOldState( project.getState() );
-
- project.setState( ContinuumProjectState.BUILDING );
-
- store.updateProject( project );
-
- notifierDispatcher.buildStarted( project );
-
Map actionContext = new HashMap();
actionContext.put( AbstractContinuumAction.KEY_PROJECT_ID, new Integer( projectId ) );