You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by tk...@apache.org on 2008/09/19 20:25:33 UTC
svn commit: r697175 -
/geronimo/devtools/eclipse-plugin/branches/2.0.1/plugins/org.apache.geronimo.st.core/src/org/apache/geronimo/st/core/GeronimoServerDelegate.java
Author: tkirby
Date: Fri Sep 19 11:25:33 2008
New Revision: 697175
URL: http://svn.apache.org/viewvc?rev=697175&view=rev
Log:
GERONIMODEVTOOLS-517 Can't delete project from defined geronimo server
Thanks for the patch, Delos!
Modified:
geronimo/devtools/eclipse-plugin/branches/2.0.1/plugins/org.apache.geronimo.st.core/src/org/apache/geronimo/st/core/GeronimoServerDelegate.java
Modified: geronimo/devtools/eclipse-plugin/branches/2.0.1/plugins/org.apache.geronimo.st.core/src/org/apache/geronimo/st/core/GeronimoServerDelegate.java
URL: http://svn.apache.org/viewvc/geronimo/devtools/eclipse-plugin/branches/2.0.1/plugins/org.apache.geronimo.st.core/src/org/apache/geronimo/st/core/GeronimoServerDelegate.java?rev=697175&r1=697174&r2=697175&view=diff
==============================================================================
--- geronimo/devtools/eclipse-plugin/branches/2.0.1/plugins/org.apache.geronimo.st.core/src/org/apache/geronimo/st/core/GeronimoServerDelegate.java (original)
+++ geronimo/devtools/eclipse-plugin/branches/2.0.1/plugins/org.apache.geronimo.st.core/src/org/apache/geronimo/st/core/GeronimoServerDelegate.java Fri Sep 19 11:25:33 2008
@@ -86,13 +86,15 @@
* org.eclipse.wst.server.core.IModule[])
*/
public IStatus canModifyModules(IModule[] add, IModule[] remove) {
- for (int i = 0; i < add.length; i++) {
- IModule module = add[i];
- if (module.getProject() != null) {
- IStatus status = FacetUtil.verifyFacets(module.getProject(),
- getServer());
- if (status != null && !status.isOK())
- return status;
+ if (add != null) {
+ for (int i = 0; i < add.length; i++) {
+ IModule module = add[i];
+ if (module.getProject() != null) {
+ IStatus status = FacetUtil.verifyFacets(
+ module.getProject(), getServer());
+ if (status != null && !status.isOK())
+ return status;
+ }
}
}
return Status.OK_STATUS;