On Fri, Jan 8, 2010 at 1:28 PM, Brett Porter <[email protected]> wrote:
> > On 08/01/2010, at 2:42 PM, [email protected] wrote: > > > - buildResult = getContinuum().getBuildResult( getBuildId() ); > > + try > > + { > > + buildResult = getContinuum().getBuildResult( getBuildId() ); > > + } > > + catch ( ContinuumException e ) > > + { > > + buildResult = null; > > + } > > Is there a need to log the exception or does it provide no value? > > No. The reason why I tried to catch the exception is so that I'll know if the user is actually trying to view the current build result. I think i'll change this. > > + > > + ConfigurationService configuration = > getContinuum().getConfiguration(); > > + > > + if ( configuration.isDistributedBuildEnabled() && > > + project.getState() == > ContinuumProjectState.BUILDING ) > > + { > > + model.setLatestBuildId( 0 ); > > + } > > Why is this only 0'd on the distributed build scenario? > > In a distributed build scenario, the latestBuild will point to a previous build result because the current build result will be created after the project finishes its build. > - Brett > > -- > Brett Porter > [email protected] > http://brettporter.wordpress.com/ > > > > >
