From ba9afbb1c988e6aad88db09bafc598fe00fd76ee Mon Sep 17 00:00:00 2001
From: Kenny Root <kenny@the-b.org>
Date: Sun, 7 Feb 2010 22:36:05 +0000
Subject: Check for host nickname being null

The HostBean is allowed to have a null nickname in some circumstances,
so be sure to check that when we're editing port forwards.

git-svn-id: https://connectbot.googlecode.com/svn/trunk/connectbot@468 df292f66-193f-0410-a5fc-6d59da041ff2
---
 AndroidManifest.xml                             |  2 +-
 src/org/connectbot/ConsoleActivity.java         | 14 ++++++++++----
 src/org/connectbot/PortForwardListActivity.java | 20 ++++++++++++++++----
 3 files changed, 27 insertions(+), 9 deletions(-)

diff --git a/AndroidManifest.xml b/AndroidManifest.xml
index db51eeb..07eca89 100644
--- a/AndroidManifest.xml
+++ b/AndroidManifest.xml
@@ -2,7 +2,7 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
 	package="org.connectbot"
 	android:versionName="1.7-dev"
-	android:versionCode="247">
+	android:versionCode="248">
 
 	<application
 		android:icon="@drawable/icon"
diff --git a/src/org/connectbot/ConsoleActivity.java b/src/org/connectbot/ConsoleActivity.java
index 066cc32..d27a3bf 100644
--- a/src/org/connectbot/ConsoleActivity.java
+++ b/src/org/connectbot/ConsoleActivity.java
@@ -147,13 +147,19 @@ public class ConsoleActivity extends Activity {
 
 			// first check if we need to create a new session for requested
 			boolean found = false;
-			for(TerminalBridge bridge : bound.bridges) {
-				if(bridge.host.getNickname().equals(requestedNickname))
+			for (TerminalBridge bridge : bound.bridges) {
+				String nick = bridge.host.getNickname();
+				if (nick == null)
+					continue;
+
+				if (nick.equals(requestedNickname)) {
 					found = true;
+					break;
+				}
 			}
 
 			// If we didn't find the requested connection, try opening it
-			if(!found) {
+			if (!found) {
 				try {
 					Log.d(TAG, String.format("We couldnt find an existing bridge with URI=%s (nickname=%s), so creating one now", requested.toString(), requestedNickname));
 					bound.openConnection(requested);
@@ -163,7 +169,7 @@ public class ConsoleActivity extends Activity {
 			}
 
 			// create views for all bridges on this service
-			for(TerminalBridge bridge : bound.bridges) {
+			for (TerminalBridge bridge : bound.bridges) {
 
 				// let them know about our prompt handler services
 				bridge.promptHelper.setHandler(promptHandler);
diff --git a/src/org/connectbot/PortForwardListActivity.java b/src/org/connectbot/PortForwardListActivity.java
index 1d35372..8b3655c 100644
--- a/src/org/connectbot/PortForwardListActivity.java
+++ b/src/org/connectbot/PortForwardListActivity.java
@@ -33,6 +33,7 @@ import android.content.Context;
 import android.content.DialogInterface;
 import android.content.Intent;
 import android.content.ServiceConnection;
+import android.content.res.Resources;
 import android.database.SQLException;
 import android.graphics.Paint;
 import android.os.Bundle;
@@ -112,10 +113,21 @@ public class PortForwardListActivity extends ListActivity {
 		this.hostdb = new HostDatabase(this);
 		host = hostdb.findHostById(hostId);
 
-		this.setTitle(String.format("%s: %s (%s)",
-				getResources().getText(R.string.app_name),
-				getResources().getText(R.string.title_port_forwards_list),
-				host.getNickname()));
+		{
+			String nickname = host.getNickname();
+			final Resources resources = getResources();
+
+			if (nickname != null) {
+				this.setTitle(String.format("%s: %s (%s)",
+						resources.getText(R.string.app_name),
+						resources.getText(R.string.title_port_forwards_list),
+						nickname));
+			} else {
+				this.setTitle(String.format("%s: %s",
+						resources.getText(R.string.app_name),
+						resources.getText(R.string.title_port_forwards_list)));
+			}
+		}
 
 		connection = new ServiceConnection() {
 			public void onServiceConnected(ComponentName className, IBinder service) {
-- 
cgit v1.2.3