GERONIMODEVTOOLS-517 Can't delete project from defined geronimo server

Ted Kirby · Ted Kirby · commit be8ddbf0a4d0 · 2008-09-19T18:25:33.000Z
Thanks for the patch, Delos! git-svn-id: https://svn.apache.org/repos/asf/geronimo/devtools/eclipse-plugin/branches/2.0.1@697175 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/plugins/org.apache.geronimo.st.core/src/org/apache/geronimo/st/core/GeronimoServerDelegate.java b/plugins/org.apache.geronimo.st.core/src/org/apache/geronimo/st/core/GeronimoServerDelegate.java
@@ -86,13 +86,15 @@ abstract public class GeronimoServerDelegate extends ServerDelegate implements I
 	 *      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;
