You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@forrest.apache.org by br...@apache.org on 2004/04/07 22:06:00 UTC
svn commit: rev 9915 - xml/forrest/trunk/scratchpad/forrestbot2/webapp/src/java/org/apache/forrest/forrestbot/webapp/action
Author: brondsem
Date: Wed Apr 7 13:05:59 2004
New Revision: 9915
Modified:
xml/forrest/trunk/scratchpad/forrestbot2/webapp/src/java/org/apache/forrest/forrestbot/webapp/action/ExecuteAction.java
Log:
fix the check for each button
Modified: xml/forrest/trunk/scratchpad/forrestbot2/webapp/src/java/org/apache/forrest/forrestbot/webapp/action/ExecuteAction.java
==============================================================================
--- xml/forrest/trunk/scratchpad/forrestbot2/webapp/src/java/org/apache/forrest/forrestbot/webapp/action/ExecuteAction.java (original)
+++ xml/forrest/trunk/scratchpad/forrestbot2/webapp/src/java/org/apache/forrest/forrestbot/webapp/action/ExecuteAction.java Wed Apr 7 13:05:59 2004
@@ -64,7 +64,7 @@
return mapping.findForward(Constants.FORWARD_NAME_SUCCESS);
}
- if (build != null) {
+ if (build != null && !build.equals("")) {
if (p.asDTO().isBuildable()) {
try {
Executor.build(project);
@@ -75,7 +75,7 @@
} else {
errors.add("execute", new ActionError("error.authorization"));
}
- } else if (deploy != null) {
+ } else if (deploy != null && !deploy.equals("")) {
if (p.asDTO().isDeployable()) {
try {
Executor.deploy(project);