# HG changeset patch
# User Ed Swartz <ed.swartz@nokia.com>
# Date 1272049004 18000
# Node ID dcb72c9e21265278edafefcead1f20dc2211397a
# Parent  6a0e8b61069563c8f91dc28142b9bddbcf77654a
Fix NPE and default connection selector case

diff -r 6a0e8b610695 -r dcb72c9e2126 connectivity/com.nokia.carbide.remoteConnections/src/com/nokia/carbide/remoteconnections/internal/ui/ClientServiceSiteUI2.java
--- a/connectivity/com.nokia.carbide.remoteConnections/src/com/nokia/carbide/remoteconnections/internal/ui/ClientServiceSiteUI2.java	Fri Apr 23 13:43:04 2010 -0500
+++ b/connectivity/com.nokia.carbide.remoteConnections/src/com/nokia/carbide/remoteconnections/internal/ui/ClientServiceSiteUI2.java	Fri Apr 23 13:56:44 2010 -0500
@@ -95,7 +95,7 @@
 			public void selectionChanged(SelectionChangedEvent event) {
 				IStructuredSelection selection = (IStructuredSelection) event.getSelection();
 				String connection = (String) selection.getFirstElement();
-				if (!connection.equals(ClientServiceSiteUI2.this.connection)) {
+				if (connection != null && !connection.equals(ClientServiceSiteUI2.this.connection)) {
 					ClientServiceSiteUI2.this.connection = connection;
 					fireConnectionSelected();
 				}
@@ -225,7 +225,8 @@
 			if (connection == null) {
 				String lastConnectionId = Registry.instance().getLastConnectionId();
 				if (lastConnectionId != null) {
-					selectConnection(lastConnectionId);
+					viewer.getCombo().select(0);  // the default setting
+					selectConnection(lastConnectionId);  // should override if connection still exists
 				} else {
 					viewer.getCombo().select(0);
 					viewer.setSelection(viewer.getSelection());