[MessagingEmail] - Fixing a bug on sending message
authorAndrzej Popowski <a.popowski@samsung.com>
Thu, 21 May 2015 08:39:58 +0000 (10:39 +0200)
committerAndrzej Popowski <a.popowski@samsung.com>
Thu, 21 May 2015 08:49:41 +0000 (10:49 +0200)
Change-Id: Iea1af989e765c514b145e6c1ecbc12cee58fe1ea
Signed-off-by: Andrzej Popowski <a.popowski@samsung.com>
src/messaging/email_manager.cc

index 7f43002102c75750ceb690ff0d0b5f3250132586..308204885f9397ba37fb06e07e2f3c1678b5e8e9 100644 (file)
@@ -443,30 +443,28 @@ PlatformResult EmailManager::sendMessage(MessageRecipientsCallbackData* callback
   if (message) {
     if (!(message->is_id_set())) {
       platform_result = addOutboxMessagePlatform(callback->getAccountId(), message);
-      if (platform_result.IsError()) return platform_result;
-    }
-
-    err = email_get_mail_data(message->getId(),&mail_data);
-
-    if (EMAIL_ERROR_NONE != err) {
-      LoggerE("email_get_mail_data failed. [%d]\n", err);
-      platform_result = PlatformResult(ErrorCode::UNKNOWN_ERR, "Failed to get platform email structure");
     } else {
-      LoggerD("email_get_mail_data success.\n");
+      err = email_get_mail_data(message->getId(),&mail_data);
+      if (EMAIL_ERROR_NONE != err) {
+        LoggerE("email_get_mail_data failed. [%d]\n", err);
+        platform_result = PlatformResult(ErrorCode::UNKNOWN_ERR, "Failed to get platform email structure");
+      } else {
+        LoggerD("email_get_mail_data success.\n");
 
-      // Sending EMAIL
-      mail_data->save_status = EMAIL_MAIL_STATUS_SENDING;
+        // Sending EMAIL
+        mail_data->save_status = EMAIL_MAIL_STATUS_SENDING;
 
-      int req_id = 0;
-      err = email_send_mail(mail_data->mail_id, &req_id);
+        int req_id = 0;
+        err = email_send_mail(mail_data->mail_id, &req_id);
 
-      if (EMAIL_ERROR_NONE != err) {
-        LoggerE("Failed to send message %d", err);
-        platform_result = PlatformResult(ErrorCode::UNKNOWN_ERR, "Failed to send message");
-      } else {
-        LoggerD("req_id: %d", req_id);
-        callback->getMessage()->setMessageStatus(MessageStatus::STATUS_SENDING);
-        m_sendRequests[req_id] = callback;
+        if (EMAIL_ERROR_NONE != err) {
+          LoggerE("Failed to send message %d", err);
+          platform_result = PlatformResult(ErrorCode::UNKNOWN_ERR, "Failed to send message");
+        } else {
+          LoggerD("req_id: %d", req_id);
+          callback->getMessage()->setMessageStatus(MessageStatus::STATUS_SENDING);
+          m_sendRequests[req_id] = callback;
+        }
       }
     }
   } else {