xan/awips2
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge "Omaha #3236 thin client connectivity dialog remembers past proxy servers" into omaha_14.4.1

Browse source 
Former-commit-id: 8ad1122d65 [formerly 42495a5551] [formerly 7b44d8ad2d] [formerly 7b44d8ad2d [formerly 589f74f6d8]] [formerly 8ad1122d65 [formerly 42495a5551] [formerly 7b44d8ad2d] [formerly 7b44d8ad2d [formerly 589f74f6d8]] [formerly 49a592b19f [formerly 7b44d8ad2d [formerly 589f74f6d8] [formerly 49a592b19f [formerly 3d352136996875ff1e66b9e41244d53e16a1b2c3]]]]]
Former-commit-id: 49a592b19f
Former-commit-id: 6ff7c4946d [formerly 78ccc8922c] [formerly c73138b29e] [formerly a8418457d697575644a626cdc5db3e39fde4f8dc [formerly 07db96ba799a55bc88db475276f8fed8e524ee12] [formerly c73138b29e [formerly 2b3f7f02e5]]]
Former-commit-id: 1706c050acacf270e0d2a75a69548c8bcb168c81 [formerly d8b0dddfe938e999c3a69db10cfa3c66ec3973ba] [formerly 30d4d8fe27 [formerly e5c63a67ac]]
Former-commit-id: 30d4d8fe27
Former-commit-id: 66ab17a9a8
This commit is contained in:
Nate Jensen 2014-06-25 12:07:28 -05:00 • committed by Gerrit Code Review
commit 0439bec93a
2 changed files with 56 additions and 32 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
cave/com.raytheon.uf.viz.thinclient/src/com/raytheon/uf/viz/thinclient/preferences/ThinClientPreferenceConstants.java
View file

@ -30,7 +30,8 @@ package com.raytheon.uf.viz.thinclient.preferences;
* ------------ ---------- ----------- --------------------------
* Oct 20, 2011 mschenke Initial creation
* Jan 14, 2013 1469 bkowal The hdf5 data directory is no longer a preference constant.
* Feb 04, 2014 2704 njensen Single proxy preference constant
* Feb 04, 2014 2704 njensen Consolidate services and pypies proxy addresses
* Jun 24, 2014 3236 njensen Added proxy address options
*
* </pre>
*
@ -69,4 +70,6 @@ public class ThinClientPreferenceConstants {
public static String P_PREFERENCE_PLACEHOLDER = "placeholderPreference";
public static String P_PROXY_ADDRESS = "proxyAddress";
public static final String P_PROXY_SERVER_OPTIONS = "proxyAddressOptions";
}

83
cave/com.raytheon.uf.viz.thinclient/src/com/raytheon/uf/viz/thinclient/ui/ThinClientConnectivityDialog.java
View file

