diff options
Diffstat (limited to 'org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/RemoveDependencyOperation.java')
-rw-r--r-- | org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/editing/RemoveDependencyOperation.java | 14 |
1 files changed, 6 insertions, 8 deletions
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 4831caf6..93fb88a3 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 @@ -26,14 +26,12 @@ public class RemoveDependencyOperation implements Operation { */ public void process(Document document) { Element dependencyElement = PomHelper.findDependency(document, dependency); - if(dependencyElement == null) { - //TODO we shall not throw exceptions from operations.. - throw new IllegalArgumentException("Dependency does not exist in pom"); - } - Element dependencies = findChild(document.getDocumentElement(), DEPENDENCIES); - removeChild(dependencies, dependencyElement); - // Remove dependencies element if it is empty + if(dependencyElement != null) { + Element dependencies = findChild(document.getDocumentElement(), DEPENDENCIES); + removeChild(dependencies, dependencyElement); + // Remove dependencies element if it is empty - removeIfNoChildElement(dependencies); + removeIfNoChildElement(dependencies); + } } } |