Updated AddMarkerHandler to an Eclipse 4 Handler

SimonScholz · SimonScholz · commit c8068d2c4bce · 2016-07-01T21:04:29.000+02:00
Change-Id: I481fe26d25f916784971197aa8e11b7afee923b2
Signed-off-by: Simon Scholz &lt;simon.scholz@vogella.com&gt;
diff --git a/de.vogella.plugin.markers/src/de/vogella/plugin/markers/handler/AddMarkerHandler.java b/de.vogella.plugin.markers/src/de/vogella/plugin/markers/handler/AddMarkerHandler.java
@@ -1,40 +1,41 @@
 package de.vogella.plugin.markers.handler;
 
-import org.eclipse.core.commands.AbstractHandler;
-import org.eclipse.core.commands.ExecutionEvent;
-import org.eclipse.core.commands.ExecutionException;
+import javax.inject.Named;
+
 import org.eclipse.core.resources.IMarker;
 import org.eclipse.core.resources.IResource;
-import org.eclipse.jdt.core.IJavaProject;
+import org.eclipse.e4.core.di.annotations.Execute;
+import org.eclipse.e4.core.services.adapter.Adapter;
+import org.eclipse.e4.ui.services.IServiceConstants;
 import org.eclipse.jface.viewers.IStructuredSelection;
-import org.eclipse.ui.handlers.HandlerUtil;
 
-public class AddMarkerHandler extends AbstractHandler {
 
-	@Override
-	public Object execute(ExecutionEvent event) throws ExecutionException {
-		IStructuredSelection selection = (IStructuredSelection) HandlerUtil
-				.getActiveSite(event).getSelectionProvider().getSelection();
-		if (selection == null) {
-			return null;
+public class AddMarkerHandler {
+
+	@Execute
+	public void execute(@Named(IServiceConstants.ACTIVE_SELECTION) IStructuredSelection selection, Adapter adapter) {
+
+		if (selection == null || selection.isEmpty()) {
+			return;
 		}
+
 		Object firstElement = selection.getFirstElement();
-		if (firstElement instanceof IJavaProject) {
-			IJavaProject type = (IJavaProject) firstElement;
-			writeMarkers(type);
+		IResource resource = adapter.adapt(firstElement, IResource.class);
 
+		if (resource != null) {
+			writeMarkers(resource);
 		}
-		return null;
+
 	}
 
-	private void writeMarkers(IJavaProject type) {
+	private void writeMarkers(IResource resource) {
 		try {
-			IResource resource = type.getUnderlyingResource();
 			IMarker marker = resource.createMarker(IMarker.TASK);
-			marker.setAttribute(IMarker.MESSAGE, "This a a task");
+			marker.setAttribute(IMarker.MESSAGE, "This is a task");
 			marker.setAttribute(IMarker.PRIORITY, IMarker.PRIORITY_HIGH);
 		} catch (Exception e) {
 			e.printStackTrace();
 		}
 	}
+
 }
