NEW - bug 193876: query synchronization is missing hits

https://bugs.eclipse.org/bugs/show_bug.cgi?id=193876

2 files changed, 12 insertions, 3 deletions

diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/ITaskCollector.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/ITaskCollector.java
index 2aad2db65..a950c22b7 100644
--- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/ITaskCollector.java
+++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/ITaskCollector.java
@@ -11,12 +11,18 @@
 
 package org.eclipse.mylyn.tasks.core;
 
+import java.util.Set;
+
 import org.eclipse.core.runtime.CoreException;
 
+/**
+ * @author Rob Elves
+ */
 public interface ITaskCollector {
 
 	void accept(AbstractTask task);
 
 	void accept(RepositoryTaskData taskData) throws CoreException;
 
+	public Set<AbstractTask> getTaskHits();
 }
diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/QueryHitCollector.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/QueryHitCollector.java
index d1134a538..b22fba5e1 100644
--- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/QueryHitCollector.java
+++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/QueryHitCollector.java
@@ -41,8 +41,9 @@ public class QueryHitCollector implements ITaskCollector {
 		if (task == null) {
 			throw new IllegalArgumentException();
 		}
-		
-		taskResults.add(task);
+		if (taskResults.size() < MAX_HITS) {
+			taskResults.add(task);
+		}
 	}
 
 	public void accept(RepositoryTaskData taskData) throws CoreException {
@@ -51,7 +52,9 @@ public class QueryHitCollector implements ITaskCollector {
 		}
 
 		AbstractTask task = taskFactory.createTask(taskData, new NullProgressMonitor());
-		taskResults.add(task);
+		if (taskResults.size() < MAX_HITS) {
+			taskResults.add(task);
+		}
 	}
 
 	public Set<AbstractTask> getTaskHits() {