commit be8ddbf0a4d0e8f1ed3ad7d39409a767ea5bd870
author Ted Kirby <tkirby@apache.org> Fri Sep 19 18:25:33 2008 +0000
committer Ted Kirby <tkirby@apache.org> Fri Sep 19 18:25:33 2008 +0000
tree bc0b7aa6696619b093e402cb89daa9f40e2cfe79
parent a5ab2949efe33b00cb7d8a168b05e921888715af

GERONIMODEVTOOLS-517 Can't delete project from defined geronimo server
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

plugins/org.apache.geronimo.st.core/src/org/apache/geronimo/st/core/GeronimoServerDelegate.java

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
index 40cb1f6..10d68a8 100644
--- 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 @@
 	 *      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;