Folder related functions cleanup
[platform/core/multimedia/libmedia-service.git] / src / common / media-svc.c
index cce795b..55fb6ce 100755 (executable)
 #include "media-svc-noti.h"
 #include "media-svc-storage.h"
 
-static __thread int g_media_svc_item_validity_data_cnt = 1;
-static __thread int g_media_svc_item_validity_cur_data_cnt = 0;
-
-static __thread int g_media_svc_insert_item_data_cnt = 1;
-static __thread int g_media_svc_insert_item_cur_data_cnt = 0;
+//static __thread int g_media_svc_data_cnt = 0;
+static __thread int g_media_svc_cur_data_cnt = 0;
 
 /* Flag for items to be published by notification */
 static __thread bool g_insert_with_noti = false;
 
-#define BATCH_REQUEST_MAX 300
+#define BATCH_ITEM_COUNT_MAX 100
 
 int media_svc_get_user_version(sqlite3 *handle, int *user_version)
 {
@@ -137,18 +134,12 @@ int media_svc_create_table(uid_t uid)
                goto ERROR;
        }
 
-       sql = sqlite3_mprintf("pragma user_version = %d;", LATEST_VERSION_NUMBER);
+       sql = sqlite3_mprintf("pragma user_version = %d;", LATEST_DB_VERSION);
        ret = _media_svc_sql_query(sql, uid);
        if (ret != MS_MEDIA_ERR_NONE) {
                media_svc_error("user_version update fail.");
                goto ERROR;
        }
-
-       _media_svc_destroy_table_query();
-
-       media_svc_debug_fleave();
-
-       return MS_MEDIA_ERR_NONE;
 ERROR:
        _media_svc_destroy_table_query();
 
@@ -159,41 +150,22 @@ ERROR:
 
 int media_svc_check_item_exist_by_path(sqlite3 *handle, const char *storage_id, const char *path)
 {
-       int ret = MS_MEDIA_ERR_NONE;
-       int count = -1;
-
-       ret = _media_svc_count_record_with_path(handle, storage_id, path, &count);
-       media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
-
-       if (count > 0) {
-               media_svc_debug("item is exist in database");
-               return MS_MEDIA_ERR_NONE;
-       } else {
-               media_svc_debug("item is not exist in database");
-               return MS_MEDIA_ERR_DB_NO_RECORD;
-       }
-
-       return MS_MEDIA_ERR_NONE;
+       return _media_svc_check_data_by_path(handle, path);
 }
 
 int media_svc_get_modified_time(sqlite3 *handle, const char *storage_id, const char *path, int *modified_time)
 {
-       return _media_svc_get_modified_time(handle, storage_id, path, modified_time);
+       return _media_svc_get_modified_time(handle, path, modified_time);
 }
 
