Bug 540280 - Reverts "Bug 380555 - do not allow InjectionException fly
through in
execute()"
The fix for Bug 380555 prevents exceptions to be visible on the command
line.
This breaks all reporting for errors in handlers and hence leaves the
customer no information what went wrong.
Reverted to fix this regression.
This reverts commit ec184d7d60500ceac26b9e0c76514d34dc0029ca.
Change-Id: Ibb5a0cd8b21d378a894ce297b006cc09b9c9a167
Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>
diff --git a/bundles/org.eclipse.e4.core.commands/src/org/eclipse/e4/core/commands/internal/HandlerServiceHandler.java b/bundles/org.eclipse.e4.core.commands/src/org/eclipse/e4/core/commands/internal/HandlerServiceHandler.java
index 44fc6d3..5940317 100644
--- a/bundles/org.eclipse.e4.core.commands/src/org/eclipse/e4/core/commands/internal/HandlerServiceHandler.java
+++ b/bundles/org.eclipse.e4.core.commands/src/org/eclipse/e4/core/commands/internal/HandlerServiceHandler.java
@@ -26,7 +26,6 @@
import org.eclipse.e4.core.contexts.ContextInjectionFactory;
import org.eclipse.e4.core.contexts.EclipseContextFactory;
import org.eclipse.e4.core.contexts.IEclipseContext;
-import org.eclipse.e4.core.di.InjectionException;
import org.eclipse.e4.core.di.annotations.CanExecute;
import org.eclipse.e4.core.di.annotations.Execute;
@@ -161,12 +160,6 @@
new NotHandledException(getClass().getName()));
}
return result;
- } catch (InjectionException e) {
- if (e.getCause() instanceof ExecutionException) {
- ExecutionException executionException = (ExecutionException) e.getCause();
- throw executionException;
- }
- throw new ExecutionException("Error invoking " + handler + " in " + staticContext, e); //$NON-NLS-1$ //$NON-NLS-2$
} finally {
if (localStaticContext != null) {
localStaticContext.dispose();
