[ACR-1811] Deprecate underutilized fields

[platform/core/api/media-content.git] / src / media_uhd.c

diff --git a/src/media_uhd.c b/src/media_uhd.c
index d326be8..b9d7858 100755 (executable)
--- a/src/media_uhd.c
+++ b/src/media_uhd.c
@@ -20,53 +20,34 @@
 
 int media_uhd_get_media_count_from_db(filter_h filter, int *media_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       if (media_count) {
-               ret = _media_db_get_group_item_count(NULL, filter, MEDIA_GROUP_UHD, media_count);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media_count);
 
-       return ret;
+       return _media_db_get_group_item_count(NULL, filter, MEDIA_GROUP_UHD, media_count);
 }
 
 int media_uhd_foreach_media_from_db(filter_h filter, media_uhd_cb callback, void *user_data)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
+       content_retip_if_fail(callback);
 
-       if (callback != NULL) {
-               ret = _media_db_get_uhd(filter, callback, user_data);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
-
-       return ret;
+       return _media_db_get_uhd(filter, callback, user_data);
 }
 
 int media_uhd_destroy(media_uhd_h uhd)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
-       if (_uhd) {
-               g_free(_uhd->media_id);
-               g_free(_uhd->storage_id);
-               g_free(_uhd->path);
-               g_free(_uhd->content_id);
-               g_free(_uhd->content_title);
-               g_free(_uhd->file_name);
-               g_free(_uhd->release_date);
-               g_free(_uhd);
-
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
 
-       return ret;
+       content_retip_if_fail(uhd);
+
+       g_free(_uhd->media_id);
+       g_free(_uhd->storage_id);
+       g_free(_uhd->path);
+       g_free(_uhd->content_id);
+       g_free(_uhd->content_title);
+       g_free(_uhd->file_name);
+       g_free(_uhd->release_date);
+       g_free(_uhd);
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_uhd_clone(media_uhd_h *dst, media_uhd_h src)
@@ -102,10 +83,8 @@ int media_uhd_get_uhd_from_db(const char *media_id, media_uhd_h *uhd)
        char *select_query = NULL;
        sqlite3_stmt *stmt = NULL;
 
-       if (!STRING_VALID(media_id) || (uhd == NULL)) {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
+       content_retip_if_fail(uhd);
 
        select_query = sqlite3_mprintf(SELECT_UHD_FROM_UHD, media_id);
 
@@ -169,18 +148,14 @@ int media_uhd_get_path(media_uhd_h uhd, char **path)
 
 int media_uhd_get_size(media_uhd_h uhd, unsigned long long *size)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
 
-       if (_uhd && size) {
-               *size = _uhd->size;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(uhd);
+       content_retip_if_fail(size);
 
-       return ret;
+       *size = _uhd->size;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_uhd_get_content_id(media_uhd_h uhd, char **content_id)
@@ -233,81 +208,62 @@ int media_uhd_get_release_date(media_uhd_h uhd, char **release_date)
 
 int media_uhd_get_modified_time(media_uhd_h uhd, time_t *modified_time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
 
-       if (_uhd && modified_time) {
-               *modified_time = _uhd->modified_time;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(uhd);
+       content_retip_if_fail(modified_time);
 
-       return ret;
+       *modified_time = _uhd->modified_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_uhd_get_played_position(media_uhd_h uhd, int *played_position)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
 
-       if (_uhd) {
-               *played_position = _uhd->played_position;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(uhd);
+       content_retip_if_fail(played_position);
 
-       return ret;
+       *played_position = _uhd->played_position;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_uhd_get_sub_type(media_uhd_h uhd, int *sub_type)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
 
-       if (_uhd) {
-               *sub_type = _uhd->sub_type;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(uhd);
+       content_retip_if_fail(sub_type);
 
-       return ret;
+       *sub_type = _uhd->sub_type;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_uhd_get_played_count(media_uhd_h uhd, int *played_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
 
-       if (_uhd) {
-               *played_count = _uhd->played_count;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(uhd);
+       content_retip_if_fail(played_count);
 
-       return ret;
+       *played_count = _uhd->played_count;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_uhd_set_played_position(media_uhd_h uhd, int played_position)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
 
-       if ((_uhd != NULL) && (played_position >= 0)) {
-               _uhd->played_position = played_position;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(uhd);
+       content_retip_if_fail(played_position);
 
-       return ret;
+       _uhd->played_position = played_position;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_uhd_set_content_title(media_uhd_h uhd, const char *content_title)
@@ -338,32 +294,25 @@ int media_uhd_set_release_date(media_uhd_h uhd, const char *release_date)
 
 int media_uhd_set_sub_type(media_uhd_h uhd, int sub_type)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
 
-       if ((_uhd != NULL) && (sub_type >= 0)) {
-               _uhd->sub_type = sub_type;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(uhd);
+       content_retip_if_fail(sub_type >= 0);
 
-       return ret;
+       _uhd->sub_type = sub_type;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_uhd_set_played_count(media_uhd_h uhd, int played_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
 
-       if (_uhd != NULL) {
-               _uhd->played_count = played_count;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(uhd);
 
-       return ret;
+       _uhd->played_count = played_count;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_uhd_update_to_db(media_uhd_h uhd)
@@ -372,15 +321,14 @@ int media_uhd_update_to_db(media_uhd_h uhd)
        media_uhd_s *_uhd = (media_uhd_s*)uhd;
        char *update_query = NULL;
 
-       if (_uhd != NULL && STRING_VALID(_uhd->media_id) && STRING_VALID(_uhd->path)) {
-               update_query = sqlite3_mprintf(UPDATE_UHD_META_FROM_UHD, _uhd->content_title, _uhd->release_date, _uhd->played_position, _uhd->sub_type, _uhd->played_count, _uhd->media_id);
+       content_retip_if_fail(uhd);
+       content_retip_if_fail(STRING_VALID(_uhd->media_id));
+       content_retip_if_fail(STRING_VALID(_uhd->path));
 
-               ret = _content_query_sql(update_query);
-               sqlite3_free(update_query);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       update_query = sqlite3_mprintf(UPDATE_UHD_META_FROM_UHD, _uhd->content_title, _uhd->release_date, _uhd->played_position, _uhd->sub_type, _uhd->played_count, _uhd->media_id);
+
+       ret = _content_query_sql(update_query);
+       sqlite3_free(update_query);
 
        return ret;
 }