Attachment 97109 Details for Bug 219540 – updated patch

[patch] updated patch

clipboard.txt (text/plain), 40.09 KB, created by Robert Elves

on 2008-04-22 18:56:59 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Robert Elves

Created: 2008-04-22 18:56:59 EDT

Size: 40.09 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.mylyn.tasks.ui
>Index: src/org/eclipse/mylyn/internal/tasks/ui/views/TaskListTableSorter.java
>===================================================================
>RCS file: /cvsroot/tools/org.eclipse.mylyn/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/views/TaskListTableSorter.java,v
>retrieving revision 1.37
>diff -u -r1.37 TaskListTableSorter.java
>--- src/org/eclipse/mylyn/internal/tasks/ui/views/TaskListTableSorter.java	1 Mar 2008 21:38:51 -0000	1.37
>+++ src/org/eclipse/mylyn/internal/tasks/ui/views/TaskListTableSorter.java	22 Apr 2008 22:44:27 -0000
>@@ -35,6 +35,10 @@
> 
> 	private SortByIndex sortByIndex = SortByIndex.PRIORITY;
> 
>+	private int sortDirection2 = DEFAULT_SORT_DIRECTION;
>+
>+	private SortByIndex sortByIndex2 = SortByIndex.DATE_CREATED;
>+
> 	private final TaskListView view;
> 
> 	private final TaskKeyComparator taskKeyComparator = new TaskKeyComparator();
>@@ -127,30 +131,48 @@
> 
> 	private int compareElements(AbstractTaskContainer element1, AbstractTaskContainer element2) {
> 		if (SortByIndex.PRIORITY.equals(sortByIndex)) {
>-			int result = this.sortDirection * element1.getPriority().compareTo(element2.getPriority());
>+			int result = sortByPriority(element1, element2, sortDirection);
> 			if (result != 0) {
> 				return result;
> 			}
>-			return sortBySummary(element1, element2);
>-
>+			if (SortByIndex.DATE_CREATED.equals(sortByIndex2)) {
>+				return sortByDate(element1, element2, sortDirection2);
>+			} else {
>+				if (SortByIndex.SUMMARY.equals(sortByIndex2)) {
>+					return sortBySummary(element1, element2, sortDirection2);
>+				} else {
>+					return result;
>+				}
>+			}
> 		} else if (SortByIndex.DATE_CREATED.equals(sortByIndex)) {
>-			AbstractTask t1 = null;
>-			AbstractTask t2 = null;
>-			if (element1 instanceof AbstractTask) {
>-				t1 = (AbstractTask) element1;
>-			}
>-			if (element2 instanceof AbstractTask) {
>-				t2 = (AbstractTask) element2;
>-			}
>-			if (t1 != null && t2 != null) {
>-				if (t1.getCreationDate() != null) {
>-					return t1.getCreationDate().compareTo(t2.getCreationDate());
>+			int result = sortByDate(element1, element2, sortDirection);
>+			if (result != 0) {
>+				return result;
>+			}
>+			if (SortByIndex.PRIORITY.equals(sortByIndex2)) {
>+				return sortByPriority(element1, element2, sortDirection2);
>+			} else {
>+				if (SortByIndex.SUMMARY.equals(sortByIndex2)) {
>+					return sortBySummary(element1, element2, sortDirection2);
>+				} else {
>+					return result;
> 				}
> 			}
> 		} else {
>-			return sortBySummary(element1, element2);
>+			int result = sortBySummary(element1, element2, sortDirection);
>+			if (result != 0) {
>+				return result;
>+			}
>+			if (SortByIndex.DATE_CREATED.equals(sortByIndex2)) {
>+				return sortByDate(element1, element2, sortDirection2);
>+			} else {
>+				if (SortByIndex.PRIORITY.equals(sortByIndex2)) {
>+					return sortByPriority(element1, element2, sortDirection2);
>+				} else {
>+					return result;
>+				}
>+			}
> 		}
>-		return 0;
> 	}
> 
> 	/**
>@@ -160,12 +182,47 @@
> 	 * @param element2
> 	 * @return sort order
> 	 */
>-	private int sortBySummary(AbstractTaskContainer element1, AbstractTaskContainer element2) {
>-		return this.sortDirection
>+	private int sortBySummary(AbstractTaskContainer element1, AbstractTaskContainer element2, int sortDirection) {
>+		return sortDirection
> 				* taskKeyComparator.compare(getSortableFromElement(element1), getSortableFromElement(element2));
> 	}
> 
> 	/**
>+	 * Determine the sort order of two tasks by priority
>+	 * 
>+	 * @param element1
>+	 * @param element2
>+	 * @return sort order
>+	 */
>+	private int sortByPriority(AbstractTaskContainer element1, AbstractTaskContainer element2, int sortDirection) {
>+		return sortDirection * element1.getPriority().compareTo(element2.getPriority());
>+	}
>+
>+	/**
>+	 * Determine the sort order of two tasks by creation date
>+	 * 
>+	 * @param element1
>+	 * @param element2
>+	 * @return sort order
>+	 */
>+	private int sortByDate(AbstractTaskContainer element1, AbstractTaskContainer element2, int sortDirection) {
>+		AbstractTask t1 = null;
>+		AbstractTask t2 = null;
>+		if (element1 instanceof AbstractTask) {
>+			t1 = (AbstractTask) element1;
>+		}
>+		if (element2 instanceof AbstractTask) {
>+			t2 = (AbstractTask) element2;
>+		}
>+		if (t1 != null && t2 != null) {
>+			if (t1.getCreationDate() != null) {
>+				return sortDirection * t1.getCreationDate().compareTo(t2.getCreationDate());
>+			}
>+		}
>+		return 0;
>+	}
>+
>+	/**
> 	 * Return a sortable string in the format "key: summary"
> 	 * 
> 	 * @param element
>@@ -228,4 +285,29 @@
> 		}
> 	}
> 
>+	public SortByIndex getSortByIndex2() {
>+		return sortByIndex2;
>+	}
>+
>+	public void setSortByIndex2(SortByIndex sortByIndex) {
>+		SortByIndex oldValue = this.sortByIndex2;
>+		this.sortByIndex2 = sortByIndex;
>+		if (!oldValue.equals(sortByIndex)) {
>+			view.getViewer().refresh();
>+		}
>+
>+	}
>+
>+	public int getSortDirection2() {
>+		return sortDirection2;
>+	}
>+
>+	public void setSortDirection2(int sortDirection) {
>+		int oldValue = this.sortDirection2;
>+		this.sortDirection2 = sortDirection;
>+		if (oldValue != this.sortDirection2) {
>+			view.getViewer().refresh();
>+		}
>+	}
>+
> }
>Index: src/org/eclipse/mylyn/internal/tasks/ui/views/SortyByDropDownAction.java
>===================================================================
>RCS file: src/org/eclipse/mylyn/internal/tasks/ui/views/SortyByDropDownAction.java
>diff -N src/org/eclipse/mylyn/internal/tasks/ui/views/SortyByDropDownAction.java
>--- src/org/eclipse/mylyn/internal/tasks/ui/views/SortyByDropDownAction.java	26 Nov 2007 19:31:39 -0000	1.7
>+++ /dev/null	1 Jan 1970 00:00:00 -0000
>@@ -1,143 +0,0 @@
>-/*******************************************************************************
>- * Copyright (c) 2004, 2007 Mylyn project committers and others.
>- * All rights reserved. This program and the accompanying materials
>- * are made available under the terms of the Eclipse Public License v1.0
>- * which accompanies this distribution, and is available at
>- * http://www.eclipse.org/legal/epl-v10.html
>- *******************************************************************************/
>-
>-package org.eclipse.mylyn.internal.tasks.ui.views;
>-
>-import org.eclipse.jface.action.Action;
>-import org.eclipse.jface.action.ActionContributionItem;
>-import org.eclipse.jface.action.IMenuCreator;
>-import org.eclipse.jface.action.Separator;
>-import org.eclipse.mylyn.internal.tasks.ui.TasksUiImages;
>-import org.eclipse.mylyn.internal.tasks.ui.views.TaskListTableSorter.SortByIndex;
>-import org.eclipse.swt.widgets.Control;
>-import org.eclipse.swt.widgets.Menu;
>-
>-/**
>- * @author Mik Kersten
>- */
>-class SortyByDropDownAction extends Action implements IMenuCreator {
>-
>-	private final TaskListView taskListView;
>-
>-	private static final String LABEL = "Sort by";
>-
>-	private Action byPriority;
>-
>-	private Action bySummary;
>-
>-	private Action byDateCreated;
>-
>-	private Menu dropDownMenu = null;
>-
>-	public SortyByDropDownAction(TaskListView taskListView) {
>-		super();
>-		this.taskListView = taskListView;
>-		setText(LABEL);
>-		setToolTipText(LABEL);
>-		setMenuCreator(this);
>-	}
>-
>-	public void dispose() {
>-		if (dropDownMenu != null) {
>-			dropDownMenu.dispose();
>-			dropDownMenu = null;
>-		}
>-	}
>-
>-	public Menu getMenu(Control parent) {
>-		if (dropDownMenu != null) {
>-			dropDownMenu.dispose();
>-		}
>-		dropDownMenu = new Menu(parent);
>-		addActionsToMenu();
>-		return dropDownMenu;
>-	}
>-
>-	public Menu getMenu(Menu parent) {
>-		if (dropDownMenu != null) {
>-			dropDownMenu.dispose();
>-		}
>-		dropDownMenu = new Menu(parent);
>-		addActionsToMenu();
>-		return dropDownMenu;
>-	}
>-
>-	public void addActionsToMenu() {
>-		byPriority = new Action("", AS_CHECK_BOX) {
>-			@Override
>-			public void run() {
>-				taskListView.getSorter().setSortByIndex(SortByIndex.PRIORITY);
>-				byPriority.setChecked(true);
>-				bySummary.setChecked(false);
>-				byDateCreated.setChecked(false);
>-			}
>-		};
>-		byPriority.setEnabled(true);
>-		byPriority.setText("Priority");
>-		byPriority.setImageDescriptor(TasksUiImages.PRIORITY_1);
>-		new ActionContributionItem(byPriority).fill(dropDownMenu, -1);
>-
>-		bySummary = new Action("", AS_CHECK_BOX) {
>-			@Override
>-			public void run() {
>-				taskListView.getSorter().setSortByIndex(SortByIndex.SUMMARY);
>-				byPriority.setChecked(false);
>-				bySummary.setChecked(true);
>-				byDateCreated.setChecked(false);
>-			}
>-		};
>-		bySummary.setEnabled(true);
>-		bySummary.setText("Summary");
>-		new ActionContributionItem(bySummary).fill(dropDownMenu, -1);
>-
>-		byDateCreated = new Action("", AS_CHECK_BOX) {
>-			@Override
>-			public void run() {
>-				taskListView.getSorter().setSortByIndex(SortByIndex.DATE_CREATED);
>-				byPriority.setChecked(false);
>-				bySummary.setChecked(false);
>-				byDateCreated.setChecked(true);
>-			}
>-		};
>-		byDateCreated.setEnabled(true);
>-		byDateCreated.setText("Date Created");
>-		byDateCreated.setImageDescriptor(TasksUiImages.CALENDAR_SMALL);
>-		new ActionContributionItem(byDateCreated).fill(dropDownMenu, -1);
>-
>-		new Separator().fill(dropDownMenu, -1);
>-
>-		Action reverse = new Action("", AS_CHECK_BOX) {
>-			@Override
>-			public void run() {
>-				taskListView.getSorter().setSortDirection(taskListView.getSorter().getSortDirection() * -1);
>-				setChecked(taskListView.getSorter().getSortDirection() < 0);
>-			}
>-		};
>-		reverse.setEnabled(true);
>-		reverse.setText("Descending");
>-		reverse.setChecked(taskListView.getSorter().getSortDirection() < 0);
>-		new ActionContributionItem(reverse).fill(dropDownMenu, -1);
>-
>-		switch (taskListView.getSorter().getSortByIndex()) {
>-		case PRIORITY:
>-			byPriority.setChecked(true);
>-			break;
>-		case SUMMARY:
>-			bySummary.setChecked(true);
>-			break;
>-		case DATE_CREATED:
>-			byDateCreated.setChecked(true);
>-			break;
>-		}
>-	}
>-
>-	@Override
>-	public void run() {
>-		this.setChecked(isChecked());
>-	}
>-}
>\ No newline at end of file
>Index: src/org/eclipse/mylyn/internal/tasks/ui/views/TaskListView.java
>===================================================================
>RCS file: /cvsroot/tools/org.eclipse.mylyn/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/views/TaskListView.java,v
>retrieving revision 1.244
>diff -u -r1.244 TaskListView.java
>--- src/org/eclipse/mylyn/internal/tasks/ui/views/TaskListView.java	21 Apr 2008 07:19:50 -0000	1.244
>+++ src/org/eclipse/mylyn/internal/tasks/ui/views/TaskListView.java	22 Apr 2008 22:44:27 -0000
>@@ -87,6 +87,7 @@
> import org.eclipse.mylyn.internal.tasks.ui.actions.TaskActivateAction;
> import org.eclipse.mylyn.internal.tasks.ui.actions.TaskDeactivateAction;
> import org.eclipse.mylyn.internal.tasks.ui.actions.TaskListElementPropertiesAction;
>+import org.eclipse.mylyn.internal.tasks.ui.actions.TaskListSortAction;
> import org.eclipse.mylyn.internal.tasks.ui.editors.TaskListChangeAdapter;
> import org.eclipse.mylyn.internal.tasks.ui.util.TaskDragSourceListener;
> import org.eclipse.mylyn.internal.tasks.ui.views.TaskListTableSorter.SortByIndex;
>@@ -268,7 +269,7 @@
> 
> 	private PriorityDropDownAction filterOnPriorityAction;
> 
>-	private SortyByDropDownAction sortByAction;
>+	private TaskListSortAction sortDialogAction;
> 
> 	private PresentationDropDownSelectionAction presentationDropDownSelectionAction;
> 
>@@ -1075,7 +1076,7 @@
> 		manager.add(collapseAll);
> 		manager.add(expandAll);
> 		manager.add(new Separator(ID_SEPARATOR_FILTERS));
>-		manager.add(sortByAction);
>+		manager.add(sortDialogAction);
> 		manager.add(filterOnPriorityAction);
> 		manager.add(filterCompleteTask);
> 		//manager.add(filterArchiveCategory);
>@@ -1311,7 +1312,9 @@
> 		synchronizeAutomatically = new SynchronizeAutomaticallyAction();
> 		openPreferencesAction = new OpenTasksUiPreferencesAction();
> 		//filterArchiveCategory = new FilterArchiveContainerAction(this);
>-		sortByAction = new SortyByDropDownAction(this);
>+
>+		sortDialogAction = new TaskListSortAction(getSite(), this);
>+
> 		filterOnPriorityAction = new PriorityDropDownAction(this);
> 		linkWithEditorAction = new LinkWithEditorAction(this);
> 		presentationDropDownSelectionAction = new PresentationDropDownSelectionAction(this);
>@@ -1580,7 +1583,7 @@
> 	}
> 
> 	public void setManualFiltersEnabled(boolean enabled) {
>-		sortByAction.setEnabled(enabled);
>+		sortDialogAction.setEnabled(enabled);
> 		filterOnPriorityAction.setEnabled(enabled);
> 		filterCompleteTask.setEnabled(enabled);
> 		//filterArchiveCategory.setEnabled(enabled);
>Index: src/org/eclipse/mylyn/internal/tasks/ui/util/TasksUiExtensionReader.java
>===================================================================
>RCS file: /cvsroot/tools/org.eclipse.mylyn/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/internal/tasks/ui/util/TasksUiExtensionReader.java,v
>retrieving revision 1.47
>diff -u -r1.47 TasksUiExtensionReader.java
>--- src/org/eclipse/mylyn/internal/tasks/ui/util/TasksUiExtensionReader.java	12 Apr 2008 05:35:12 -0000	1.47
>+++ src/org/eclipse/mylyn/internal/tasks/ui/util/TasksUiExtensionReader.java	22 Apr 2008 22:44:27 -0000
>@@ -26,10 +26,10 @@
> import org.eclipse.mylyn.internal.tasks.ui.views.AbstractTaskListPresentation;
> import org.eclipse.mylyn.internal.tasks.ui.views.TaskListView;
> import org.eclipse.mylyn.monitor.core.StatusHandler;
>+import org.eclipse.mylyn.tasks.core.AbstractDuplicateDetector;
> import org.eclipse.mylyn.tasks.core.AbstractRepositoryConnector;
> import org.eclipse.mylyn.tasks.core.AbstractTaskListFactory;
> import org.eclipse.mylyn.tasks.core.RepositoryTemplate;
>-import org.eclipse.mylyn.tasks.ui.AbstractDuplicateDetector;
> import org.eclipse.mylyn.tasks.ui.AbstractRepositoryConnectorUi;
> import org.eclipse.mylyn.tasks.ui.AbstractTaskRepositoryLinkProvider;
> import org.eclipse.mylyn.tasks.ui.TasksUi;
>Index: .refactorings/2008/4/17/refactorings.history
>===================================================================
>RCS file: /cvsroot/tools/org.eclipse.mylyn/org.eclipse.mylyn.tasks.ui/.refactorings/2008/4/17/refactorings.history,v
>retrieving revision 1.2
>diff -u -r1.2 refactorings.history
>--- .refactorings/2008/4/17/refactorings.history	22 Apr 2008 17:30:50 -0000	1.2
>+++ .refactorings/2008/4/17/refactorings.history	22 Apr 2008 22:44:27 -0000
>@@ -1,3 +1,3 @@
> <?xml version="1.0" encoding="UTF-8"?>
>-<session version="1.0">&#x0A;<refactoring comment="Rename type &apos;org.eclipse.mylyn.internal.tasks.ui.views.ResetRepositoryConfigurationAction&apos; to &apos;RefreshRepositoryConfigurationAction&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Original element: &apos;org.eclipse.mylyn.internal.tasks.ui.views.ResetRepositoryConfigurationAction&apos;&#x0A;- Renamed element: &apos;org.eclipse.mylyn.internal.tasks.ui.views.RefreshRepositoryConfigurationAction&apos;&#x0A;- Update references to refactored element&#x0A;- Update textual occurrences in comments and strings" description="Rename type &apos;ResetRepositoryConfigurationAction&apos;" flags="589830" id="org.eclipse.jdt.ui.rename.type" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.views{ResetRepositoryConfigurationAction.java[ResetRepositoryConfigurationAction" matchStrategy="1" name="RefreshRepositoryConfigurationAction" qualified="false" references="true" similarDeclarations="false" stamp="1208751210799" textual="false" version="1.0"/>&#x0A;<refactoring comment="Rename type &apos;org.eclipse.mylyn.internal.tasks.ui.views.RefreshRepositoryConfigurationAction&apos; to &apos;UpdateRepositoryConfigurationAction&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Original element: &apos;org.eclipse.mylyn.internal.tasks.ui.views.RefreshRepositoryConfigurationAction&apos;&#x0A;- Renamed element: &apos;org.eclipse.mylyn.internal.tasks.ui.views.UpdateRepositoryConfigurationAction&apos;&#x0A;- Update references to refactored element&#x0A;- Update textual occurrences in comments and strings" description="Rename type &apos;RefreshRepositoryConfigurationAction&apos;" flags="589830" id="org.eclipse.jdt.ui.rename.type" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.views{RefreshRepositoryConfigurationAction.java[RefreshRepositoryConfigurationAction" matchStrategy="1" name="UpdateRepositoryConfigurationAction" qualified="false" references="true" similarDeclarations="false" stamp="1208751224911" textual="false" version="1.0"/>&#x0A;<refactoring comment="Change method &apos;public org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.AbstractTaskEditorPart(AbstractTaskEditorPage taskEditorPage)&apos; to &apos;public AbstractTaskEditorPart()&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Original element: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.AbstractTaskEditorPart(...)&apos;&#x0A;- Removed parameters:&#x0A;     AbstractTaskEditorPage taskEditorPage" delegate="false" deprecate="true" description="Change method &apos;AbstractTaskEditorPart&apos;" flags="589830" id="org.eclipse.jdt.ui.change.method.signature" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.editors{AbstractTaskEditorPart.java[AbstractTaskEditorPart~AbstractTaskEditorPart~QAbstractTaskEditorPage;" name="AbstractTaskEditorPart" parameter1="AbstractTaskEditorPage taskEditorPage 0 {deleted} {deleted} true" stamp="1208759242282" version="1.0"/>&#x0A;<refactoring comment="Rename method &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.setInput(...)&apos; to &apos;initialize&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Original element: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.setInput(...)&apos;&#x0A;- Renamed element: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.initialize(...)&apos;&#x0A;- Update references to refactored element" delegate="false" deprecate="false" description="Rename method &apos;setInput&apos;" flags="589830" id="org.eclipse.jdt.ui.rename.method" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.editors{AbstractTaskEditorPart.java[AbstractTaskEditorPart~setInput~QAttributeManager;" name="initialize" references="true" stamp="1208759295126" version="1.0"/>&#x0A;<refactoring comment="Extract method &apos;private ImageHyperlink createTitleHyperLink(final FormToolkit toolkit,final Composite toolbarComp,final TaskComment taskComment)&apos; from &apos;org.eclipse.mylyn.internal.tasks.ui.editors.TaskEditorCommentPart.createControl()&apos; to &apos;org.eclipse.mylyn.internal.tasks.ui.editors.TaskEditorCommentPart&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Method name: &apos;createTitleHyperLink&apos;&#x0A;- Destination type: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.TaskEditorCommentPart&apos;&#x0A;- Declared visibility: &apos;private&apos;" comments="false" description="Extract method &apos;createTitleHyperLink&apos;" destination="0" exceptions="false" flags="786434" id="org.eclipse.jdt.ui.extract.method" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.editors{TaskEditorCommentPart.java" name="createTitleHyperLink" replace="false" selection="4528 1255" stamp="1208861366936" version="1.0" visibility="2"/>&#x0A;<refactoring comment="Extract local variable &apos;author&apos; from expression &apos;taskComment.getAuthor()&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Variable name: &apos;author&apos;&#x0A;- Destination method: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.TaskEditorCommentPart.createTitleHyperLink()&apos;&#x0A;- Variable expression: &apos;taskComment.getAuthor()&apos;&#x0A;- Replace occurrences of expression with variable" description="Extract local variable &apos;author&apos;" final="false" id="org.eclipse.jdt.ui.extract.temp" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.editors{TaskEditorCommentPart.java" name="author" replace="true" selection="9069 23" stamp="1208861500492" version="1.0"/>&#x0A;<refactoring comment="Copy 1 elements(s) to &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AttachmentTableLabelProvider.java&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Destination element: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AttachmentTableLabelProvider.java&apos;&#x0A;- Original element: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AttachmentTableLabelProvider.java&apos;" description="Copy compilation unit" destination="/src&lt;org.eclipse.mylyn.internal.tasks.ui.editors{AttachmentTableLabelProvider.java" element1="/src&lt;org.eclipse.mylyn.internal.tasks.ui.editors{AttachmentTableLabelProvider.java" files="0" flags="589830" folders="0" id="org.eclipse.jdt.ui.copy" policy="org.eclipse.jdt.ui.copyResources" stamp="1208862191463" units="1" version="1.0"/>
>+<session version="1.0">&#x0A;<refactoring comment="Rename type &apos;org.eclipse.mylyn.internal.tasks.ui.views.ResetRepositoryConfigurationAction&apos; to &apos;RefreshRepositoryConfigurationAction&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Original element: &apos;org.eclipse.mylyn.internal.tasks.ui.views.ResetRepositoryConfigurationAction&apos;&#x0A;- Renamed element: &apos;org.eclipse.mylyn.internal.tasks.ui.views.RefreshRepositoryConfigurationAction&apos;&#x0A;- Update references to refactored element&#x0A;- Update textual occurrences in comments and strings" description="Rename type &apos;ResetRepositoryConfigurationAction&apos;" flags="589830" id="org.eclipse.jdt.ui.rename.type" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.views{ResetRepositoryConfigurationAction.java[ResetRepositoryConfigurationAction" matchStrategy="1" name="RefreshRepositoryConfigurationAction" qualified="false" references="true" similarDeclarations="false" stamp="1208751210799" textual="false" version="1.0"/>&#x0A;<refactoring comment="Rename type &apos;org.eclipse.mylyn.internal.tasks.ui.views.RefreshRepositoryConfigurationAction&apos; to &apos;UpdateRepositoryConfigurationAction&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Original element: &apos;org.eclipse.mylyn.internal.tasks.ui.views.RefreshRepositoryConfigurationAction&apos;&#x0A;- Renamed element: &apos;org.eclipse.mylyn.internal.tasks.ui.views.UpdateRepositoryConfigurationAction&apos;&#x0A;- Update references to refactored element&#x0A;- Update textual occurrences in comments and strings" description="Rename type &apos;RefreshRepositoryConfigurationAction&apos;" flags="589830" id="org.eclipse.jdt.ui.rename.type" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.views{RefreshRepositoryConfigurationAction.java[RefreshRepositoryConfigurationAction" matchStrategy="1" name="UpdateRepositoryConfigurationAction" qualified="false" references="true" similarDeclarations="false" stamp="1208751224911" textual="false" version="1.0"/>&#x0A;<refactoring comment="Change method &apos;public org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.AbstractTaskEditorPart(AbstractTaskEditorPage taskEditorPage)&apos; to &apos;public AbstractTaskEditorPart()&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Original element: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.AbstractTaskEditorPart(...)&apos;&#x0A;- Removed parameters:&#x0A;     AbstractTaskEditorPage taskEditorPage" delegate="false" deprecate="true" description="Change method &apos;AbstractTaskEditorPart&apos;" flags="589830" id="org.eclipse.jdt.ui.change.method.signature" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.editors{AbstractTaskEditorPart.java[AbstractTaskEditorPart~AbstractTaskEditorPart~QAbstractTaskEditorPage;" name="AbstractTaskEditorPart" parameter1="AbstractTaskEditorPage taskEditorPage 0 {deleted} {deleted} true" stamp="1208759242282" version="1.0"/>&#x0A;<refactoring comment="Rename method &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.setInput(...)&apos; to &apos;initialize&apos;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0A;- Original element: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.setInput(...)&apos;&#x0A;- Renamed element: &apos;org.eclipse.mylyn.internal.tasks.ui.editors.AbstractTaskEditorPart.initialize(...)&apos;&#x0A;- Update references to refactored element" delegate="false" deprecate="false" description="Rename method &apos;setInput&apos;" flags="589830" id="org.eclipse.jdt.ui.rename.method" input="/src&lt;org.eclipse.mylyn.internal.tasks.ui.editors{AbstractTaskEditorPart.java[AbstractTaskEditorPart~setInput~QAttributeManager;" name="initialize" references="true" stamp="1208759295126" version="1.0"/>&#x0A;<refactoring comment="Move 1 elements(s) to &apos;org.eclipse.mylyn.tasks.core/src/org.eclipse.mylyn.tasks.core&apos;&#x0D;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0D;&#x0A;- Destination element: &apos;org.eclipse.mylyn.tasks.core/src/org.eclipse.mylyn.tasks.core&apos;&#x0D;&#x0A;- Original element: &apos;org.eclipse.mylyn.tasks.ui.AbstractDuplicateDetector.java&apos;&#x0D;&#x0A;- Update references to refactored element" description="Move compilation unit" destination="=org.eclipse.mylyn.tasks.core/src&lt;org.eclipse.mylyn.tasks.core" element1="/src&lt;org.eclipse.mylyn.tasks.ui{AbstractDuplicateDetector.java" files="0" flags="589830" folders="0" id="org.eclipse.jdt.ui.move" policy="org.eclipse.jdt.ui.moveResources" qualified="false" references="true" stamp="1208900485972" units="1" version="1.0"/>&#x0A;<refactoring comment="Move &apos;org.eclipse.mylyn.tasks.ui.editors.AbstractNewRepositoryTaskEditor.getStackTraceFromDescription(...)&apos; to &apos;org.eclipse.mylyn.tasks.core.AbstractDuplicateDetector&apos;&#x0D;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0D;&#x0A;- Original element: &apos;org.eclipse.mylyn.tasks.ui.editors.AbstractNewRepositoryTaskEditor.getStackTraceFromDescription(...)&apos;&#x0D;&#x0A;- Destination type: &apos;org.eclipse.mylyn.tasks.core.AbstractDuplicateDetector&apos;" delegate="false" deprecate="true" description="Move member &apos;getStackTraceFromDescription&apos;" element1="/src&lt;org.eclipse.mylyn.tasks.ui.editors{AbstractNewRepositoryTaskEditor.java[AbstractNewRepositoryTaskEditor~getStackTraceFromDescription~QString;" flags="589830" id="org.eclipse.jdt.ui.move.static" input="=org.eclipse.mylyn.tasks.core/src&lt;org.eclipse.mylyn.tasks.core{AbstractDuplicateDetector.java[AbstractDuplicateDetector" stamp="1208900968231" version="1.0"/>&#x0A;<refactoring comment="Rename method &apos;org.eclipse.mylyn.tasks.ui.editors.AbstractRepositoryTaskEditor.getDuplicateSearchCollector(...)&apos; to &apos;getDuplicateDetector&apos;&#x0D;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0D;&#x0A;- Original element: &apos;org.eclipse.mylyn.tasks.ui.editors.AbstractRepositoryTaskEditor.getDuplicateSearchCollector(...)&apos;&#x0D;&#x0A;- Renamed element: &apos;org.eclipse.mylyn.tasks.ui.editors.AbstractRepositoryTaskEditor.getDuplicateDetector(...)&apos;&#x0D;&#x0A;- Update references to refactored element" delegate="false" deprecate="false" description="Rename method &apos;getDuplicateSearchCollector&apos;" flags="589830" id="org.eclipse.jdt.ui.rename.method" input="/src&lt;org.eclipse.mylyn.tasks.ui.editors{AbstractRepositoryTaskEditor.java[AbstractRepositoryTaskEditor~getDuplicateSearchCollector~QString;" name="getDuplicateDetector" references="true" stamp="1208903018379" version="1.0"/>&#x0A;<refactoring comment="Rename method &apos;org.eclipse.mylyn.tasks.ui.editors.AbstractRepositoryTaskEditor.getDuplicateSearchCollectorsList()&apos; to &apos;getDuplicateDetectorList&apos;&#x0D;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0D;&#x0A;- Original element: &apos;org.eclipse.mylyn.tasks.ui.editors.AbstractRepositoryTaskEditor.getDuplicateSearchCollectorsList()&apos;&#x0D;&#x0A;- Renamed element: &apos;org.eclipse.mylyn.tasks.ui.editors.AbstractRepositoryTaskEditor.getDuplicateDetectorList()&apos;&#x0D;&#x0A;- Update references to refactored element" delegate="false" deprecate="false" description="Rename method &apos;getDuplicateSearchCollectorsList&apos;" flags="589830" id="org.eclipse.jdt.ui.rename.method" input="/src&lt;org.eclipse.mylyn.tasks.ui.editors{AbstractRepositoryTaskEditor.java[AbstractRepositoryTaskEditor~getDuplicateSearchCollectorsList" name="getDuplicateDetectorList" references="true" stamp="1208903049251" version="1.0"/>&#x0A;<refactoring comment="Change method &apos;protected AbstractRepositoryQuery org.eclipse.mylyn.tasks.ui.editors.AbstractRepositoryTaskEditor.getDuplicateDetector(String name)&apos; to &apos;protected AbstractDuplicateDetector getDuplicateDetector(String name)&apos;&#x0D;&#x0A;- Original project: &apos;org.eclipse.mylyn.tasks.ui&apos;&#x0D;&#x0A;- Original element: &apos;org.eclipse.mylyn.tasks.ui.editors.AbstractRepositoryTaskEditor.getDuplicateDetector(...)&apos;&#x0D;&#x0A;- New return type: &apos;AbstractDuplicateDetector&apos;" delegate="false" deprecate="true" description="Change method &apos;getDuplicateDetector&apos;" flags="589830" id="org.eclipse.jdt.ui.change.method.signature" input="/src&lt;org.eclipse.mylyn.tasks.ui.editors{AbstractRepositoryTaskEditor.java[AbstractRepositoryTaskEditor~getDuplicateDetector~QString;" name="getDuplicateDetector" parameter1="String name 0 String name false" return="AbstractDuplicateDetector" stamp="1208903086583" version="1.0"/>
> </session>
>\ No newline at end of file
>Index: src/org/eclipse/mylyn/internal/tasks/ui/actions/TaskListSortAction.java
>===================================================================
>RCS file: src/org/eclipse/mylyn/internal/tasks/ui/actions/TaskListSortAction.java
>diff -N src/org/eclipse/mylyn/internal/tasks/ui/actions/TaskListSortAction.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ src/org/eclipse/mylyn/internal/tasks/ui/actions/TaskListSortAction.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,31 @@
>+/*******************************************************************************
>+ * Copyright (c) 2004, 2007 Mylyn project committers and others.
>+ * All rights reserved. This program and the accompanying materials
>+ * are made available under the terms of the Eclipse Public License v1.0
>+ * which accompanies this distribution, and is available at
>+ * http://www.eclipse.org/legal/epl-v10.html
>+ *******************************************************************************/
>+
>+package org.eclipse.mylyn.internal.tasks.ui.actions;
>+
>+import org.eclipse.jface.action.Action;
>+import org.eclipse.mylyn.internal.tasks.ui.dialogs.TaskListSortDialog;
>+import org.eclipse.mylyn.internal.tasks.ui.views.TaskListView;
>+import org.eclipse.ui.IWorkbenchPartSite;
>+
>+public class TaskListSortAction extends Action {
>+
>+	private final TaskListSortDialog dialog;
>+
>+	public TaskListSortAction(IWorkbenchPartSite site, TaskListView taskListView) {
>+		super("Task Sort Order");
>+		setEnabled(true);
>+		dialog = new TaskListSortDialog(site, taskListView);
>+	}
>+
>+	@Override
>+	public void run() {
>+		dialog.open();
>+	}
>+
>+}
>Index: src/org/eclipse/mylyn/internal/tasks/ui/dialogs/TaskListSortDialog.java
>===================================================================
>RCS file: src/org/eclipse/mylyn/internal/tasks/ui/dialogs/TaskListSortDialog.java
>diff -N src/org/eclipse/mylyn/internal/tasks/ui/dialogs/TaskListSortDialog.java
>--- /dev/null	1 Jan 1970 00:00:00 -0000
>+++ src/org/eclipse/mylyn/internal/tasks/ui/dialogs/TaskListSortDialog.java	1 Jan 1970 00:00:00 -0000
>@@ -0,0 +1,265 @@
>+/*******************************************************************************
>+ * Copyright (c) 2004, 2007 Mylyn project committers and others.
>+ * All rights reserved. This program and the accompanying materials
>+ * are made available under the terms of the Eclipse Public License v1.0
>+ * which accompanies this distribution, and is available at
>+ * http://www.eclipse.org/legal/epl-v10.html
>+ *******************************************************************************/
>+
>+package org.eclipse.mylyn.internal.tasks.ui.dialogs;
>+
>+import java.util.ArrayList;
>+import java.util.Arrays;
>+import java.util.Comparator;
>+
>+import org.eclipse.jface.dialogs.Dialog;
>+import org.eclipse.jface.window.IShellProvider;
>+import org.eclipse.mylyn.internal.tasks.ui.views.TaskListView;
>+import org.eclipse.mylyn.internal.tasks.ui.views.TaskListTableSorter.SortByIndex;
>+import org.eclipse.swt.SWT;
>+import org.eclipse.swt.events.SelectionAdapter;
>+import org.eclipse.swt.events.SelectionEvent;
>+import org.eclipse.swt.layout.GridData;
>+import org.eclipse.swt.layout.GridLayout;
>+import org.eclipse.swt.widgets.Button;
>+import org.eclipse.swt.widgets.Combo;
>+import org.eclipse.swt.widgets.Composite;
>+import org.eclipse.swt.widgets.Control;
>+import org.eclipse.swt.widgets.Label;
>+
>+public class TaskListSortDialog extends Dialog {
>+	private Combo[] priorityCombos;
>+
>+	private Button[] ascendingButtons;
>+
>+	private Button[] descendingButtons;
>+
>+	private final String[] propertyText;
>+
>+	private boolean dirty = false;
>+
>+	private final TaskListView taskListView;
>+
>+	public TaskListSortDialog(IShellProvider parentShell, TaskListView taskListView) {
>+		super(parentShell);
>+		propertyText = new String[3];
>+		propertyText[0] = "Priority";
>+		propertyText[1] = "Summary";
>+		propertyText[2] = "Date_Created";
>+		this.taskListView = taskListView;
>+	}
>+
>+	@Override
>+	protected Control createDialogArea(Composite parent) {
>+		Composite composite = (Composite) super.createDialogArea(parent);
>+
>+		initializeDialogUnits(composite);
>+
>+		Composite prioritiesArea = new Composite(composite, SWT.NULL);
>+		prioritiesArea.setLayout(new GridLayout(3, false));
>+
>+		Label sortByLabel = new Label(prioritiesArea, SWT.NULL);
>+		sortByLabel.setText("Sort by:");
>+		GridData data = new GridData();
>+		data.horizontalSpan = 3;
>+		sortByLabel.setLayoutData(data);
>+
>+		ascendingButtons = new Button[2];
>+		descendingButtons = new Button[2];
>+		priorityCombos = new Combo[2];
>+
>+		for (int i = 0; i < 2; i++) {
>+			final int index = i;
>+			Label numberLabel = new Label(prioritiesArea, SWT.NULL);
>+			numberLabel.setText("" + (i + 1) + ".");
>+			priorityCombos[i] = new Combo(prioritiesArea, SWT.READ_ONLY);
>+			priorityCombos[i].setLayoutData(new GridData(GridData.FILL_HORIZONTAL));
>+
>+			Composite directionGroup = new Composite(prioritiesArea, SWT.NONE);
>+			directionGroup.setLayout(new GridLayout(2, false));
>+			ascendingButtons[i] = new Button(directionGroup, SWT.RADIO);
>+			ascendingButtons[i].setText("Ascending");
>+			ascendingButtons[i].addSelectionListener(new SelectionAdapter() {
>+				@Override
>+				public void widgetSelected(SelectionEvent e) {
>+					markDirty();
>+				}
>+			});
>+			descendingButtons[i] = new Button(directionGroup, SWT.RADIO);
>+			descendingButtons[i].setText("Descending");
>+			descendingButtons[i].addSelectionListener(new SelectionAdapter() {
>+				@Override
>+				public void widgetSelected(SelectionEvent e) {
>+					markDirty();
>+				}
>+			});
>+			if (i < priorityCombos.length - 1) {
>+				priorityCombos[i].addSelectionListener(new SelectionAdapter() {
>+					@Override
>+					public void widgetSelected(SelectionEvent e) {
>+						int oldSelectionDirection = 1;
>+						if (descendingButtons[index].getSelection()) {
>+							oldSelectionDirection = -1;
>+						}
>+						ArrayList<String> oldSelectionList = new ArrayList<String>(
>+								Arrays.asList(priorityCombos[index].getItems()));
>+						oldSelectionList.removeAll(Arrays.asList(priorityCombos[index + 1].getItems()));
>+						if (oldSelectionList.size() != 1) {
>+							return;
>+						}
>+						String oldSelection = oldSelectionList.get(0);
>+						String newSelection = priorityCombos[index].getItem(priorityCombos[index].getSelectionIndex());
>+						if (oldSelection.equals(newSelection)) {
>+							return;
>+						}
>+						for (int j = index + 1; j < priorityCombos.length; j++) {
>+							int newSelectionIndex = priorityCombos[j].indexOf(newSelection);
>+							//this combo's current selection is equal to newSelection
>+							if (priorityCombos[j].getSelectionIndex() == newSelectionIndex) {
>+								priorityCombos[j].remove(newSelection);
>+								int insertionPoint = -1
>+										- Arrays.binarySearch(priorityCombos[j].getItems(), oldSelection,
>+												columnComparator);
>+								if (insertionPoint >= 0 && insertionPoint <= priorityCombos[j].getItemCount()) {
>+									priorityCombos[j].add(oldSelection, insertionPoint);
>+								} else {
>+									priorityCombos[j].add(oldSelection);
>+								}
>+								priorityCombos[j].select(priorityCombos[j].indexOf(oldSelection));
>+								ascendingButtons[index].setSelection(ascendingButtons[j].getSelection());
>+								descendingButtons[index].setSelection(descendingButtons[j].getSelection());
>+								ascendingButtons[j].setSelection(oldSelectionDirection == 1);
>+								descendingButtons[j].setSelection(oldSelectionDirection == -1);
>+							}
>+							//this combo contains newSelection
>+							else if (newSelectionIndex >= 0) {
>+								String currentText = priorityCombos[j].getText();
>+								priorityCombos[j].remove(newSelection);
>+								int insertionPoint = -1
>+										- Arrays.binarySearch(priorityCombos[j].getItems(), oldSelection,
>+												columnComparator);
>+								if (insertionPoint >= 0 && insertionPoint <= priorityCombos[j].getItemCount()) {
>+									priorityCombos[j].add(oldSelection, insertionPoint);
>+									priorityCombos[j].select(priorityCombos[j].indexOf(currentText));
>+								} else {
>+									priorityCombos[j].add(oldSelection);
>+								}
>+							}
>+						}
>+						markDirty();
>+					}
>+				});
>+			} else {
>+				priorityCombos[i].addSelectionListener(new SelectionAdapter() {
>+					@Override
>+					public void widgetSelected(SelectionEvent e) {
>+						markDirty();
>+					}
>+				});
>+			}
>+
>+		}
>+		int a[] = new int[2];
>+		int b[] = new int[2];
>+		switch (taskListView.getSorter().getSortByIndex()) {
>+		case PRIORITY:
>+			a[0] = 0;
>+			break;
>+		case SUMMARY:
>+			a[0] = 1;
>+			break;
>+		case DATE_CREATED:
>+			a[0] = 2;
>+			break;
>+		}
>+
>+		switch (taskListView.getSorter().getSortByIndex2()) {
>+		case PRIORITY:
>+			a[1] = 0;
>+			break;
>+		case SUMMARY:
>+			a[1] = 1;
>+			break;
>+		case DATE_CREATED:
>+			a[1] = 2;
>+			break;
>+		}
>+		b[0] = taskListView.getSorter().getSortDirection();
>+		b[1] = taskListView.getSorter().getSortDirection2();
>+		updateUI(a, b);
>+		return composite;
>+	}
>+
>+	@Override
>+	protected void okPressed() {
>+		if (isDirty()) {
>+			taskListView.getSorter()
>+					.setSortByIndex(
>+							SortByIndex.valueOf(priorityCombos[0].getItem(priorityCombos[0].getSelectionIndex())
>+									.toUpperCase()));
>+			taskListView.getSorter()
>+					.setSortByIndex2(
>+							SortByIndex.valueOf(priorityCombos[1].getItem(priorityCombos[1].getSelectionIndex())
>+									.toUpperCase()));
>+			if (descendingButtons[0].getSelection()) {
>+				taskListView.getSorter().setSortDirection(-1);
>+			} else {
>+				taskListView.getSorter().setSortDirection(1);
>+			}
>+			if (descendingButtons[1].getSelection()) {
>+				taskListView.getSorter().setSortDirection2(-1);
>+			} else {
>+				taskListView.getSorter().setSortDirection2(1);
>+			}
>+
>+		}
>+		super.okPressed();
>+	}
>+
>+	/**
>+	 * @return boolean
>+	 */
>+	public boolean isDirty() {
>+		return dirty;
>+	}
>+
>+	/**
>+	 * Sets the dirty flag to true.
>+	 */
>+	public void markDirty() {
>+		dirty = true;
>+	}
>+
>+	private final Comparator<String> columnComparator = new Comparator<String>() {
>+		public int compare(String arg0, String arg1) {
>+			int index0 = -1;
>+			int index1 = -1;
>+			for (int i = 0; i < propertyText.length; i++) {
>+				if (propertyText[i].equals(arg0)) {
>+					index0 = i;
>+				}
>+				if (propertyText[i].equals(arg1)) {
>+					index1 = i;
>+				}
>+			}
>+			return index0 - index1;
>+		}
>+	};
>+
>+	private void updateUI(int[] priorities, int[] directions) {
>+		ArrayList<String> availablePriorities = new ArrayList<String>(Arrays.asList(propertyText));
>+
>+		for (int i = 0; i < priorityCombos.length; i++) {
>+			priorityCombos[i].removeAll();
>+			for (int j = 0; j < availablePriorities.size(); j++) {
>+				priorityCombos[i].add(availablePriorities.get(j));
>+			}
>+			priorityCombos[i].select(priorityCombos[i].indexOf(propertyText[priorities[i]]));
>+			availablePriorities.remove(propertyText[priorities[i]]);
>+
>+			ascendingButtons[i].setSelection(directions[i] == 1);
>+			descendingButtons[i].setSelection(directions[i] == -1);
>+		}
>+	}
>+
>+}

Actions: View | Diff

Attachments on bug 219540: 90402 | 90403 | 90581 | 90582 | 94449 | 94974 | 94975 | 94976 | 94977 | 97109 | 97531 | 97532 | 98030 | 98540 | 98541 | 98951 | 98952