recursive empty element removal

2 files changed, 4 insertions, 4 deletions

diff --git a/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/PomEdits.java b/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/PomEdits.java
index 7b3b6a3e..b6441504 100644
--- a/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/PomEdits.java
+++ b/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/PomEdits.java
@@ -226,7 +226,8 @@ public class PomEdits {
   }
   
   /**
-   * remove the current element if it doesn't contain any sublements, useful for lists etc. 
+   * remove the current element if it doesn't contain any sublements, useful for lists etc,
+   * works recursively removing all parents up that don't have any children elements.
    * @param el
    */
   public static void removeIfNoChildElement(Element el) {
@@ -242,6 +243,7 @@ public class PomEdits {
       Node parent = el.getParentNode();
       if (parent != null && parent instanceof Element) {
         removeChild((Element)parent, el);
+        removeIfNoChildElement((Element)parent);
       }
     }
   }
diff --git a/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/RemoveDependencyOperation.java b/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/RemoveDependencyOperation.java
index 45711d9d..4831caf6 100644
--- a/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/RemoveDependencyOperation.java
+++ b/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/RemoveDependencyOperation.java
@@ -34,8 +34,6 @@ public class RemoveDependencyOperation implements Operation {
     removeChild(dependencies, dependencyElement);
     // Remove dependencies element if it is empty
 
-    if(PomHelper.findDependencies(document.getDocumentElement()).isEmpty()) {
-      removeChild(document.getDocumentElement(), dependencies);
-    }
+    removeIfNoChildElement(dependencies);
   }
 }