RESOLVED - bug 159223: reduce amount of refresh required by Task List

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

4 files changed, 15 insertions, 7 deletions

diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/.AbstractRepositoryTask.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/.AbstractRepositoryTask.java
index 1d6ef0afd..cdc2f9311 100644
--- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/.AbstractRepositoryTask.java
+++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/.AbstractRepositoryTask.java
@@ -109,7 +109,7 @@ public abstract class AbstractRepositoryTask extends Task {
 	 * null if no such label exists.
 	 */
 	public String getIdLabel() {
-		return getTaskId(handle);
+		return getTaskId(handleIdentifier);
 	}
 	
 	public static String getTaskId(String taskHandle) {
diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractQueryHit.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractQueryHit.java
index a5cf5f19f..3f7ab8669 100644
--- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractQueryHit.java
+++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractQueryHit.java
@@ -34,11 +34,14 @@ public abstract class AbstractQueryHit implements ITaskListElement {
 
 	private AbstractRepositoryQuery parent;
 
+	private String handleIDentifier;
+	
 	protected AbstractQueryHit(TaskList taskList, String repositoryUrl, String description, String id) {
 		this.taskList = taskList;
 		this.repositoryUrl = repositoryUrl;
 		this.description = description;
 		this.id = id;
+		this.handleIDentifier = AbstractRepositoryTask.getHandle(repositoryUrl, id);
 	}
 
 	public AbstractRepositoryQuery getParent() {
@@ -107,7 +110,12 @@ public abstract class AbstractQueryHit implements ITaskListElement {
 	}
 	
 	public String getHandleIdentifier() {
-		return AbstractRepositoryTask.getHandle(repositoryUrl, id);
+		if (task != null) {
+			return task.getHandleIdentifier();
+		} else {
+			return handleIDentifier;
+		}
+//		return AbstractRepositoryTask.getHandle(repositoryUrl, id);
 	}
 
 	/**
diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractRepositoryTask.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractRepositoryTask.java
index 1d6ef0afd..cdc2f9311 100644
--- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractRepositoryTask.java
+++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/AbstractRepositoryTask.java
@@ -109,7 +109,7 @@ public abstract class AbstractRepositoryTask extends Task {
 	 * null if no such label exists.
 	 */
 	public String getIdLabel() {
-		return getTaskId(handle);
+		return getTaskId(handleIdentifier);
 	}
 	
 	public static String getTaskId(String taskHandle) {
diff --git a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/Task.java b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/Task.java
index 5cd38e50a..8448a4020 100644
--- a/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/Task.java
+++ b/org.eclipse.mylyn.tasks.core/src/org/eclipse/mylyn/tasks/core/Task.java
@@ -100,7 +100,7 @@ public class Task implements ITask {
 
 	private boolean active = false;
 
-	protected String handle = "-1";
+	protected String handleIdentifier = "-1";
 
 	private boolean category = false;
 
@@ -143,7 +143,7 @@ public class Task implements ITask {
 	}
 
 	public Task(String handle, String label, boolean newTask) {
-		this.handle = handle;
+		this.handleIdentifier = handle;
 		this.description = label;
 		if (newTask) {
 			creationDate = new Date();
@@ -151,11 +151,11 @@ public class Task implements ITask {
 	}
 
 	public String getHandleIdentifier() {
-		return handle;
+		return handleIdentifier;
 	}
 
 	public void setHandleIdentifier(String id) {
-		this.handle = id;
+		this.handleIdentifier = id;
 	}
 
 	public ITask getParent() {