Continuum

Catch ContinuumObjectNotFoundException in buildResult.action

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.1-alpha-1
  • Fix Version/s: 1.1-alpha-1
  • Component/s: Web interface
  • Labels:
    None
  • Environment:
    Linux FC4, Maven 2.0.4, JDK 1.5
  • Complexity:
    Intermediate
  • Number of attachments :
    3

Description

ContinuumObjectNotFoundException is thrown by the buildResult.action if a project isn't built yet.

  1. CONTINUUM-818.patch
    15/Aug/06 8:31 AM
    1 kB
    Nap Ramirez
  2. CONTINUUM-818-2.patch
    15/Aug/06 9:34 AM
    0.6 kB
    Nap Ramirez
  3. CONTINUUM-818-3.patch
    16/Aug/06 1:55 AM
    0.8 kB
    Nap Ramirez

Activity

Hide
Nap Ramirez added a comment -

Catches ContinuumObjectNotFoundException in the buildResult.action

Show
Nap Ramirez added a comment - Catches ContinuumObjectNotFoundException in the buildResult.action
Hide
Nap Ramirez added a comment -

As suggested by Emmanuel, the DefaultContinuum shouldn't be throwing ContinuumObjectNotFoundException so the action doesn't have to catch it. Returning an empty List will do.

Show
Nap Ramirez added a comment - As suggested by Emmanuel, the DefaultContinuum shouldn't be throwing ContinuumObjectNotFoundException so the action doesn't have to catch it. Returning an empty List will do.
Hide
Carlos Sanchez added a comment -

You don't want to ignore all ContinuumStoreException, why would you do that?

Is this what you wanted to do?

catch ( ContinuumObjectNotFoundException e )

{ return Collections.EMPTY_LIST; }

catch ( ContinuumStoreException e )

{ throw logAndCreateException( "Exception while getting build results for project.", e ); }
Show
Carlos Sanchez added a comment - You don't want to ignore all ContinuumStoreException, why would you do that? Is this what you wanted to do? catch ( ContinuumObjectNotFoundException e ) { return Collections.EMPTY_LIST; } catch ( ContinuumStoreException e ) { throw logAndCreateException( "Exception while getting build results for project.", e ); }
Hide
Nap Ramirez added a comment -

Thanks Carlos, I missed that part.

Show
Nap Ramirez added a comment - Thanks Carlos, I missed that part.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: