Attachment 66353 Details for Bug 185150 – Show conflict v01

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] Show conflict v01

keyConflict-v01.txt (text/plain), 3.05 KB, created by Paul Webster

on 2007-05-08 15:38:42 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Paul Webster

Created: 2007-05-08 15:38:42 EDT

Size: 3.05 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.ui.workbench
>Index: Eclipse UI/org/eclipse/ui/internal/keys/NewKeysPreferencePage.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/keys/NewKeysPreferencePage.java,v
>retrieving revision 1.59
>diff -u -r1.59 NewKeysPreferencePage.java
>--- Eclipse UI/org/eclipse/ui/internal/keys/NewKeysPreferencePage.java	8 May 2007 19:02:46 -0000	1.59
>+++ Eclipse UI/org/eclipse/ui/internal/keys/NewKeysPreferencePage.java	8 May 2007 19:37:33 -0000
>@@ -133,14 +133,14 @@
>  * </p>
>  * <p>
>  * This preference page has four general types of methods. Create methods are
>- * called when the page is first made visisble. They are responsible for
>- * creating all of the widgets, and laying them out within the preference page.
>- * Fill methods populate the contents of the widgets that contain collections of
>- * data from which items can be selected. The select methods respond to
>- * selection events from the user, such as a button press or a table selection.
>- * The update methods update the contents of various widgets based on the
>- * current state of the user interface. For example, the command name label will
>- * always try to match the current select in the binding table.
>+ * called when the page is first made visible. They are responsible for creating
>+ * all of the widgets, and laying them out within the preference page. Fill
>+ * methods populate the contents of the widgets that contain collections of data
>+ * from which items can be selected. The select methods respond to selection
>+ * events from the user, such as a button press or a table selection. The update
>+ * methods update the contents of various widgets based on the current state of
>+ * the user interface. For example, the command name label will always try to
>+ * match the current select in the binding table.
>  * </p>
>  * 
>  * @since 3.2
>@@ -603,6 +603,23 @@
> 			return null;
> 		}
> 
>+		private boolean checkConflict(Binding binding) {
>+			Collection matches = (Collection) localChangeManager
>+					.getActiveBindingsDisregardingContext().get(
>+							binding.getTriggerSequence());
>+			if (matches != null) {
>+				Iterator i = matches.iterator();
>+				while (i.hasNext()) {
>+					Binding b = (Binding) i.next();
>+					if (binding != b
>+							&& b.getContextId().equals(binding.getContextId())) {
>+						return true;
>+					}
>+				}
>+			}
>+			return false;
>+		}
>+
> 		public final String getColumnText(final Object element,
> 				final int columnIndex) {
> 			final Object value = element;
>@@ -617,6 +634,9 @@
> 						return NewKeysPreferenceMessages.Undefined_Command;
> 					}
> 				case COLUMN_TRIGGER_SEQUENCE:
>+					if (checkConflict(binding)) {
>+						return "*" + binding.getTriggerSequence().format(); //$NON-NLS-1$
>+					}
> 					return binding.getTriggerSequence().format();
> 
> 				case COLUMN_WHEN:
>@@ -701,6 +721,7 @@
> 			for (int i = 1; i < COLUMN_USER; i++) {
> 				String text = getColumnText(element, i);
> 				if (text != null) {
>+					buf.append(' ');
> 					buf.append(text);
> 				}
> 			}

Actions: View | Diff

Attachments on bug 185150: 66353 | 66383 | 66386