Add support to search bundles in the host by version range

Currently PDE#findPluginInHost assumes that a bundle is unique by its id
what is not true in all cases.

This now do the following:

- index the plugins by id but keep them in a list to support multiple
- if more than one version is found return the highest
- allow to query all models for an id

Author: laeubi

ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/PDECore.java

@@ -15,9 +15,12 @@
 
 import java.lang.reflect.InvocationTargetException;
 import java.net.URI;
+import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Comparator;
 import java.util.HashMap;
 import java.util.Hashtable;
+import java.util.List;
 import java.util.Map;
 import java.util.stream.Stream;
 
@@ -37,6 +40,7 @@
 import org.eclipse.osgi.service.debug.DebugOptions;
 import org.eclipse.osgi.service.debug.DebugOptionsListener;
 import org.eclipse.osgi.service.debug.DebugTrace;
+import org.eclipse.osgi.service.resolver.BundleDescription;
 import org.eclipse.pde.core.IBundleClasspathResolver;
 import org.eclipse.pde.core.IClasspathContributor;
 import org.eclipse.pde.core.plugin.IPluginModelBase;
@@ -56,6 +60,7 @@
 import org.osgi.framework.FrameworkUtil;
 import org.osgi.framework.ServiceReference;
 import org.osgi.framework.ServiceRegistration;
+import org.osgi.framework.Version;
 import org.osgi.util.tracker.ServiceTracker;
 
 import aQute.bnd.build.Workspace;
@@ -109,7 +114,7 @@ public class PDECore extends Plugin implements DebugOptionsListener {
 	 */
 	private static PDEPreferencesManager fPreferenceManager;
 
-	private Map<String, IPluginModelBase> fHostPlugins;
+	private Map<String, List<IPluginModelBase>> fHostPlugins;
 
 	public static PDECore getDefault() {
 		return inst;
@@ -221,15 +226,20 @@ public PDECore() {
 		inst = this;
 	}
 
-	public synchronized IPluginModelBase findPluginInHost(String id) {
+	public IPluginModelBase findPluginInHost(String id) {
+		return findPluginsInHost(id).max(Comparator.comparing(plugin -> {
+			return getOSGiVersion(plugin);
+		})).orElse(null);
+	}
+
+	public synchronized Stream<IPluginModelBase> findPluginsInHost(String id) {
 		if (fHostPlugins == null) {
 			fHostPlugins = new HashMap<>();
-
 			ITargetDefinition defaultTarget = TargetPlatformService.getDefault().newDefaultTarget();
 			IStatus status = defaultTarget.resolve(new NullProgressMonitor());
 			if (!status.isOK()) {
 				log(status);
-				return null;
+				return Stream.empty();
 			}
 
 			URI[] pluginPaths = Arrays.stream(defaultTarget.getBundles()) //
@@ -240,11 +250,22 @@ public synchronized IPluginModelBase findPluginInHost(String id) {
 			state.resolveState(false);
 
 			for (IPluginModelBase plugin : state.getTargetModels()) {
-				fHostPlugins.put(plugin.getPluginBase().getId(), plugin);
+				fHostPlugins.computeIfAbsent(plugin.getPluginBase().getId(), nil -> new ArrayList<>(1)).add(plugin);
 			}
 		}
+		List<IPluginModelBase> list = fHostPlugins.get(id);
+		if (list == null) {
+			return Stream.empty();
+		}
+		return list.stream();
+	}
 
-		return fHostPlugins.get(id);
+	public static Version getOSGiVersion(IPluginModelBase plugin) {
+		BundleDescription description = plugin.getBundleDescription();
+		if (description == null) {
+			return Version.emptyVersion;
+		}
+		return description.getVersion();
 	}
 
 	public PluginModelManager getModelManager() {