From: Tuukka Lehtonen <tuukka.lehtonen@semantum.fi>
Date: Mon, 8 Oct 2018 18:33:55 +0000 (+0300)
Subject: Fixed Job to return Status.OK_STATUS instead of null
X-Git-Tag: v1.43.0~136^2~342
X-Git-Url: https://gerrit.simantics.org/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F84%2F2284%2F1;p=simantics%2Fplatform.git

Fixed Job to return Status.OK_STATUS instead of null

gitlab #74
gitlab #143

Change-Id: Ib1c59eca854b6daafb21bad3aa90971deb8f5481
---

diff --git a/bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/modelBrowser/handlers/StandardCopyHandler.java b/bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/modelBrowser/handlers/StandardCopyHandler.java
index 002a31381..1ce5b2d36 100644
--- a/bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/modelBrowser/handlers/StandardCopyHandler.java
+++ b/bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/modelBrowser/handlers/StandardCopyHandler.java
@@ -21,6 +21,7 @@ import org.eclipse.core.commands.ExecutionEvent;
 import org.eclipse.core.commands.ExecutionException;
 import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.core.runtime.IStatus;
+import org.eclipse.core.runtime.Status;
 import org.eclipse.core.runtime.SubMonitor;
 import org.eclipse.core.runtime.jobs.Job;
 import org.eclipse.jface.action.IStatusLineManager;
@@ -97,12 +98,11 @@ public class StandardCopyHandler extends AbstractHandler {
         formatSelectionToClipboardText(event);
         final Resource[] rs = ResourceAdaptionUtils.toResources(selection);
         Job job = new Job("Copy resources") {
-            
             @Override
             protected IStatus run(IProgressMonitor monitor) {
                 monitor.beginTask("Copy resources to clipboard", 1);
                 copyResourcesToClipboard(rs, selection, SubMonitor.convert(monitor, 1));
-                return null;
+                return Status.OK_STATUS;
             }
         };
         job.setUser(true);