@ -31,7 +31,6 @@ import org.eclipse.swt.layout.GridLayout;
import org.eclipse.swt.widgets.Button;
import org.eclipse.swt.widgets.Composite;
import org.eclipse.swt.widgets.Label;
import org.eclipse.swt.widgets.Text;
import com.raytheon.uf.common.localization.msgs.GetServersResponse;
import com.raytheon.uf.common.status.IUFStatusHandler;
@ -44,6 +43,8 @@ import com.raytheon.uf.viz.core.exception.VizException;
import com.raytheon.uf.viz.core.localization.ConnectivityPreferenceDialog;
import com.raytheon.uf.viz.core.localization.LocalizationConstants;
import com.raytheon.uf.viz.core.localization.LocalizationManager;
import com.raytheon.uf.viz.core.localization.ServerRemembrance;
import com.raytheon.uf.viz.core.localization.TextOrCombo;
import com.raytheon.uf.viz.thinclient.Activator;
import com.raytheon.uf.viz.thinclient.ThinClientUriUtil;
import com.raytheon.uf.viz.thinclient.preferences.ThinClientPreferenceConstants;
@ -65,6 +66,7 @@ import com.raytheon.uf.viz.thinclient.preferences.ThinClientPreferenceConstants;
* Feb 20, 2014 2704 njensen Fix issues where settings are valid
* but dialog doesn't realize it
* Jun 03, 2014 3217 bsteffen Add option to always open startup dialog.
* Jun 24, 2014 3236 njensen Add ability to remember multiple servers
*
*
*
@ -141,7 +143,7 @@ public class ThinClientConnectivityDialog extends ConnectivityPreferenceDialog {
private IConnectivityCallback jmsCallback = new JmsCallback();
private Text proxyText;
private TextOrCombo proxySrv;
private String proxyAddress;
@ -183,11 +185,15 @@ public class ThinClientConnectivityDialog extends ConnectivityPreferenceDialog {
}
});
proxyText = new Text(proxyComp, SWT.NONE | SWT.BORDER);
IPreferenceStore thinPrefs = Activator.getDefault()
.getPreferenceStore();
String[] proxyOptions = ServerRemembrance.getServerOptions(thinPrefs,
ThinClientPreferenceConstants.P_PROXY_SERVER_OPTIONS);
proxySrv = new TextOrCombo(proxyComp, SWT.BORDER, proxyOptions);
gd = new GridData(SWT.FILL, SWT.CENTER, true, true);
proxyText.setLayoutData(gd);
proxyText.setText(proxyAddress == null ? "" : proxyAddress);
proxyText.setBackground(getTextColor(servicesGood && pypiesGood));
proxySrv.widget.setLayoutData(gd);
proxySrv.setText(proxyAddress == null ? "" : proxyAddress);
proxySrv.widget.setBackground(getTextColor(servicesGood && pypiesGood));
new Label(textBoxComp, SWT.NONE);
@ -235,20 +241,28 @@ public class ThinClientConnectivityDialog extends ConnectivityPreferenceDialog {
@Override
protected void applySettings() {
IPersistentPreferenceStore localStore = LocalizationManager
.getInstance()
.getLocalizationStore();
IPersistentPreferenceStore store = Activator.getDefault()
IPersistentPreferenceStore thinStore = Activator.getDefault()
.getPreferenceStore();
store.setValue(ThinClientPreferenceConstants.P_DISABLE_JMS, disableJms);
thinStore.setValue(ThinClientPreferenceConstants.P_DISABLE_JMS,
disableJms);
thinStore.setValue(ThinClientPreferenceConstants.P_USE_PROXIES,
useProxy);
IPersistentPreferenceStore localStore = LocalizationManager
.getInstance().getLocalizationStore();
localStore.setValue(
LocalizationConstants.P_LOCALIZATION_PROMPT_ON_STARTUP,
alwaysPrompt);
if (useProxy) {
store.setValue(ThinClientPreferenceConstants.P_USE_PROXIES,
useProxy);
store.setValue(ThinClientPreferenceConstants.P_PROXY_ADDRESS,
thinStore.setValue(ThinClientPreferenceConstants.P_PROXY_ADDRESS,
proxyAddress);
String proxyServerOptions = ServerRemembrance.formatServerOptions(
proxyAddress, thinStore,
ThinClientPreferenceConstants.P_PROXY_SERVER_OPTIONS);
thinStore.setValue(
ThinClientPreferenceConstants.P_PROXY_SERVER_OPTIONS,
proxyServerOptions);
if (getAlertVizServer() != null) {
localStore.setValue(LocalizationConstants.P_ALERT_SERVER,
@ -256,13 +270,6 @@ public class ThinClientConnectivityDialog extends ConnectivityPreferenceDialog {
}
LocalizationManager.getInstance().setCurrentSite(getSite());
try {
store.save();
} catch (IOException e) {
statusHandler.handle(Priority.SIGNIFICANT,
"Unable to persist thinclient localization preference store",
e);
}
try {
localStore.save();
} catch (IOException e) {
@ -273,6 +280,19 @@ public class ThinClientConnectivityDialog extends ConnectivityPreferenceDialog {
super.applySettings();
}
/*
* Have to store the thin client preferences either way to remember the
* JMS and proxy checkboxes correctly
*/
try {
thinStore.save();
} catch (IOException e) {
statusHandler
.handle(Priority.SIGNIFICANT,
"Unable to persist thinclient localization preference store",
e);
}
}
@Override
@ -287,9 +307,9 @@ public class ThinClientConnectivityDialog extends ConnectivityPreferenceDialog {
details = null;
// validate proxy
if (proxyText != null && !proxyText.isDisposed()
&& proxyText.isEnabled()) {
proxyAddress = proxyText.getText();
if (proxySrv != null && !proxySrv.widget.isDisposed()
&& proxySrv.widget.isEnabled()) {
proxyAddress = proxySrv.getText();
}
if (proxyAddress != null && proxyAddress.length() > 0) {
validateServices();
@ -297,8 +317,9 @@ public class ThinClientConnectivityDialog extends ConnectivityPreferenceDialog {
} else {
status = "Please enter a thin client proxy server address";
}
if (proxyText != null && !proxyText.isDisposed()) {
proxyText.setBackground(getTextColor(servicesGood && pypiesGood));
if (proxySrv != null && !proxySrv.widget.isDisposed()) {
proxySrv.widget.setBackground(getTextColor(servicesGood
&& pypiesGood));
}
validateJms(servicesGood);
@ -345,15 +366,15 @@ public class ThinClientConnectivityDialog extends ConnectivityPreferenceDialog {
private void updateProxyEnabled() {
useProxy = useProxyCheck.getSelection();
proxyText.setEnabled(useProxy);
proxySrv.widget.setEnabled(useProxy);
super.setLocalizationEnabled(!useProxy);
if (useProxy) {
if (localizationText != null && !localizationText.isDisposed()) {
localizationText.setBackground(getTextColor(true));
if (localizationSrv != null && !localizationSrv.widget.isDisposed()) {
localizationSrv.widget.setBackground(getTextColor(true));
}
} else {
if (proxyText != null && !proxyText.isDisposed()) {
proxyText.setBackground(getTextColor(true));
if (proxySrv != null && !proxySrv.widget.isDisposed()) {
proxySrv.widget.setBackground(getTextColor(true));
}
}
validate();