]> gerrit.simantics Code Review - simantics/platform.git/commitdiff
Fixed Job to return Status.OK_STATUS instead of null 84/2284/1
authorTuukka Lehtonen <tuukka.lehtonen@semantum.fi>
Mon, 8 Oct 2018 18:33:55 +0000 (21:33 +0300)
committerTuukka Lehtonen <tuukka.lehtonen@semantum.fi>
Mon, 8 Oct 2018 18:33:55 +0000 (21:33 +0300)
gitlab #74
gitlab #143

Change-Id: Ib1c59eca854b6daafb21bad3aa90971deb8f5481

bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/modelBrowser/handlers/StandardCopyHandler.java

index 002a31381439d3df4320a22dd79b28d19c012674..1ce5b2d3698ddee5ce7989059dc58de5d6a9fe04 100644 (file)
@@ -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);