avoiding problems when a package is not found

Instead of null pointer exception a warning is logged.
Also a trim is added to the tools name; because...well you know...

Author: jantje

io.sloeber.core/src/io/sloeber/core/managers/Package.java

@@ -126,7 +126,7 @@ public List<Tool> getTools() {
 
 	public Tool getTool(String toolName, String version) {
 		for (Tool tool : this.tools) {
-			if (tool.getName().equals(toolName) && tool.getVersion().equals(version)) {
+			if (tool.getName().trim().equals(toolName) && tool.getVersion().equals(version)) {
 				return tool;
 			}
 		}

io.sloeber.ui/src/io/sloeber/ui/actions/LinkCommandHandler.java

@@ -5,21 +5,28 @@
 import org.eclipse.core.commands.ExecutionException;
 import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.Status;
+import org.osgi.framework.FrameworkUtil;
 
 import io.sloeber.ui.Activator;
 import io.sloeber.ui.Messages;
 
 public class LinkCommandHandler extends AbstractHandler {
 
-    @Override
-    public Object execute(ExecutionEvent event) throws ExecutionException {
-	try {
-	    String url = event.getParameter("io.sloeber.core.link.parameter");//$NON-NLS-1$
-	    org.eclipse.swt.program.Program.launch(url);
-	} catch (IllegalArgumentException e) {
-	    Activator.log(new Status(IStatus.ERROR, Activator.getId(), Messages.json_browser_fail, e));
+	@Override
+	public Object execute(ExecutionEvent event) throws ExecutionException {
+		try {
+			org.osgi.framework.Version version = FrameworkUtil.getBundle(getClass()).getVersion();
+			String url = event.getParameter("io.sloeber.core.link.parameter");//$NON-NLS-1$
+
+			url = url.replace("${MAJOR_VERSION}", String.valueOf(version.getMajor())); //$NON-NLS-1$
+			url = url.replace("${MINOR_VERSION}", String.valueOf(version.getMinor())); //$NON-NLS-1$
+			url = url.replace("${MICRO_VERSION}", String.valueOf(version.getMinor())); //$NON-NLS-1$
+			url = url.replace("${QUALIFIER_VERSION}", String.valueOf(version.getMinor())); //$NON-NLS-1$
+			org.eclipse.swt.program.Program.launch(url);
+		} catch (IllegalArgumentException e) {
+			Activator.log(new Status(IStatus.ERROR, Activator.getId(), Messages.json_browser_fail, e));
+		}
+		return null;
 	}
-	return null;
-    }
 
 }