Minor fixes

3 files changed, 16 insertions, 10 deletions

diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/BugzillaTask.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/BugzillaTask.java
index bd5cc65d4..f0fac7247 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/BugzillaTask.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/BugzillaTask.java
@@ -327,10 +327,14 @@ public class BugzillaTask extends Task {
 	}
 	
 	public void updateTaskDetails() {
-		setPriority(bugReport.getAttribute("Priority").getValue());
-		String status = bugReport.getAttribute("Status").getValue();
-		if (status.equals("RESOLVED")) {
-			setCompleted(true);
+		try {
+			setPriority(bugReport.getAttribute("Priority").getValue());
+			String status = bugReport.getAttribute("Status").getValue();
+			if (status.equals("RESOLVED")) {
+				setCompleted(true);
+			}
+		} catch (NullPointerException npe) {
+			// TODO: handle this better
 		}
 	}
 	
diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/views/TaskListLabelProvider.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/views/TaskListLabelProvider.java
index 19c6325a3..605750f17 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/views/TaskListLabelProvider.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/views/TaskListLabelProvider.java
@@ -63,23 +63,28 @@ public class TaskListLabelProvider extends LabelProvider implements ITableLabelP
         if (element instanceof ITask) {
             ITask task = (ITask)element;            
             if (task.isActive()) return UiUtil.BOLD;            
-            if (task.isCompleted()) return UiUtil.ITALIC;
+//            if (task.isCompleted()) return UiUtil.ITALIC;
             for (ITask child : task.getChildren()) {
 				if (child.isActive())
 					return UiUtil.BOLD;
 			}
+            if (task instanceof BugzillaTask) {
+            	if (((BugzillaTask)task).getState() != BugzillaTask.BugTaskState.FREE) {
+            		return UiUtil.ITALIC;
+            	}
+            }
         } else if (element instanceof TaskCategory) {
         	TaskCategory cat = (TaskCategory) element;
             for (ITask child : cat.getChildren()) {
 				if (child.isActive())
 					return UiUtil.BOLD;
 			}
-        }else if (element instanceof BugzillaHit) {
+        } else if (element instanceof BugzillaHit) {
         	BugzillaHit hit = (BugzillaHit)element;
         	BugzillaTask task = hit.getAssociatedTask(); 
         	if(task != null){
 	            if (task.isActive()) return UiUtil.BOLD;            
-	            if (task.isCompleted()) return UiUtil.ITALIC;
+//	            if (task.isCompleted()) return UiUtil.ITALIC;
         	}
         } else if (element instanceof BugzillaQueryCategory) {
         	BugzillaQueryCategory cat = (BugzillaQueryCategory) element;
diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/views/TaskListView.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/views/TaskListView.java
index ea462bf15..36a751b9b 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/views/TaskListView.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/views/TaskListView.java
@@ -350,9 +350,6 @@ public class TaskListView extends ViewPart {
     		}
     	}
     	
-    	public void hidePriority(String p) {
-    		priorities.remove(p);
-    	}
 		@Override
 		public boolean select(Viewer viewer, Object parentElement, Object element) {
 			if (element instanceof ITaskListElement) {