X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Fmedia_tag.c;h=1c5fca6b123bf525f0707adc2e60bbea7ff9083b;hb=e8b091cc04a938fd171d3eec72b091150bbb18d7;hp=de495796dc6accafaa6b4cd0693e495af90bc0b2;hpb=0637dbc89b39a9d72434afb341d004a4dcc539c4;p=platform%2Fcore%2Fapi%2Fmedia-content.git

diff --git a/src/media_tag.c b/src/media_tag.c
index de49579..1c5fca6 100755
--- a/src/media_tag.c
+++ b/src/media_tag.c
@@ -17,40 +17,22 @@
 
 #include <media_info_private.h>
 
-static void __media_tag_item_add(media_tag_s *tag_s, media_tag_item_s *item_s);
-static void __media_tag_item_release(media_tag_s *tag_s);
-static int __media_tag_insert_item_to_tag(int tag_id, const char *media_id);
-static int __media_tag_remove_item_from_tag(int tag_id, const char *media_id);
-static int __media_tag_update_tag_name(int tag_id, const char *tag_name);
-static int __media_tag_get_tag_info_from_db(const char *name, media_tag_h tag);
-
-static void __media_tag_item_add(media_tag_s *tag_s, media_tag_item_s *item_s)
+static void __media_tag_item_free(gpointer item)
 {
-	tag_s->item_list = g_list_append(tag_s->item_list, item_s);
+	media_tag_item_s *_item = (media_tag_item_s *)item;
+
+	g_free(_item->media_id);
+	g_free(_item->tag_name);
+	g_free(_item);
 }
 
-static void __media_tag_item_release(media_tag_s *tag_s)
+static void __media_tag_item_release(media_tag_s *tag)
 {
-	int idx = 0;
-	int list_cnt = 0;
-	media_tag_item_s *item = NULL;
-
-	list_cnt = g_list_length(tag_s->item_list);
-
-	media_content_debug("list_cnt : [%d]", list_cnt);
-
-	for (idx = 0; idx < list_cnt; idx++) {
-		item = (media_tag_item_s*)g_list_nth_data(tag_s->item_list, idx);
-		if (item != NULL) {
-			SAFE_FREE(item->media_id);
-			SAFE_FREE(item->tag_name);
-			SAFE_FREE(item);
-		}
-	}
-
-	g_list_free(tag_s->item_list);
-	tag_s->item_list = NULL;
+	if (!tag || !tag->item_list)
+		return;
 
+	g_list_free_full(tag->item_list, __media_tag_item_free);
+	tag->item_list = NULL;
 }
 
 static int __media_tag_insert_item_to_tag(int tag_id, const char *media_id)
@@ -58,10 +40,10 @@ static int __media_tag_insert_item_to_tag(int tag_id, const char *media_id)
 	int ret = MEDIA_CONTENT_ERROR_NONE;
 	char *query_str = NULL;
 
-	query_str = sqlite3_mprintf("INSERT INTO %q (tag_id, media_uuid) values (%d, '%q');", DB_TABLE_TAG_MAP, tag_id, media_id);
+	query_str = sqlite3_mprintf("INSERT OR IGNORE INTO %q (tag_id, media_id) values (%d, '%q');", DB_TABLE_TAG_MAP, tag_id, media_id);
 	ret = media_svc_append_query(query_str, _content_get_uid());
 
-	return _content_error_capi(MEDIA_CONTENT_TYPE, ret);
+	return _content_error_capi(ret);
 }
 
 static int __media_tag_remove_item_from_tag(int tag_id, const char *media_id)
@@ -72,7 +54,7 @@ static int __media_tag_remove_item_from_tag(int tag_id, const char *media_id)
 	query_str = sqlite3_mprintf(REMOVE_TAG_ITEM_FROM_TAG_MAP, tag_id, media_id);
 	ret = media_svc_append_query(query_str, _content_get_uid());
 
-	return _content_error_capi(MEDIA_CONTENT_TYPE, ret);
+	return _content_error_capi(ret);
 }
 
 static int __media_tag_update_tag_name(int tag_id, const char *tag_name)
@@ -83,7 +65,7 @@ static int __media_tag_update_tag_name(int tag_id, const char *tag_name)
 	query_str = sqlite3_mprintf(UPDATE_TAG_NAME_FROM_TAG, tag_name, tag_id);
 	ret = media_svc_append_query(query_str, _content_get_uid());
 
