You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ol...@apache.org on 2008/05/14 01:15:37 UTC
svn commit: r656051 -
/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/CancelBuildAction.java
Author: olamy
Date: Tue May 13 16:15:37 2008
New Revision: 656051
URL: http://svn.apache.org/viewvc?rev=656051&view=rev
Log:
[CONTINUUM-1725] Using the Cancel Button without an selected project will produce NPE
Modified:
continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/CancelBuildAction.java
Modified: continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/CancelBuildAction.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/CancelBuildAction.java?rev=656051&r1=656050&r2=656051&view=diff
==============================================================================
--- continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/CancelBuildAction.java (original)
+++ continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/CancelBuildAction.java Tue May 13 16:15:37 2008
@@ -64,8 +64,7 @@
public String cancelBuilds()
throws ContinuumException
{
- // first we remove from the build queue
- if ( getSelectedProjects() != null && getSelectedProjects().isEmpty() )
+ if ( getSelectedProjects() == null || getSelectedProjects().isEmpty() )
{
return SUCCESS;
}