From 433b8aaa71ab6283231b95022954d6d3298b93e0 Mon Sep 17 00:00:00 2001
From: Sebastian Kaspari <s.kaspari@googlemail.com>
Date: Sun, 25 Apr 2010 11:00:58 +0200
Subject: [PATCH] Database: Added setChannels(), getChannelsByServerId() will
 now return an ArrayList of channel names

---
 src/org/yaaic/db/Database.java | 47 ++++++++++++++++++++++++++++------
 1 file changed, 39 insertions(+), 8 deletions(-)

diff --git a/src/org/yaaic/db/Database.java b/src/org/yaaic/db/Database.java
index d27f524..b60394a 100644
--- a/src/org/yaaic/db/Database.java
+++ b/src/org/yaaic/db/Database.java
@@ -20,6 +20,7 @@ along with Yaaic.  If not, see <http://www.gnu.org/licenses/>.
 */
 package org.yaaic.db;
 
+import java.util.ArrayList;
 import java.util.HashMap;
 
 import org.yaaic.model.Identity;
@@ -171,21 +172,42 @@ public class Database extends SQLiteOpenHelper
 	/**
 	 * Add a channel to the database
 	 * 
-	 * @param server Unique id of server
+	 * @param serverId Unique id of server
 	 * @param name Name of channel
 	 * @param password Password to join (if needed)
 	 */
-	public void addChannel(int server, String name, String password)
+	public void addChannel(int serverId, String name, String password)
 	{
 		ContentValues values = new ContentValues();
 		
 		values.put(ChannelConstants.NAME, name);
 		values.put(ChannelConstants.PASSWORD, password);
-		values.put(ChannelConstants.SERVER, server);
+		values.put(ChannelConstants.SERVER, serverId);
 		
 		this.getWritableDatabase().insert(ServerConstants.TABLE_NAME, null, values);
 	}
 	
+	/**
+	 * Replace list of channels for the given server
+	 * 
+	 * @param serverId Unique id of server
+	 * @param channels List of channel names
+	 */
+	public void setChannels(int serverId, ArrayList<String> channels)
+	{
+		// Remove old channels
+		this.getWritableDatabase().delete(
+			ChannelConstants.TABLE_NAME,
+			ChannelConstants.SERVER + " = " + serverId,
+			null
+		);
+		
+		// Add new channels
+		for (String channel : channels) {
+			addChannel(serverId, channel, "");
+		}
+	}
+	
 	/**
 	 * Get all servers from database
 	 * 
@@ -318,13 +340,13 @@ public class Database extends SQLiteOpenHelper
 	 * Get all channels of server
 	 * 
 	 * @param server Unique id of server
-	 * @return
+	 * @return list of channel names
 	 */
-	public Cursor getChannelsByServerId(int serverId)
+	public ArrayList<String> getChannelsByServerId(int serverId)
 	{
-		// XXX: Should no return a cursor but the populated objects
+		ArrayList<String> channels = new ArrayList<String>();
 		
-		return this.getReadableDatabase().query(
+		Cursor cursor = this.getReadableDatabase().query(
 			ChannelConstants.TABLE_NAME,
 			ChannelConstants.ALL,
 			ChannelConstants.SERVER + "=" + serverId,
@@ -332,7 +354,16 @@ public class Database extends SQLiteOpenHelper
 			null,
 			null,
 			ChannelConstants.NAME + " ASC"
-		);	
+		);
+		
+		while (cursor.moveToNext()) {
+			String channel = cursor.getString(cursor.getColumnIndex(ChannelConstants.NAME));
+			channels.add(channel);
+		}
+		
+		cursor.close();
+		
+		return channels;
 	}
 	
 	/**