Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Green2011-11-30 12:17:57 -0500
committerDavid Green2011-11-30 12:17:57 -0500
commit97872cf82ab65b87e346842bef2bc475c76f8301 (patch)
tree5b5ba7504b0f7d66097b73c3a88cd0db98d6b98f
parent28360dc9d84bbc35c94cfa45f67f113dc02eb2e8 (diff)
downloadorg.eclipse.mylyn.incubator-97872cf82ab65b87e346842bef2bc475c76f8301.tar.gz
org.eclipse.mylyn.incubator-97872cf82ab65b87e346842bef2bc475c76f8301.tar.xz
org.eclipse.mylyn.incubator-97872cf82ab65b87e346842bef2bc475c76f8301.zip
bug 365224: make it easy to create a TaskListIndex that indexes a
subset of tasks in the task list https://bugs.eclipse.org/bugs/show_bug.cgi?id=365224
-rw-r--r--org.eclipse.mylyn.tasks.index.core/src/org/eclipse/mylyn/internal/tasks/index/core/TaskListIndex.java36
1 files changed, 33 insertions, 3 deletions
diff --git a/org.eclipse.mylyn.tasks.index.core/src/org/eclipse/mylyn/internal/tasks/index/core/TaskListIndex.java b/org.eclipse.mylyn.tasks.index.core/src/org/eclipse/mylyn/internal/tasks/index/core/TaskListIndex.java
index 41cafe16..8c746c10 100644
--- a/org.eclipse.mylyn.tasks.index.core/src/org/eclipse/mylyn/internal/tasks/index/core/TaskListIndex.java
+++ b/org.eclipse.mylyn.tasks.index.core/src/org/eclipse/mylyn/internal/tasks/index/core/TaskListIndex.java
@@ -562,6 +562,9 @@ public class TaskListIndex implements ITaskDataManagerListener, ITaskListChangeL
// this can happen when edits are discarded
return;
}
+ if (!taskIsIndexable(task, taskData)) {
+ return;
+ }
synchronized (reindexQueue) {
reindexQueue.put(task, taskData);
}
@@ -746,7 +749,11 @@ public class TaskListIndex implements ITaskDataManagerListener, ITaskListChangeL
taskList.run(new ITaskListRunnable() {
public void execute(IProgressMonitor monitor) throws CoreException {
- allTasks.addAll(taskList.getAllTasks());
+ for (ITask task : taskList.getAllTasks()) {
+ if (taskIsIndexable(task, null)) {
+ allTasks.add(task);
+ }
+ }
}
}, monitor.newChild(1));
@@ -754,6 +761,9 @@ public class TaskListIndex implements ITaskDataManagerListener, ITaskListChangeL
reindexMonitor.beginTask(Messages.TaskListIndex_task_rebuildingIndex, allTasks.size());
for (ITask task : allTasks) {
+ if (!taskIsIndexable(task, null)) {
+ continue;
+ }
try {
TaskData taskData = dataManager.getTaskData(task);
add(writer, task, taskData);
@@ -863,6 +873,9 @@ public class TaskListIndex implements ITaskDataManagerListener, ITaskListChangeL
* @throws IOException
*/
private void add(IndexWriter writer, ITask task, TaskData taskData) throws CorruptIndexException, IOException {
+ if (!taskIsIndexable(task, taskData)) {
+ return;
+ }
Document document = new Document();
@@ -899,8 +912,25 @@ public class TaskListIndex implements ITaskDataManagerListener, ITaskListChangeL
}
/**
- * escapes special characters in the given literal value so that they are not interpreted as special characters in a
- * query
+ * Indicates if the given task is indexable. The default implementation returns true, subclasses may override to
+ * filter some tasks from the task list. This method may be called more than once per task, with some calls omitting
+ * the task data. In this way implementations can avoid loading task data if the decision to filter tasks can be
+ * based on the ITask alone. Implementations that must read the task data in order to determine eligibility for
+ * indexing should return true for tasks where the provided task data is null.
+ *
+ * @param task
+ * the task
+ * @param taskData
+ * the task data, or null if there is no task data
+ * @return true if the given task should be indexed, otherwise false.
+ */
+ protected boolean taskIsIndexable(ITask task, TaskData taskData) {
+ return true;
+ }
+
+ /**
+ * Escapes special characters in the given literal value so that they are not interpreted as special characters in a
+ * query.
*
* @param value
* the value to escape

Back to the top