From 1d9e074af19ca4171ebed0d7697c37d7af69d1ad Mon Sep 17 00:00:00 2001
From: David Dykstal <david_dykstal@us.ibm.com>
Date: Wed, 23 May 2007 18:47:43 +0000
Subject: [PATCH] [188718] fixing remote search dialog mnemonics, fixing create
 remote project strings

---
 .../org.eclipse.rse.eclipse.filesystem/plugin.xml |  4 ++--
 .../CreateRemoteProjectActionDelegate.java        | 10 +++++-----
 .../eclipse/filesystem/ui/actions/Messages.java   | 15 +++++++++++++++
 .../filesystem/ui/actions/messages.properties     | 14 ++++++++++++++
 .../files/ui/search/SystemSearchPage.java         |  9 ++++++++-
 .../UI/org/eclipse/rse/ui/Mnemonics.java          | 10 +++++++++-
 6 files changed, 53 insertions(+), 9 deletions(-)
 create mode 100644 rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/Messages.java
 create mode 100644 rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/messages.properties

diff --git a/rse/plugins/org.eclipse.rse.eclipse.filesystem/plugin.xml b/rse/plugins/org.eclipse.rse.eclipse.filesystem/plugin.xml
index ef154ae54a0..0ab12b326e8 100644
--- a/rse/plugins/org.eclipse.rse.eclipse.filesystem/plugin.xml
+++ b/rse/plugins/org.eclipse.rse.eclipse.filesystem/plugin.xml
@@ -41,8 +41,8 @@ Kushal Munir (IBM) - Changed paths to reflect internal packages
                 	<objectState name="isdirectory" value="true"/>                  
             </visibility>
          <action
-               label="%CreateRemoteProject.label"
-                tooltip="%CreateRemoteProject.tooltip"
+               label="%createRemoteProject.label"
+                tooltip="%createRemoteProject.tooltip"
                class="org.eclipse.rse.internal.eclipse.filesystem.ui.actions.CreateRemoteProjectActionDelegate"
                menubarPath="group.importexport"
                enablesFor="1"