-int media_svc_insert_item_begin(int data_cnt, bool with_noti, int from_pid)
+int media_svc_insert_item_begin(bool with_noti, int from_pid)
 {
-       media_svc_retvm_if(data_cnt < 1, MS_MEDIA_ERR_INVALID_PARAMETER, "data_cnt shuld be bigger than 1");
-
-       g_media_svc_insert_item_data_cnt = data_cnt;
-       g_media_svc_insert_item_cur_data_cnt = 0;
+       g_media_svc_cur_data_cnt = 0;
 
        /* Prepare for making noti item list */
        if (with_noti) {
                media_svc_debug("making noti list from pid[%d]", from_pid);
-               if (_media_svc_create_noti_list(data_cnt) != MS_MEDIA_ERR_NONE)
-                       return MS_MEDIA_ERR_OUT_OF_MEMORY;
-
+               _media_svc_initialize_noti_list();
                _media_svc_set_noti_from_pid(from_pid);
                g_insert_with_noti = true;
        }
@@ -207,20 +179,17 @@ int media_svc_insert_item_end(uid_t uid)
 
        media_svc_debug_fenter();
 
-       if (g_media_svc_insert_item_cur_data_cnt > 0) {
-
-               ret = _media_svc_list_query_do(MEDIA_SVC_QUERY_INSERT_ITEM, uid);
+       if (g_media_svc_cur_data_cnt > 0) {
+               ret = _media_svc_list_query_do(MEDIA_SVC_QUERY_SCANNER, uid);
                if (g_insert_with_noti) {
                        media_svc_debug("sending noti list");
-                       _media_svc_publish_noti_list(g_media_svc_insert_item_cur_data_cnt);
-                       _media_svc_destroy_noti_list(g_media_svc_insert_item_cur_data_cnt);
+                       _media_svc_publish_noti_list();
                        g_insert_with_noti = false;
                        _media_svc_set_noti_from_pid(-1);
                }
        }
 
-       g_media_svc_insert_item_data_cnt = 1;
-       g_media_svc_insert_item_cur_data_cnt = 0;
+       g_media_svc_cur_data_cnt = 0;
 
        return ret;
 }
@@ -254,7 +223,8 @@ int media_svc_insert_item_bulk(sqlite3 *handle, const char *storage_id, ms_user_
                ret = _media_svc_extract_media_metadata(handle, true, &content_info, uid);
                break;
        default:
-               media_svc_debug("Do nothing[%d]", content_info.media_type);
+               /* The 'TITLE' should always be filled in */
+               content_info.media_meta.title = _media_svc_get_title_by_path(content_info.path);
                break;
        }
 
@@ -267,50 +237,21 @@ int media_svc_insert_item_bulk(sqlite3 *handle, const char *storage_id, ms_user_
        content_info.folder_uuid = g_strdup(folder_uuid);
        media_svc_retv_del_if(content_info.folder_uuid == NULL, MS_MEDIA_ERR_INTERNAL, &content_info);
 
-       if (g_media_svc_insert_item_data_cnt == 1) {
-
-               ret = _media_svc_insert_item_with_data(true, storage_id, &content_info, false, uid);
-               media_svc_retv_del_if(ret != MS_MEDIA_ERR_NONE, ret, &content_info);
-
-               if (g_insert_with_noti)
-                       _media_svc_insert_item_to_noti_list(&content_info, g_media_svc_insert_item_cur_data_cnt++);
-
-       } else if (g_media_svc_insert_item_cur_data_cnt < (g_media_svc_insert_item_data_cnt - 1)) {
-
-               ret = _media_svc_insert_item_with_data(true, storage_id, &content_info, true, uid);
-               media_svc_retv_del_if(ret != MS_MEDIA_ERR_NONE, ret, &content_info);
-
-               if (g_insert_with_noti)
-                       _media_svc_insert_item_to_noti_list(&content_info, g_media_svc_insert_item_cur_data_cnt);
-
-               g_media_svc_insert_item_cur_data_cnt++;
+       ret = _media_svc_insert_item_with_data(true, &content_info, true, uid);
+       media_svc_retv_del_if(ret != MS_MEDIA_ERR_NONE, ret, &content_info);
 
-       } else if (g_media_svc_insert_item_cur_data_cnt == (g_media_svc_insert_item_data_cnt - 1)) {
+       if (g_insert_with_noti)
+               _media_svc_insert_item_to_noti_list(&content_info);
 
-               ret = _media_svc_insert_item_with_data(true, storage_id, &content_info, true, uid);
+       /* To avoid over-occupying memory, update per BATCH_ITEM_COUNT_MAX. */
+       if (++g_media_svc_cur_data_cnt == BATCH_ITEM_COUNT_MAX) {
+               ret = _media_svc_list_query_do(MEDIA_SVC_QUERY_SCANNER, uid);
                media_svc_retv_del_if(ret != MS_MEDIA_ERR_NONE, ret, &content_info);
 
                if (g_insert_with_noti)
-                       _media_svc_insert_item_to_noti_list(&content_info, g_media_svc_insert_item_cur_data_cnt);
-
-               ret = _media_svc_list_query_do(MEDIA_SVC_QUERY_INSERT_ITEM, uid);
-               media_svc_retv_del_if(ret != MS_MEDIA_ERR_NONE, ret, &content_info);
-
-               if (g_insert_with_noti) {
-                       _media_svc_publish_noti_list(g_media_svc_insert_item_cur_data_cnt + 1);
-                       _media_svc_destroy_noti_list(g_media_svc_insert_item_cur_data_cnt + 1);
+                       _media_svc_publish_noti_list();
 
-                       /* Recreate noti list */
-                       ret = _media_svc_create_noti_list(g_media_svc_insert_item_data_cnt);
-                       media_svc_retv_del_if(ret != MS_MEDIA_ERR_NONE, ret, &content_info);
-               }
-
-               g_media_svc_insert_item_cur_data_cnt = 0;
-
-       } else {
-               media_svc_error("Error in media_svc_insert_item_bulk");
-               _media_svc_destroy_content_info(&content_info);
-               return MS_MEDIA_ERR_INTERNAL;
+               g_media_svc_cur_data_cnt = 0;
        }
 
        _media_svc_destroy_content_info(&content_info);
@@ -346,7 +287,8 @@ int media_svc_insert_item_immediately(sqlite3 *handle, const char *storage_id, m
                ret = _media_svc_extract_media_metadata(handle, false, &content_info, uid);
                break;
        default:
-               media_svc_debug("Do nothing[%d]", content_info.media_type);
+               /* The 'TITLE' should always be filled in */
+               content_info.media_meta.title = _media_svc_get_title_by_path(content_info.path);
                break;
        }
 
@@ -370,7 +312,7 @@ int media_svc_insert_item_immediately(sqlite3 *handle, const char *storage_id, m
                }
        }
 
-       ret = _media_svc_insert_item_with_data(false, storage_id, &content_info, false, uid);
+       ret = _media_svc_insert_item_with_data(false, &content_info, false, uid);
 
        if (ret == MS_MEDIA_ERR_NONE) {
                media_svc_debug("Insertion is successful. Sending noti for this");
@@ -397,7 +339,6 @@ int media_svc_move_item(sqlite3 *handle,
        int modified_time = 0;
        char folder_uuid[MEDIA_SVC_UUID_SIZE + 1] = {0, };
        char old_thumb_path[MEDIA_SVC_PATHNAME_SIZE] = {0, };
-       char org_stg_id[MEDIA_SVC_UUID_SIZE + 1] = {0, };
        char dst_stg_id[MEDIA_SVC_UUID_SIZE + 1] = {0, };
        ms_user_storage_type_e org_stg_type = MS_USER_STORAGE_INTERNAL;
        ms_user_storage_type_e dst_stg_type = MS_USER_STORAGE_INTERNAL;
@@ -411,8 +352,6 @@ int media_svc_move_item(sqlite3 *handle,
        media_svc_retvm_if(!STRING_VALID(mime_type), MS_MEDIA_ERR_INVALID_PARAMETER, "mime_type is NULL");
 
        /* Get storage_id */
-       ret = _media_svc_get_storage_uuid(handle, src_path, org_stg_id, uid);
-       media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
        ret = _media_svc_get_storage_uuid(handle, dest_path, dst_stg_id, uid);
        media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
        /* Get storage_type */
@@ -439,14 +378,10 @@ int media_svc_move_item(sqlite3 *handle,
                return ret;
        }
 
-       if (STRING_VALID(old_thumb_path)) {
-               ret = _media_svc_remove_file(old_thumb_path);
-               if (ret != MS_MEDIA_ERR_NONE)
-                       media_svc_error("_media_svc_remove_file failed : %d", ret);
-       }
+       _media_svc_remove_file(old_thumb_path);
 
        /*move item*/
-       ret = _media_svc_update_item_by_path(org_stg_id, src_path, dst_stg_id, dst_stg_type, dest_path, file_name, modified_time, folder_uuid, uid);
+       ret = _media_svc_update_item_by_path(src_path, dst_stg_id, dst_stg_type, dest_path, file_name, modified_time, folder_uuid, uid);
        SAFE_FREE(file_name);
        media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
 
@@ -462,65 +397,22 @@ int media_svc_move_item(sqlite3 *handle,
        return MS_MEDIA_ERR_NONE;
 }
 
-int media_svc_set_item_validity_begin(int data_cnt)
-{
-       media_svc_debug("Transaction data count : [%d]", data_cnt);
-
-       media_svc_retvm_if(data_cnt < 1, MS_MEDIA_ERR_INVALID_PARAMETER, "data_cnt shuld be bigger than 1");
-
-       g_media_svc_item_validity_data_cnt = data_cnt;
-       g_media_svc_item_validity_cur_data_cnt = 0;
-
-       return MS_MEDIA_ERR_NONE;
-}
-
-int media_svc_set_item_validity_end(uid_t uid)
-{
-       int ret = MS_MEDIA_ERR_NONE;
-
-       media_svc_debug_fenter();
-
-       if (g_media_svc_item_validity_cur_data_cnt > 0)
-               ret = _media_svc_list_query_do(MEDIA_SVC_QUERY_SET_ITEM_VALIDITY, uid);
-
-       g_media_svc_item_validity_data_cnt = 1;
-       g_media_svc_item_validity_cur_data_cnt = 0;
-
-       return ret;
-}
-
-int media_svc_set_item_validity(const char *storage_id, const char *path, int validity, uid_t uid)
+int media_svc_set_item_validity(const char *path, int validity, uid_t uid)
 {
        int ret = MS_MEDIA_ERR_NONE;
 
-       if (g_media_svc_item_validity_data_cnt == 1) {
-
-               return _media_svc_update_item_validity(storage_id, path, validity, false, uid);
-
-       } else if (g_media_svc_item_validity_cur_data_cnt < (g_media_svc_item_validity_data_cnt - 1)) {
-
-               ret = _media_svc_update_item_validity(storage_id, path, validity, true, uid);
-               media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
-
-               g_media_svc_item_validity_cur_data_cnt++;
-
-       } else if (g_media_svc_item_validity_cur_data_cnt == (g_media_svc_item_validity_data_cnt - 1)) {
-
-               ret = _media_svc_update_item_validity(storage_id, path, validity, true, uid);
-               media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
+       ret = _media_svc_update_item_validity(path, validity, true, uid);
+       media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
 
-               ret = _media_svc_list_query_do(MEDIA_SVC_QUERY_SET_ITEM_VALIDITY, uid);
+       /* To avoid over-occupying memory, update per BATCH_ITEM_COUNT_MAX. */
+       if (++g_media_svc_cur_data_cnt == BATCH_ITEM_COUNT_MAX) {
+               ret = _media_svc_list_query_do(MEDIA_SVC_QUERY_SCANNER, uid);
                media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
 
-               g_media_svc_item_validity_cur_data_cnt = 0;
-
-       } else {
-
-               media_svc_error("Error in media_svc_set_item_validity");
-               return MS_MEDIA_ERR_INTERNAL;
+               g_media_svc_cur_data_cnt = 0;
        }
 
-       return MS_MEDIA_ERR_NONE;
+       return ret;
 }
 
 int media_svc_delete_item_by_path(sqlite3 *handle, const char *storage_id, const char *path, uid_t uid)
@@ -536,11 +428,11 @@ int media_svc_delete_item_by_path(sqlite3 *handle, const char *storage_id, const
        media_svc_retv_if((ret != MS_MEDIA_ERR_NONE) && (ret != MS_MEDIA_ERR_DB_NO_RECORD), ret);
 
        /* Get notification info */
-       ret = _media_svc_get_noti_info(handle, storage_id, path, &noti_item);
+       ret = _media_svc_get_noti_info(handle, path, &noti_item);
        media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
 
        /*Delete item*/
-       ret = _media_svc_delete_item_by_path(storage_id, path, uid);
+       ret = _media_svc_delete_item_by_path(path, uid);
        if (ret != MS_MEDIA_ERR_NONE) {
                media_svc_error("_media_svc_delete_item_by_path failed : %d", ret);
                _media_svc_destroy_noti_item(noti_item);
@@ -554,11 +446,7 @@ int media_svc_delete_item_by_path(sqlite3 *handle, const char *storage_id, const
        _media_svc_destroy_noti_item(noti_item);
 
        /*Delete thumbnail*/
-       if (STRING_VALID(thumb_path)) {
-               ret = _media_svc_remove_file(thumb_path);
-               if (ret != MS_MEDIA_ERR_NONE)
-                       media_svc_error("fail to remove thumbnail file.");
-       }
+       _media_svc_remove_file(thumb_path);
 
        return MS_MEDIA_ERR_NONE;
 }
@@ -589,13 +477,9 @@ int media_svc_refresh_item(sqlite3 *handle, bool is_direct, const char *storage_
        }
 
        if (STRING_VALID(thumb_path)) {
-               if (g_file_test(thumb_path, G_FILE_TEST_EXISTS)) {
-                       ret = _media_svc_remove_file(thumb_path);
-                       if (ret != MS_MEDIA_ERR_NONE)
-                               media_svc_error("_media_svc_remove_file failed : %s", thumb_path);
-               }
+               _media_svc_remove_file(thumb_path);
 
-               ret = _media_svc_update_thumbnail_path(storage_id, path, NULL, uid);
+               ret = _media_svc_update_thumbnail_path(path, NULL, uid);
                if (ret != MS_MEDIA_ERR_NONE) {
                        _media_svc_destroy_content_info(&content_info);
                        return ret;
@@ -604,23 +488,29 @@ int media_svc_refresh_item(sqlite3 *handle, bool is_direct, const char *storage_
 
        /* Get notification info */
        media_svc_noti_item *noti_item = NULL;
-       ret = _media_svc_get_noti_info(handle, storage_id, path, &noti_item);
+       ret = _media_svc_get_noti_info(handle, path, &noti_item);
        if (ret != MS_MEDIA_ERR_NONE) {
                _media_svc_destroy_content_info(&content_info);
                return ret;
        }
 
        content_info.media_type = noti_item->media_type;
+       content_info.mime_type = g_strdup(noti_item->mime_type);
 
-       if (content_info.media_type == MEDIA_SVC_MEDIA_TYPE_OTHER
-       || (content_info.media_type == MEDIA_SVC_MEDIA_TYPE_PVR)
-       || (content_info.media_type == MEDIA_SVC_MEDIA_TYPE_UHD)
-       || (content_info.media_type == MEDIA_SVC_MEDIA_TYPE_SCSA))
-               media_svc_debug("Do nothing [%d]", content_info.media_type);
-       else if (content_info.media_type == MEDIA_SVC_MEDIA_TYPE_IMAGE)
+       switch (content_info.media_type) {
+       case MEDIA_SVC_MEDIA_TYPE_IMAGE:
                ret = _media_svc_extract_image_metadata(&content_info);
-       else
+               break;
+       case MEDIA_SVC_MEDIA_TYPE_VIDEO:
+       case MEDIA_SVC_MEDIA_TYPE_SOUND:
+       case MEDIA_SVC_MEDIA_TYPE_MUSIC:
                ret = _media_svc_extract_media_metadata(handle, is_direct, &content_info, uid);
+               break;
+       default:
+               /* The 'TITLE' should always be filled in */
+               content_info.media_meta.title = _media_svc_get_title_by_path(content_info.path);
+               break;
+       }
 
        if (ret != MS_MEDIA_ERR_NONE) {
                _media_svc_destroy_noti_item(noti_item);
@@ -639,7 +529,7 @@ int media_svc_refresh_item(sqlite3 *handle, bool is_direct, const char *storage_
                }
        }
 
-       ret = _media_svc_update_item_with_data(is_direct, storage_id, &content_info, uid);
+       ret = _media_svc_update_item_with_data(is_direct, &content_info, uid);
 
        if (ret == MS_MEDIA_ERR_NONE) {
                media_svc_debug("Update is successful. Sending noti for this");
@@ -675,7 +565,9 @@ static void __media_svc_noti_all_storage(sqlite3 *handle, uid_t uid)
        GPtrArray *path_list = NULL;
        int i = 0;
 
-       ms_user_get_internal_root_path(uid, &root_path);
+       ret = ms_user_get_internal_root_path(uid, &root_path);
+       media_svc_retm_if(ret != MS_MEDIA_ERR_NONE, "Fail to get root path");
+
        ret = _media_svc_publish_dir_noti(MS_MEDIA_ITEM_UPDATE, root_path, NULL, 0);
        if (ret != MS_MEDIA_ERR_NONE)
                media_svc_error("Fail to send noti");
@@ -696,38 +588,31 @@ static void __media_svc_noti_all_storage(sqlite3 *handle, uid_t uid)
        g_ptr_array_free(path_list, TRUE);
 }
 
-static void __file_info_free(gpointer data)
-{
-       SAFE_FREE(*(media_svc_file_s **)data);
-}
-
 int media_svc_update_item_meta(sqlite3 *handle, uid_t uid)
 {
        int ret = MS_MEDIA_ERR_NONE;
        int i = 0;
-       /* NOTICE : After 6.0, change 'media_view' to 'media', add 'AND validity=1', and no need storage id */
        char *sql = NULL;
+       char *file_path = NULL;
        media_svc_content_info_s content_info;
-       media_svc_file_s *file_info = NULL;
-       GArray *path_list = NULL;
+       GPtrArray *path_list = NULL;
 
-       path_list = g_array_new(FALSE, FALSE, sizeof(media_svc_file_s *));
+       path_list = g_ptr_array_new_with_free_func(g_free);
        media_svc_retvm_if(!path_list, MS_MEDIA_ERR_OUT_OF_MEMORY, "Allocation failed");
-       g_array_set_clear_func(path_list, __file_info_free);
 
-       sql = sqlite3_mprintf("SELECT media_path, storage_uuid FROM %q WHERE media_type=3", MEDIA_SVC_DB_VIEW_MEDIA);
+       sql = sqlite3_mprintf("SELECT media_path FROM %q WHERE media_type=3 AND validity=1", MEDIA_SVC_DB_TABLE_MEDIA);
        ret = _media_svc_get_media(handle, sql, &path_list);
        if (ret != MS_MEDIA_ERR_NONE) {
                media_svc_error("Fail to get media list");
-               g_array_free(path_list, TRUE);
+               g_ptr_array_free(path_list, TRUE);
                return ret;
        }
 
        for (i = 0; i < path_list->len; i++) {
-               file_info = g_array_index(path_list, media_svc_file_s *, i);
+               file_path = g_ptr_array_index(path_list, i);
 
                memset(&content_info, 0, sizeof(media_svc_content_info_s));
-               ret = _media_svc_extract_music_metadata_for_update(&content_info, file_info->storage_id, file_info->path);
+               ret = _media_svc_extract_music_metadata_for_update(&content_info, file_path);
                if (ret != MS_MEDIA_ERR_NONE) {
                        media_svc_error("Fail to extract metadata");
                        _media_svc_destroy_content_info(&content_info);
@@ -741,9 +626,9 @@ int media_svc_update_item_meta(sqlite3 *handle, uid_t uid)
                _media_svc_destroy_content_info(&content_info);
        }
 
-       g_array_free(path_list, TRUE);
+       g_ptr_array_free(path_list, TRUE);
 
-       ret = _media_svc_list_query_do(MEDIA_SVC_QUERY_UPDATE_ITEM, uid);
+       ret = _media_svc_list_query_do(MEDIA_SVC_QUERY_SCANNER, uid);
        media_svc_retvm_if(ret != MS_MEDIA_ERR_NONE, ret, "_media_svc_list_query_do failed");
 
        /* Noti for this */
@@ -778,9 +663,6 @@ int media_svc_set_storage_validity(sqlite3 *handle, const char *storage_id, int
        ret = _media_svc_update_storage_validity(storage_id, validity, uid);
        media_svc_retvm_if(ret != MS_MEDIA_ERR_NONE, ret, "update storage validity failed: %d", ret);
 
-       ret = _media_svc_update_media_view(handle, uid);
-       media_svc_retvm_if(ret != MS_MEDIA_ERR_NONE, ret, "update media view failed : %d", ret);
-
        return ret;
 }
 
@@ -800,9 +682,9 @@ int media_svc_generate_uuid(char **uuid)
        return MS_MEDIA_ERR_NONE;
 }
 
-int media_svc_check_storage(sqlite3 *handle, const char *storage_id, char **storage_path, int *validity, uid_t uid)
+int media_svc_check_storage(sqlite3 *handle, const char *storage_id, char **storage_path, int *validity)
 {
-       return _media_svc_check_storage(handle, storage_id, storage_path, validity, uid);
+       return _media_svc_check_storage(handle, storage_id, storage_path, validity);
 }
 
 int media_svc_update_storage(sqlite3 *handle, const char *storage_id, const char *storage_path, uid_t uid)
@@ -822,12 +704,6 @@ int media_svc_insert_storage(sqlite3 *handle, const char *storage_id, const char
        ret = _media_svc_append_storage(storage_id, storage_path, storage_type, uid);
        media_svc_retvm_if(ret != MS_MEDIA_ERR_NONE, ret, "append storage failed : %d", ret);
 
-       ret = _media_svc_create_media_table_with_id(storage_id, uid);
-       media_svc_retvm_if(ret != MS_MEDIA_ERR_NONE, ret, "create media table failed : %d", ret);
-
-       ret = _media_svc_update_media_view(handle, uid);
-       media_svc_retvm_if(ret != MS_MEDIA_ERR_NONE, ret, "update media view failed : %d", ret);
-
        /* Remove external storage that validity is 0 */
        ret = _media_svc_delete_invalid_storage(handle, uid);
        media_svc_retvm_if(ret != MS_MEDIA_ERR_NONE, ret, "Delete invalid storage failed : %d", ret);
@@ -852,21 +728,7 @@ int media_svc_set_folder_validity(sqlite3 *handle, const char *storage_id, const
 
 int media_svc_check_folder_exist_by_path(sqlite3 *handle, const char *storage_id, const char *folder_path)
 {
-       int ret = MS_MEDIA_ERR_NONE;
-       int count = -1;
-
-       ret = _media_svc_count_folder_with_path(handle, storage_id, folder_path, &count);
-       media_svc_retv_if(ret != MS_MEDIA_ERR_NONE, ret);
-
-       if (count > 0) {
-               media_svc_debug("item is exist in database");
-               return MS_MEDIA_ERR_NONE;
-       } else {
-               media_svc_debug("item is not exist in database");
-               return MS_MEDIA_ERR_DB_NO_RECORD;
-       }
-
-       return MS_MEDIA_ERR_NONE;
+       return _media_svc_check_folder_by_path(handle, storage_id, folder_path);
 }
 
 int media_svc_append_query(const char *query, uid_t uid)
@@ -884,7 +746,7 @@ int media_svc_get_media_type(const char *path, int *mediatype)
        return _media_svc_get_media_type(path, mediatype);
 }
 
-int media_svc_create_thumbnail(const char *storage_id, const char *file_path, int media_type, uid_t uid, char **thumbnail_path)
+int media_svc_create_thumbnail(const char *file_path, int media_type, uid_t uid, char **thumbnail_path)
 {
        int ret = MS_MEDIA_ERR_NONE;
        char thumb_path[MEDIA_SVC_PATHNAME_SIZE + 1] = { 0, };
@@ -903,7 +765,7 @@ int media_svc_create_thumbnail(const char *storage_id, const char *file_path, in
        }
 
        // 3. Update creation result to media db
-       sql = sqlite3_mprintf("UPDATE '%q' SET media_thumbnail_path='%q' WHERE media_path='%q';", storage_id, thumb_path, file_path);
+       sql = sqlite3_mprintf("UPDATE %q SET media_thumbnail_path='%q' WHERE media_path='%q';", MEDIA_SVC_DB_TABLE_MEDIA, thumb_path, file_path);
 
        ret = _media_svc_sql_query(sql, uid);
        SQLITE3_SAFE_FREE(sql);