package org.sagebionetworks.repo.model.dbo.persistence.discussion;

import java.nio.charset.Charset;
import java.util.ArrayList;
import java.util.List;

import org.sagebionetworks.util.ValidateArgument;

public class DiscussionThreadUtils {
	public static final Charset UTF8 = Charset.forName("UTF-8");

	/**
	 * Create a DBO from user's request
	 * 
	 * @param forumId
	 * @param title
	 * @param messageKey
	 * @param userId
	 * @param id
	 * @param etag
	 * @return
	 */
	public static DBODiscussionThread createDBO(String forumId, String title,
			String messageKey, Long userId, String id, String etag) {
		ValidateArgument.required(forumId, "forumId");
		ValidateArgument.required(title, "title");
		ValidateArgument.required(messageKey, "messageUrl");
		ValidateArgument.required(userId, "userId");
		ValidateArgument.required(id, "id");
		ValidateArgument.required(etag, "etag");
		DBODiscussionThread dbo = new DBODiscussionThread();
		dbo.setId(Long.parseLong(id));
		dbo.setForumId(Long.parseLong(forumId));
		dbo.setTitle(title.getBytes(UTF8));
		dbo.setMessageKey(messageKey);
		dbo.setCreatedBy(userId);
		dbo.setIsEdited(false);
		dbo.setIsDeleted(false);
		dbo.setEtag(etag);
		return dbo;
	}

	/**
	 * convert an input String to a list of String
	 * the input has the following format:
	 * <string1>,<string2>,...
	 * 
	 * @param inputString
	 * @return
	 */
	public static List<String> toList(String inputString) {
		if (inputString == null) {
			throw new IllegalArgumentException("inputString");
		}
		List<String> list = new ArrayList<String>();
		if (inputString.equals("")) {
			return list;
		}
		String[] elements = inputString.split(",");
		for (String string : elements) {
			list.add(string);
		}
		return list;
	}

	/**
	 * convert a list of String to a String with format:
	 * <string1>,<string2>,...
	 * 
	 * @param listOfString
	 * @return
	 */
	public static String toString(List<String> listOfString) {
		String result = "";
		if (listOfString.isEmpty()) {
			return result;
		}
		if (listOfString.size() == 1) {
			return listOfString.get(0);
		}
		result += listOfString.get(0);
		for (int i = 1; i < listOfString.size(); i++) {
			result += ","+listOfString.get(i);
		}
		return result;
	}
}

@@ -1432,73 +1432,47 @@
 %7B%0A%09%09
-if (inputString == null) %7B%0A%09%09%09throw new IllegalArgumentException(
+ValidateArgument.required(inputString, 
 %22inp
@@ -1483,20 +1483,16 @@
 ring%22);%0A
-%09%09%7D%0A
 %09%09List%3CS

use util method