diff --git a/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/CreateRemoteProjectActionDelegate.java b/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/CreateRemoteProjectActionDelegate.java
index 8246e51db48..2d61e7149dd 100644
--- a/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/CreateRemoteProjectActionDelegate.java
+++ b/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/CreateRemoteProjectActionDelegate.java
@@ -118,7 +118,7 @@ public class CreateRemoteProjectActionDelegate implements IActionDelegate {
 	 * @return java.lang.String
 	 */
 	protected String getErrorsTitle() {
-		return "Error creating remote project";
+		return "Error creating remote project"; //$NON-NLS-1$
 	}
 
 	/**
@@ -152,14 +152,14 @@ public class CreateRemoteProjectActionDelegate implements IActionDelegate {
 			ProgressMonitorDialog mon = new ProgressMonitorDialog(getShell()) {
 			    protected void configureShell(Shell shell) {
 			        super.configureShell(shell);
-					shell.setText("Creating remote Project");
+					shell.setText(Messages.CreateRemoteProjectActionDelegate_CREATING_TITLE);
 			    }
 			};
 			mon.run(true, true, op);
 		} catch (InterruptedException e) {
 			return false;
 		} catch (InvocationTargetException e) {
-			displayError("Internal Error: "+e.getTargetException().getMessage());
+			displayError("Internal Error: "+e.getTargetException().getMessage()); //$NON-NLS-1$
 			return false;
 		}
 
@@ -202,11 +202,11 @@ public class CreateRemoteProjectActionDelegate implements IActionDelegate {
 		}
 		catch (CoreException e)
 		{
-			SystemBasePlugin.logError("Error creating temp project", e);
+			SystemBasePlugin.logError("Error creating temp project", e); //$NON-NLS-1$
 		}
 		catch (Exception e)
 		{
-			SystemBasePlugin.logError("Error creating temp project", e);
+			SystemBasePlugin.logError("Error creating temp project", e); //$NON-NLS-1$
 		}
 		return editProject;
 	}
diff --git a/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/Messages.java b/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/Messages.java
new file mode 100644
index 00000000000..ea35b29ae77
--- /dev/null
+++ b/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/Messages.java
@@ -0,0 +1,15 @@
+package org.eclipse.rse.internal.eclipse.filesystem.ui.actions;
+
+import org.eclipse.osgi.util.NLS;
+
+public class Messages extends NLS {
+	private static final String BUNDLE_NAME = "org.eclipse.rse.internal.eclipse.filesystem.ui.actions.messages"; //$NON-NLS-1$
+	public static String CreateRemoteProjectActionDelegate_CREATING_TITLE;
+	static {
+// initialize resource bundle
+		NLS.initializeMessages(BUNDLE_NAME, Messages.class);
+	}
+
+	private Messages() {
+	}
+}
diff --git a/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/messages.properties b/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/messages.properties
new file mode 100644
index 00000000000..128e0b6df9d
--- /dev/null
+++ b/rse/plugins/org.eclipse.rse.eclipse.filesystem/src/org/eclipse/rse/internal/eclipse/filesystem/ui/actions/messages.properties
@@ -0,0 +1,14 @@
+################################################################################
+# Copyright (c) 2007 IBM Corporation. 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
+# 
+# Contributors:
+# David Dykstal (IBM) - initial contribution
+################################################################################
+
+# NLS_MESSAGEFORMAT_NONE
+# NLS_ENCODING=UTF-8
+
+CreateRemoteProjectActionDelegate_CREATING_TITLE=Creating Remote Project
diff --git a/rse/plugins/org.eclipse.rse.files.ui/src/org/eclipse/rse/internal/files/ui/search/SystemSearchPage.java b/rse/plugins/org.eclipse.rse.files.ui/src/org/eclipse/rse/internal/files/ui/search/SystemSearchPage.java
index a016f550fdb..c7ea2804f8f 100644
--- a/rse/plugins/org.eclipse.rse.files.ui/src/org/eclipse/rse/internal/files/ui/search/SystemSearchPage.java
+++ b/rse/plugins/org.eclipse.rse.files.ui/src/org/eclipse/rse/internal/files/ui/search/SystemSearchPage.java
@@ -48,10 +48,12 @@ import org.eclipse.rse.subsystems.files.core.servicesubsystem.FileServiceSubSyst
 import org.eclipse.rse.subsystems.files.core.subsystems.IRemoteFile;
 import org.eclipse.rse.subsystems.files.core.subsystems.IRemoteFileSubSystem;
 import org.eclipse.rse.ui.ISystemMessages;
+import org.eclipse.rse.ui.Mnemonics;
 import org.eclipse.rse.ui.RSEUIPlugin;
 import org.eclipse.rse.ui.SystemBasePlugin;
 import org.eclipse.rse.ui.SystemWidgetHelpers;
 import org.eclipse.rse.ui.messages.SystemMessageDialog;
+import org.eclipse.search.internal.ui.SearchMessages;
 import org.eclipse.search.ui.ISearchPage;
 import org.eclipse.search.ui.ISearchPageContainer;
 import org.eclipse.swt.SWT;
@@ -1001,7 +1003,12 @@ public class SystemSearchPage extends DialogPage implements ISearchPage {
 		createAdvancedControls(main);
 			
 		// set mnemonics
-		SystemWidgetHelpers.setMnemonics(parent);
+		Mnemonics m = new Mnemonics();
+		StringBuffer usedMnemonics = new StringBuffer(5);
+		usedMnemonics.append(Mnemonics.getMnemonic(SearchMessages.SearchDialog_customize));
+		usedMnemonics.append(Mnemonics.getMnemonic(SearchMessages.SearchDialog_searchAction));
+		m.clear(usedMnemonics.toString());
+		m.setMnemonics(parent);
 		
 		// set the top level control
 		setControl(main);
diff --git a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/Mnemonics.java b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/Mnemonics.java
index 6dab111de17..8628f77b875 100644
--- a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/Mnemonics.java
+++ b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/Mnemonics.java
@@ -139,6 +139,14 @@ public class Mnemonics {
 	public void clear() {
 		usedSet.clear();
 	}
+	
+	/**
+	 * Resets the list of used mnemonic characters to those in the string.
+	 * @param usedMnemonics
+	 */
+	public void clear(String usedMnemonics) {
+		makeUsed(usedMnemonics);
+	}
 
 	/**
 	 * Given a string, this starts at the first character and iterates until
@@ -491,7 +499,7 @@ public class Mnemonics {
 			}
 		}
 	}
-
+	
 	/** 
 	 * Set if the mnemonics are for a preference page
 	 * Preference pages already have a few buttons with mnemonics set by Eclipse