From b18f2b60043f10ab56eb485e70a6d9abf149350e Mon Sep 17 00:00:00 2001 From: David McKnight Date: Wed, 9 Jul 2008 14:33:12 +0000 Subject: [PATCH] [237300] Problem with setDefaultHistory for SystemHistoryCombo. --- .../org/eclipse/rse/ui/SystemPreferencesManager.java | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/SystemPreferencesManager.java b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/SystemPreferencesManager.java index ff4fc5dfc0f..df732c9d5f2 100644 --- a/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/SystemPreferencesManager.java +++ b/rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/SystemPreferencesManager.java @@ -15,6 +15,7 @@ * the org.eclipse.rse.core package of the UI plugin. * Martin Oberhuber (Wind River) - [186773] split ISystemRegistryUI from ISystemRegistry * Martin Oberhuber (Wind River) - [215820] Move SystemRegistry implementation to Core + * David McKnight (IBM) - [237300] Problem with setDefaultHistory for SystemHistoryCombo. ********************************************************************************/ package org.eclipse.rse.ui; @@ -410,7 +411,16 @@ public class SystemPreferencesManager { */ public static String[] getWidgetHistory(String key) { Preferences store = RSEUIPlugin.getDefault().getPluginPreferences(); - return parseStrings(store.getString(key)); + String result = store.getString(key); + + // bug 237300 + // don't parse strings if we have "" + if (result == null || result.length() == 0){ + return null; + } + else { + return parseStrings(result); + } } /**