-	return _content_error_capi(MEDIA_CONTENT_TYPE, ret);
+	return _content_error_capi(ret);
 }
 
 static int __media_tag_get_tag_info_from_db(const char *name, media_tag_h tag)
@@ -91,21 +73,21 @@ static int __media_tag_get_tag_info_from_db(const char *name, media_tag_h tag)
 	int ret = MEDIA_CONTENT_ERROR_NONE;
 	sqlite3_stmt *stmt = NULL;
 	char *select_query = NULL;
-	media_tag_s *_tag = (media_tag_s*)tag;
+	media_tag_s *_tag = (media_tag_s *)tag;
 
-	media_content_retvm_if(_tag == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid tag");
+	content_retip_if_fail(tag);
 
 	select_query = sqlite3_mprintf(SELECT_TAG_BY_NAME, name);
 
-	ret = _content_query_prepare(&stmt, select_query, NULL, NULL);
+	ret = _content_get_result(select_query, &stmt);
 	SQLITE3_SAFE_FREE(select_query);
-	media_content_retv_if(ret != MEDIA_CONTENT_ERROR_NONE, ret);
+	content_retv_if(ret != MEDIA_CONTENT_ERROR_NONE, ret);
 
 	if (sqlite3_step(stmt) == SQLITE_ROW) {
-		_tag->tag_id = (int)sqlite3_column_int(stmt, 0);
-		_tag->name = g_strdup((const char *)sqlite3_column_text(stmt, 1));
+		_tag->tag_id = sqlite3_column_int(stmt, 0);
+		_tag->name = g_strdup(name);
 	} else {
-		media_content_error("Nonexistent tag name[%s]", name);
+		content_error("Nonexistent tag name[%s]", name);
 		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
 	}
 
@@ -118,23 +100,20 @@ int media_tag_insert_to_db(const char *tag_name, media_tag_h *tag)
 {
 	int ret = MEDIA_CONTENT_ERROR_NONE;
 	char *query_str = NULL;
+	media_tag_s *_tag = NULL;
 
-	if (!STRING_VALID(tag_name)) {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	content_retip_if_fail(STRING_VALID(tag_name));
+	content_retip_if_fail(tag);
 
 	query_str = sqlite3_mprintf(INSERT_TAG_TO_TAG, tag_name);
 	ret = _content_query_sql(query_str);
 	SQLITE3_SAFE_FREE(query_str);
+	content_retvm_if(ret != MEDIA_CONTENT_ERROR_NONE, ret, "fail to insert tag");
 
-	if (ret == MEDIA_CONTENT_ERROR_NONE) {
-		media_tag_s *_tag = (media_tag_s*)calloc(1, sizeof(media_tag_s));
-		media_content_retvm_if(_tag == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
+	_tag = g_new0(media_tag_s, 1);
 
-		ret = __media_tag_get_tag_info_from_db(tag_name, (media_tag_h)_tag);
-		*tag = (media_tag_h)_tag;
-	}
+	ret = __media_tag_get_tag_info_from_db(tag_name, (media_tag_h)_tag);
+	*tag = (media_tag_h)_tag;
 
 	return ret;
 }
@@ -144,13 +123,9 @@ int media_tag_delete_from_db(int tag_id)
 	int ret = MEDIA_CONTENT_ERROR_NONE;
 	char *query_str = NULL;
 
-	if (tag_id < 0) {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	content_retipm_if_fail(tag_id > 0, "%d", tag_id);
 
 	query_str = sqlite3_mprintf(DELETE_TAG_FROM_TAG, tag_id);
-
 	ret = _content_query_sql(query_str);
 
 	SQLITE3_SAFE_FREE(query_str);
@@ -160,177 +135,113 @@ int media_tag_delete_from_db(int tag_id)
 
 int media_tag_get_tag_count_from_db(filter_h filter, int *tag_count)
 {
-	int ret = MEDIA_CONTENT_ERROR_NONE;
-
-	if (tag_count != NULL) {
-		ret = _media_db_get_group_count(filter, MEDIA_GROUP_TAG, tag_count);
-	} else {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	content_retip_if_fail(tag_count);
 
-	return ret;
+	return _media_db_get_group_count(filter, MEDIA_GROUP_TAG, tag_count);
 }
 
 int media_tag_foreach_tag_from_db(filter_h filter, media_tag_cb callback, void *user_data)
 {
-	int ret = MEDIA_CONTENT_ERROR_NONE;
-
-	if (callback == NULL) {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	content_retip_if_fail(callback);
 
-	ret = _media_db_get_tag(NULL, filter, callback, user_data);
-
-	return ret;
+	return _media_db_get_tag(NULL, filter, callback, user_data);
 }
 
 int media_tag_get_media_count_from_db(int tag_id, filter_h filter, int *media_count)
 {
-	int ret = MEDIA_CONTENT_ERROR_NONE;
+	content_retipm_if_fail(tag_id > 0, "%d", tag_id);
+	content_retip_if_fail(media_count);
 
-	if ((tag_id > 0) && (media_count != NULL)) {
-		ret = _media_db_get_group_item_count_by_id(tag_id, filter, MEDIA_GROUP_TAG, media_count);
-	} else {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
-
-	return ret;
+	return _media_db_get_group_item_count_by_id(tag_id, filter, MEDIA_GROUP_TAG, media_count);
 }
 
 int media_tag_foreach_media_from_db(int tag_id, filter_h filter, media_info_cb callback, void *user_data)
 {
-	int ret = MEDIA_CONTENT_ERROR_NONE;
+	content_retip_if_fail(callback);
 
-	if (callback == NULL) {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
-
-	ret = _media_db_get_group_item_by_id(tag_id, filter, callback, user_data, MEDIA_GROUP_TAG);
-
-	return ret;
+	return _media_db_get_group_item_by_id(tag_id, filter, callback, user_data, MEDIA_GROUP_TAG);
 }
 
 int media_tag_destroy(media_tag_h tag)
 {
-	int ret = MEDIA_CONTENT_ERROR_NONE;
-	media_tag_s *_tag = (media_tag_s*)tag;
-	if (_tag) {
-		SAFE_FREE(_tag->name);
-		SAFE_FREE(_tag);
-		ret = MEDIA_CONTENT_ERROR_NONE;
-	} else {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	media_tag_s *_tag = (media_tag_s *)tag;
 
-	return ret;
+	content_retip_if_fail(tag);
+
+	__media_tag_item_release(_tag);
+
+	g_free(_tag->name);
+	g_free(_tag);
+
+	return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_tag_clone(media_tag_h *dst, media_tag_h src)
 {
-	int ret = MEDIA_CONTENT_ERROR_NONE;
-	media_tag_s *_src = (media_tag_s*)src;
+	media_tag_s *_src = (media_tag_s *)src;
 	media_tag_s *_dst = NULL;
 
-	if ((_src != NULL)) {
-		_dst = (media_tag_s*)calloc(1, sizeof(media_tag_s));
-		media_content_retvm_if(_dst == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
-
-		_dst->tag_id = _src->tag_id;
-
-		if (STRING_VALID(_src->name)) {
-			_dst->name = strdup(_src->name);
-			if (_dst->name == NULL) {
-				SAFE_FREE(_dst);
-				media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-				return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-			}
-		}
-		*dst = (media_tag_h)_dst;
-		ret = MEDIA_CONTENT_ERROR_NONE;
-	} else {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	content_retip_if_fail(src);
+	content_retip_if_fail(dst);
 
-	return ret;
+	_dst = g_new0(media_tag_s, 1);
+
+	_dst->tag_id = _src->tag_id;
+	_dst->name = g_strdup(_src->name);
+
+	*dst = (media_tag_h)_dst;
+
+	return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_tag_get_tag_id(media_tag_h tag, int *tag_id)
 {
-	int ret = MEDIA_CONTENT_ERROR_NONE;
-	media_tag_s *_tag = (media_tag_s*)tag;
+	media_tag_s *_tag = (media_tag_s *)tag;
 
-	if ((_tag != NULL) && (tag_id != NULL)) {
-		*tag_id = _tag->tag_id;
-		ret = MEDIA_CONTENT_ERROR_NONE;
-	} else {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	content_retip_if_fail(tag);
+	content_retip_if_fail(tag_id);
 
-	return ret;
+	*tag_id = _tag->tag_id;
+
+	return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_tag_get_name(media_tag_h tag, char **name)
 {
-	int ret = MEDIA_CONTENT_ERROR_NONE;
-	media_tag_s *_tag = (media_tag_s*)tag;
-	if (_tag) {
-		if (STRING_VALID(_tag->name)) {
-			*name = strdup(_tag->name);
-			media_content_retvm_if(*name == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
-		} else {
-			*name = NULL;
-		}
-		ret = MEDIA_CONTENT_ERROR_NONE;
+	media_tag_s *_tag = (media_tag_s *)tag;
 
-	} else {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	content_retip_if_fail(tag);
+	content_retip_if_fail(name);
 
-	return ret;
+	*name = g_strdup(_tag->name);
+
+	return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_tag_get_tag_from_db(int tag_id, media_tag_h *tag)
 {
 	int ret = MEDIA_CONTENT_ERROR_NONE;
 	sqlite3_stmt *stmt = NULL;
-	char select_query[DEFAULT_QUERY_SIZE] = {0, };
-
-	if (tag_id <= 0) {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
-
-	memset(select_query, 0x00, sizeof(select_query));
-
-	snprintf(select_query, sizeof(select_query), SELECT_TAG_FROM_TAG, tag_id);
+	char *query = NULL;
+	media_tag_s *_tag = NULL;
 
-	ret = _content_query_prepare(&stmt, select_query, NULL, NULL);
-	media_content_retv_if(ret != MEDIA_CONTENT_ERROR_NONE, ret);
+	content_retipm_if_fail(tag_id > 0, "%d", tag_id);
+	content_retip_if_fail(tag);
 
-	media_tag_s *_tag = NULL;
+	query = sqlite3_mprintf(SELECT_TAG_FROM_TAG, tag_id);
+	ret = _content_get_result(query, &stmt);
+	SQLITE3_SAFE_FREE(query);
+	content_retv_if(ret != MEDIA_CONTENT_ERROR_NONE, ret);
 
 	if (sqlite3_step(stmt) == SQLITE_ROW) {
-		_tag = (media_tag_s*)calloc(1, sizeof(media_tag_s));
-		if (_tag == NULL) {
-			SQLITE3_FINALIZE(stmt);
-			media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-			return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-		}
+		_tag = g_new0(media_tag_s, 1);
 
-		_tag->tag_id = (int)sqlite3_column_int(stmt, 0);
-		_tag->name = g_strdup((const char *)sqlite3_column_text(stmt, 1));
+		_tag->tag_id = tag_id;
+		_tag->name = g_strdup((const char *)sqlite3_column_text(stmt, 0));
 
 		*tag = (media_tag_h)_tag;
 	} else {
-		media_content_error("Nonexistent tag id[%d]", tag_id);
+		content_error("Nonexistent tag id[%d]", tag_id);
 		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
 	}
 
@@ -342,26 +253,18 @@ int media_tag_get_tag_from_db(int tag_id, media_tag_h *tag)
 int media_tag_add_media(media_tag_h tag, const char *media_id)
 {
 	int ret = MEDIA_CONTENT_ERROR_NONE;
-	media_tag_s *_tag = (media_tag_s*)tag;
+	media_tag_s *_tag = (media_tag_s *)tag;
+	media_tag_item_s *_item = NULL;
 
-	if ((_tag != NULL) && STRING_VALID(media_id)) {
-		media_tag_item_s *_item = (media_tag_item_s*)calloc(1, sizeof(media_tag_item_s));
-		media_content_retvm_if(_item == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
+	content_retip_if_fail(tag);
+	content_retip_if_fail(STRING_VALID(media_id));
 
-		_item->media_id = strdup(media_id);
-		_item->function = MEDIA_TAG_ADD;
+	_item = g_new0(media_tag_item_s, 1);
 
-		if (_item->media_id == NULL) {
-			SAFE_FREE(_item);
-			media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-			return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-		}
+	_item->media_id = g_strdup(media_id);
+	_item->function = MEDIA_TAG_ADD;
 
-		__media_tag_item_add(_tag, _item);
-	} else {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	_tag->item_list = g_list_append(_tag->item_list, _item);
 
 	return ret;
 }
@@ -369,26 +272,18 @@ int media_tag_add_media(media_tag_h tag, const char *media_id)
 int media_tag_remove_media(media_tag_h tag, const char *media_id)
 {
 	int ret = MEDIA_CONTENT_ERROR_NONE;
-	media_tag_s *_tag = (media_tag_s*)tag;
+	media_tag_s *_tag = (media_tag_s *)tag;
+	media_tag_item_s *_item = NULL;
 
-	if (_tag != NULL && STRING_VALID(media_id)) {
-		media_tag_item_s *_item = (media_tag_item_s*)calloc(1, sizeof(media_tag_item_s));
-		media_content_retvm_if(_item == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
+	content_retip_if_fail(tag);
+	content_retip_if_fail(STRING_VALID(media_id));
 
-		_item->media_id = strdup(media_id);
-		_item->function = MEDIA_TAG_REMOVE;
+	_item = g_new0(media_tag_item_s, 1);
 
-		if (_item->media_id == NULL) {
-			SAFE_FREE(_item);
-			media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-			return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-		}
+	_item->media_id = g_strdup(media_id);
+	_item->function = MEDIA_TAG_REMOVE;
 
-		__media_tag_item_add(_tag, _item);
-	} else {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	_tag->item_list = g_list_append(_tag->item_list, _item);
 
 	return ret;
 }
@@ -396,106 +291,78 @@ int media_tag_remove_media(media_tag_h tag, const char *media_id)
 int media_tag_set_name(media_tag_h tag, const char *tag_name)
 {
 	int ret = MEDIA_CONTENT_ERROR_NONE;
-	media_tag_s *_tag = (media_tag_s*)tag;
+	media_tag_s *_tag = (media_tag_s *)tag;
+	media_tag_item_s *_item = NULL;
 
-	if (_tag != NULL && STRING_VALID(tag_name)) {
-		SAFE_FREE(_tag->name);
+	content_retip_if_fail(tag);
+	content_retip_if_fail(STRING_VALID(tag_name));
 
-		media_tag_item_s *_item = (media_tag_item_s*)calloc(1, sizeof(media_tag_item_s));
-		media_content_retvm_if(_item == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
+	g_free(_tag->name);
 
-		_item->tag_name = strdup(tag_name);
-		_item->function = MEDIA_TAG_UPDATE_TAG_NAME;
+	_item = g_new0(media_tag_item_s, 1);
 
-		if (_item->tag_name == NULL) {
-			SAFE_FREE(_item);
-			media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-			return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-		}
+	_item->tag_name = g_strdup(tag_name);
+	_item->function = MEDIA_TAG_UPDATE_TAG_NAME;
 
-		_tag->name = strdup(tag_name);
-		if (_tag->name == NULL) {
-			SAFE_FREE(_item->tag_name);
-			SAFE_FREE(_item);
-			media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-			return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-		}
+	_tag->name = g_strdup(tag_name);
 
-		__media_tag_item_add(_tag, _item);
-	} else {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	_tag->item_list = g_list_append(_tag->item_list, _item);
 
 	return ret;
 }
 
-int media_tag_update_to_db(media_tag_h tag)
+void __media_tag_foreach_request(gpointer data, gpointer user_data)
 {
 	int ret = MEDIA_CONTENT_ERROR_NONE;
-	media_tag_s *_tag = (media_tag_s*)tag;
-	int idx = 0;
-	int length = 0;
-	media_tag_item_s *_tag_item = NULL;
-
-	if (_tag == NULL) {
-		media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-		return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-	}
+	media_tag_item_s *_tag_item = (media_tag_item_s *)data;
+	int tag_id = GPOINTER_TO_INT(user_data);
 
-	if (_tag->item_list != NULL) {
-		length = g_list_length(_tag->item_list);
-	} else {
-		media_content_debug("operation list length is 0");
-		return MEDIA_CONTENT_ERROR_NONE;
-	}
+	if (!_tag_item)
+		return;
+
+	switch (_tag_item->function) {
+	case MEDIA_TAG_ADD:
+		ret = __media_tag_insert_item_to_tag(tag_id, _tag_item->media_id);
+		break;
+
+	case MEDIA_TAG_REMOVE:
+		ret = __media_tag_remove_item_from_tag(tag_id, _tag_item->media_id);
+		break;
 
-	for (idx = 0; idx < length; idx++) {
-		_tag_item = (media_tag_item_s*)g_list_nth_data(_tag->item_list, idx);
-		if (_tag_item != NULL) {
-			switch (_tag_item->function) {
-			case MEDIA_TAG_ADD:
-			{
-				ret = __media_tag_insert_item_to_tag(_tag->tag_id, _tag_item->media_id);
-			}
-			break;
-
-			case MEDIA_TAG_REMOVE:
-			{
-				ret = __media_tag_remove_item_from_tag(_tag->tag_id, _tag_item->media_id);
-			}
-			break;
-
-			case MEDIA_TAG_UPDATE_TAG_NAME:
-			{
-				ret = __media_tag_update_tag_name(_tag->tag_id, _tag_item->tag_name);
-			}
-			break;
-
-			default:
-			break;
-			}
-
-			if (ret != MEDIA_CONTENT_ERROR_NONE)
-				media_content_error("Failed some operation[%d]", _tag_item->function);
-		}
+	case MEDIA_TAG_UPDATE_TAG_NAME:
+		ret = __media_tag_update_tag_name(tag_id, _tag_item->tag_name);
+		break;
+
+	default:
+		break;
 	}
 
+	if (ret != MEDIA_CONTENT_ERROR_NONE)
+		content_error("Failed some operation[%d]", _tag_item->function);
+}
+
+int media_tag_update_to_db(media_tag_h tag)
+{
+	int ret = MEDIA_CONTENT_ERROR_NONE;
+	media_tag_s *_tag = (media_tag_s *)tag;
+
+	content_retip_if_fail(tag);
+	content_retvm_if(_tag->item_list == NULL, MEDIA_CONTENT_ERROR_NONE, "operation list length is 0");
+
+	g_list_foreach(_tag->item_list, __media_tag_foreach_request, GINT_TO_POINTER(_tag->tag_id));
+
 	ret = media_svc_send_query(_content_get_uid());
 
 	__media_tag_item_release(_tag);
 
-	return _content_error_capi(MEDIA_CONTENT_TYPE, ret);
+	return _content_error_capi(ret);
 }
 
 int media_tag_create(media_tag_h *tag)
 {
-	int ret = MEDIA_CONTENT_ERROR_NONE;
-
-	media_content_retvm_if(tag == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid tag");
+	content_retip_if_fail(tag);
 
-	media_tag_s *_tag = (media_tag_s*)calloc(1, sizeof(media_tag_s));
-	media_content_retvm_if(_tag == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
+	media_tag_s *_tag = g_new0(media_tag_s, 1);
 
 	_tag->tag_id = -1;
 	_tag->name = NULL;
@@ -503,29 +370,29 @@ int media_tag_create(media_tag_h *tag)
 
 	*tag = (media_tag_h)_tag;
 
-	return ret;
+	return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_tag_insert_to_db_v2(media_tag_h tag)
 {
 	int ret = MEDIA_CONTENT_ERROR_NONE;
-	media_tag_s *_tag = (media_tag_s*)tag;
+	media_tag_s *_tag = (media_tag_s *)tag;
 	media_tag_h tag_1;
 	int tag_id = -1;
 
-	media_content_retvm_if(_tag == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid tag");
-	media_content_retvm_if(!STRING_VALID(_tag->name), MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid tag name");
-	media_content_retvm_if(_tag->tag_id != -1, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid tag insert");
+	content_retip_if_fail(tag);
+	content_retip_if_fail(STRING_VALID(_tag->name));
+	content_retip_if_fail(_tag->tag_id == -1);
 
 	ret = media_tag_insert_to_db(_tag->name, &tag_1);
 	if (ret != MEDIA_CONTENT_ERROR_NONE) {
-		media_content_error("fail media_tag_insert_to_db");
+		content_error("fail media_tag_insert_to_db");
 		goto ERROR;
 	}
 
 	ret = media_tag_get_tag_id(tag_1, &tag_id);
 	if (ret != MEDIA_CONTENT_ERROR_NONE) {
-		media_content_error("fail media_tag_get_tag_id");
+		content_error("fail media_tag_get_tag_id");
 		goto ERROR;
 	}
 
@@ -542,10 +409,10 @@ ERROR:
 
 int media_tag_update_to_db_v2(int tag_id, media_tag_h tag)
 {
-	media_tag_s *_tag = (media_tag_s*)tag;
+	media_tag_s *_tag = (media_tag_s *)tag;
 
-	media_content_retvm_if(_tag == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid tag");
-	media_content_retvm_if(tag_id == -1, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "invalid tag update");
+	content_retip_if_fail(tag);
+	content_retip_if_fail(tag_id > 0);
 
 	_tag->tag_id = tag_id;