From: Andrzej Popowski <a.popowski@samsung.com>
Date: Mon, 19 Oct 2015 09:34:35 +0000 (+0200)
Subject: [Content] - Fix procedure for calling error callback
X-Git-Tag: submit/tizen/20151026.073646^2^2~6^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=13099bcc46bfae8723999fcba12cf280b1431d06;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[Content] - Fix procedure for calling error callback

Change-Id: I911997be7db55fe14a9a74fed864ccf2ddb2cc93
Signed-off-by: Andrzej Popowski <a.popowski@samsung.com>
---

diff --git a/src/content/content.gyp b/src/content/content.gyp
index b732b7b5..9cc2f8c3 100755
--- a/src/content/content.gyp
+++ b/src/content/content.gyp
@@ -26,7 +26,7 @@
             'packages': [
               'capi-content-media-content',
               'capi-media-metadata-extractor',
-				      'capi-base-common',
+              'capi-base-common',
               'dlog',
             ]
           },
diff --git a/src/content/content_instance.cc b/src/content/content_instance.cc
index 2bed6d7d..bb2a3608 100755
--- a/src/content/content_instance.cc
+++ b/src/content/content_instance.cc
@@ -109,10 +109,15 @@ static gboolean CompletedCallback(const std::shared_ptr<ReplyCallbackData>& user
 static void* WorkThread(const std::shared_ptr<ReplyCallbackData>& user_data) {
   LoggerD("entered");
 
+  int ret = MEDIA_CONTENT_ERROR_NONE;
   ContentCallbacks cbType = user_data->cbType;
   switch(cbType) {
     case ContentManagerUpdatebatchCallback: {
-      ContentManager::getInstance()->updateBatch(user_data->args);
+      ret = ContentManager::getInstance()->updateBatch(user_data->args);
+      if(ret != MEDIA_CONTENT_ERROR_NONE){
+        LoggerD("UpdateBatch Failed");
+        user_data->isSuccess = ContentManager::getInstance()->convertError(ret);
+      }
       break;
     }
     case ContentManagerGetdirectoriesCallback: {
@@ -126,9 +131,9 @@ static void* WorkThread(const std::shared_ptr<ReplyCallbackData>& user_data) {
     case ContentManagerScanfileCallback: {
       std::string contentURI = user_data->args.get("contentURI").get<std::string>();
       std::string real_path = common::VirtualFs::GetInstance().GetRealPath(contentURI);
-      int res = ContentManager::getInstance()->scanFile(real_path);
-      if (res != MEDIA_CONTENT_ERROR_NONE) {
-        LOGGER(ERROR) << "Scan file failed, error: " << res;
+      ret = ContentManager::getInstance()->scanFile(real_path);
+      if (ret != MEDIA_CONTENT_ERROR_NONE) {
+        LOGGER(ERROR) << "Scan file failed, error: " << ret;
         common::PlatformResult err(common::ErrorCode::UNKNOWN_ERR, "Scan file failed.");
         user_data->isSuccess = err;
       }
diff --git a/src/content/content_manager.cc b/src/content/content_manager.cc
index 6e1f0f2f..8e092bb8 100755
--- a/src/content/content_manager.cc
+++ b/src/content/content_manager.cc
@@ -895,7 +895,7 @@ PlatformResult ContentManager::unSetV2ChangeListener(media_content_noti_h* noti_
 void ContentManager::createPlaylist(std::string name,
   const std::shared_ptr<ReplyCallbackData>& user_data) {
   LoggerD("Enter");
-  media_playlist_h	playlist = NULL;
+  media_playlist_h playlist = NULL;
 
   int ret = media_playlist_insert_to_db(name.c_str(),&playlist);
   std::unique_ptr<std::remove_pointer<media_playlist_h>::type, int(*)(media_playlist_h)>
@@ -961,7 +961,7 @@ void ContentManager::getPlaylists(const std::shared_ptr<ReplyCallbackData>& user
 
   LoggerD("Enter");
   int ret;
-  filter_h 	filter = nullptr;
+  filter_h filter = nullptr;
   media_filter_create(&filter);
   std::unique_ptr<std::remove_pointer<filter_h>::type, int(*)(filter_h)>
       filter_ptr(filter, &media_filter_destroy); // automatically release the memory
@@ -1027,8 +1027,16 @@ int ContentManager::updateBatch(picojson::value args) {
     media_info_h media = NULL;
     ret = media_info_get_media_from_db(id.c_str(), &media);
     if (media != NULL && ret == MEDIA_CONTENT_ERROR_NONE) {
-      setContent(media, content);
+      ret = setContent(media, content);
+      if(ret != MEDIA_CONTENT_ERROR_NONE){
+        LoggerE("setContent failed");
+        return ret;
+      }
+			
       ret = media_info_update_to_db(media);
+      if(ret != MEDIA_CONTENT_ERROR_NONE){
+        LoggerE("update to db failed");
+      }
       media_info_destroy(media);
     } else {
       return ret;