Minor fixes.

3 files changed, 12 insertions, 3 deletions

diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/MylarTasksPlugin.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/MylarTasksPlugin.java
index 4d0b70f05..053cef9d4 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/MylarTasksPlugin.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/MylarTasksPlugin.java
@@ -195,8 +195,8 @@ public class MylarTasksPlugin extends AbstractUIPlugin {
     protected void initializeDefaultPreferences(IPreferenceStore store) {
     	store.setDefault(SELECTED_PRIORITY, "P5");
     	store.setDefault(MylarPlugin.CLOSE_EDITORS, true);
-    	store.setDefault(REPORT_OPEN_EDITOR, false);
-    	store.setDefault(REPORT_OPEN_INTERNAL, true);
+    	store.setDefault(REPORT_OPEN_EDITOR, true);
+    	store.setDefault(REPORT_OPEN_INTERNAL, false);
     	store.setDefault(REPORT_OPEN_EXTERNAL, false);
     }
 
diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/internal/TaskListExternalizer.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/internal/TaskListExternalizer.java
index 050948a8a..da672eef3 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/internal/TaskListExternalizer.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/internal/TaskListExternalizer.java
@@ -261,6 +261,9 @@ public class TaskListExternalizer {
 		if (hasCaughtException) {
 			// if exception was caught, write out the new task file, so that it doesn't happen again.
 			// this is OK, since the original (corrupt) tasklist is saved.
+			// TODO: The problem with this is that if the orignal tasklist has tasks and bug reports, but a 
+			// task is corrupted, the new tasklist that is written will not include the bug reports (since the
+			// bugzilla externalizer is not loaded. So there is a potentila that we can lose bug reports.
 			writeTaskList(tlist, inFile);
 		}
 	}
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 57f841dfa..eaaf9ef03 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
@@ -276,6 +276,9 @@ public class TaskListView extends ViewPart {
 			if (element instanceof ITaskListElement) {
 				if(element instanceof ITask && ((ITaskListElement)element).hasCorrespondingActivatableTask()){
 					ITask task = ((ITaskListElement)element).getOrCreateCorrespondingTask();
+					if (task.isActive()) {
+						return true;
+					}
 					if(task != null){
 						return !task.isCompleted();
 					} else {
@@ -347,7 +350,10 @@ public class TaskListView extends ViewPart {
 			}
 		}
 		private boolean checkTask(ITaskListElement task) {
-			for (String filter : priorities) {
+			if (task instanceof ITask && ((ITask)task).isActive()) {
+				return true;
+			}
+			for (String filter : priorities) {				
 				if (task.getPriority().equals(filter)) {
 					return true;
 				}