Target Explorer: Allow status handler callback to be "null" and avoid default status handler overwrite by tests plug-in

1 files changed, 5 insertions, 2 deletions

diff --git a/target_explorer/plugins/org.eclipse.tm.te.ui/src/org/eclipse/tm/te/ui/statushandler/DefaultStatusHandler.java b/target_explorer/plugins/org.eclipse.tm.te.ui/src/org/eclipse/tm/te/ui/statushandler/DefaultStatusHandler.java
index 73ab6dee4..c87c62037 100644
--- a/target_explorer/plugins/org.eclipse.tm.te.ui/src/org/eclipse/tm/te/ui/statushandler/DefaultStatusHandler.java
+++ b/target_explorer/plugins/org.eclipse.tm.te.ui/src/org/eclipse/tm/te/ui/statushandler/DefaultStatusHandler.java
@@ -44,7 +44,6 @@ public class DefaultStatusHandler extends AbstractStatusHandler {
 	@Override

 	public void handleStatus(final IStatus status, final IPropertiesContainer data, final DoneHandleStatus done) {

 		Assert.isNotNull(status);

-		Assert.isNotNull(done);

 

 		// If the platform UI is not longer running or the display does not

 		// exist or is disposed already, don't do anything.

@@ -63,7 +62,11 @@ public class DefaultStatusHandler extends AbstractStatusHandler {
 			display.asyncExec(new Runnable() {

 				@Override

 				public void run() {

-					doHandleStatus(status, data, done);

+					doHandleStatus(status, data, done != null ? done : new DoneHandleStatus() {

+						@Override

+						public void doneHandleStatus(Throwable error, IPropertiesContainer data) {

+						}

+					});

 				}

 			});

 		}