continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ol...@apache.org
Subject svn commit: r656051 - /continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/CancelBuildAction.java
Date Tue, 13 May 2008 23:15:37 GMT
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;
         }



Mime
View raw message