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;
         }