[messaging] MessageFolder alias has been removed. 26/219826/2
authorMichal Michalski <m.michalski2@partner.samsung.com>
Tue, 10 Dec 2019 11:00:32 +0000 (12:00 +0100)
committerMichal Michalski <m.michalski2@partner.samsung.com>
Tue, 10 Dec 2019 13:51:06 +0000 (14:51 +0100)
[Verification] code compiles.

Signed-off-by: Michal Michalski <m.michalski2@partner.samsung.com>
Change-Id: I631a2a7ef9ecd0727857002d235c867a6175d1b6

src/messaging/DBus/MessageProxy.cpp
src/messaging/folders_change_callback.cc
src/messaging/message_folder.h
src/messaging/message_service.cc
src/messaging/message_service.h
src/messaging/message_storage_email.cc
src/messaging/messaging_instance.cc
src/messaging/messaging_native.cc
src/messaging/messaging_native.h

index ed918a2cac1697447ade865cda82c5e1f26b568d..3cfb9ec0b32a3e10323ce4e22654bbaf1b160e58 100644 (file)
@@ -252,7 +252,7 @@ PlatformResult MessageProxy::handleMailboxEvent(int account_id, int mailbox_id,
   eventFolder->service_type = MessageType::EMAIL;
   eventFolder->service_id = account_id;
 
-  auto folder = std::make_shared<MessageFolder>(createMessageFolder(std::to_string(mailbox_id)));
+  auto folder = std::make_shared<picojson::value>(createMessageFolder(std::to_string(mailbox_id)));
   if (event != NOTI_MAILBOX_DELETE) {
     auto ret = native::EmailGetMailboxById(mailbox_id, *folder);
     if (ret.IsError()) {
index c92d04b17ec97f7180f2b82b109921f1a66ccee5..d7d329976326ab1d1d731c2d2324c333cf7b50aa 100644 (file)
@@ -53,7 +53,7 @@ void FoldersChangeCallback::Handler(const FolderPtrVector& folders, const char*
   }
 
   picojson::array array;
-  auto each = [&array](std::shared_ptr<MessageFolder> f) -> void { array.push_back(*f); };
+  auto each = [&array](std::shared_ptr<picojson::value> f) -> void { array.push_back(*f); };
   for_each(filtered.begin(), filtered.end(), each);
 
   LoggerD("Calling:%s with:%zu added folders", action, filtered.size());
index 852d5810ea6257742679baac2940c93714b7d8bb..5daea21570cb2b847841dbfd9f704337a70c09a3 100644 (file)
@@ -45,8 +45,7 @@ picojson::value createMessageFolder(const std::string& id, const std::string& na
                                     const std::string& path = "",
                                     const std::string& parent_id = "");
 
-using MessageFolder = picojson::value;
-typedef std::vector<std::shared_ptr<MessageFolder>> FolderPtrVector;
+typedef std::vector<std::shared_ptr<picojson::value>> FolderPtrVector;
 FolderPtrVector filterFolders(tizen::AbstractFilterPtr filter,
                               const FolderPtrVector& source_folders);
 
index 6f683e67f77c637c802ea76d3cda52e158a47272..81055ac6d01c8d02f10973638e2a8ef75ceb1a5c 100644 (file)
@@ -234,11 +234,11 @@ SyncFolderCallbackData::~SyncFolderCallbackData() {
   ScopeLogger();
 }
 
-void SyncFolderCallbackData::setMessageFolder(std::shared_ptr<MessageFolder> message_folder) {
+void SyncFolderCallbackData::setMessageFolder(std::shared_ptr<picojson::value> message_folder) {
   m_message_folder = message_folder;
 }
 
-std::shared_ptr<MessageFolder> SyncFolderCallbackData::getMessageFolder() const {
+std::shared_ptr<picojson::value> SyncFolderCallbackData::getMessageFolder() const {
   return m_message_folder;
 }
 
index ca16baf8ffc64c10794a53e919ae6b663fcdde41..df0bee4b34f8845bd0c4a5ade9d5885fe51791e2 100644 (file)
@@ -144,11 +144,11 @@ class SyncFolderCallbackData : public SyncCallbackData {
   using SyncCallbackData::SyncCallbackData;
   virtual ~SyncFolderCallbackData();
 
-  void setMessageFolder(std::shared_ptr<MessageFolder> message_folder);
-  std::shared_ptr<MessageFolder> getMessageFolder() const;
+  void setMessageFolder(std::shared_ptr<picojson::value> message_folder);
+  std::shared_ptr<picojson::value> getMessageFolder() const;
 
  private:
-  std::shared_ptr<MessageFolder> m_message_folder;
+  std::shared_ptr<picojson::value> m_message_folder;
 };
 
 class MessageService {
index 47700cce4d5b4c570baea99051f529196cbe51e7..d875ce3edf7a9a5209578e09e5fe451c95482533 100644 (file)
@@ -219,7 +219,7 @@ static gboolean FindFoldersGlibTask(void* data) {
     return FALSE;
   }
 
-  std::vector<MessageFolder> folders;
+  picojson::array folders;
   PlatformResult ret = native::EmailGetMailboxList(params->account_id, folders);
   if (ret.IsError()) {
     ReportError(ret, &response);
index 89dd5b802d68282f69ba10e1436a8db1b1bfafb0..0a5e057db9dae19a8397cc33ea1e6a195c0a00c1 100644 (file)
@@ -345,7 +345,7 @@ void MessagingInstance::MessageServiceSyncFolder(const picojson::value& args,
 
   SyncFolderCallbackData* callback = new SyncFolderCallbackData(queue_, callbackId, *this);
   callback->setAccountId(id);
-  auto folder_ptr = std::make_shared<MessageFolder>(v_folder);
+  auto folder_ptr = std::make_shared<picojson::value>(v_folder);
   callback->setMessageFolder(folder_ptr);
   callback->setLimit(limit);
 
index 47b7a416a2a0b81f181bbfde805070be257bb080..d3cd2529ecba7ea8fae8fbf90e46c8cc118ab5da 100644 (file)
@@ -72,7 +72,7 @@ PlatformResult EmailGetMailboxList(int account_id, email_mailbox_t** mailboxes,
   return PlatformResult();
 }
 
-PlatformResult EmailGetMailboxList(int account_id, std::vector<MessageFolder>& folders) {
+PlatformResult EmailGetMailboxList(int account_id, picojson::array& folders) {
   ScopeLogger();
   email_mailbox_t* mailboxes = nullptr;
   int count = 0;
@@ -87,15 +87,13 @@ PlatformResult EmailGetMailboxList(int account_id, std::vector<MessageFolder>& f
   }
 
   for (int i = 0; i < count; ++i) {
-    MessageFolder folder;
-    folder = ConvertMailboxToJson(mailboxes[i]);
-    folders.push_back(folder);
+    folders.push_back(ConvertMailboxToJson(mailboxes[i]));
   }
 
   return PlatformResult();
 }
 
-PlatformResult EmailGetMailboxById(int mailbox_id, MessageFolder& folder) {
+PlatformResult EmailGetMailboxById(int mailbox_id, picojson::value& folder) {
   email_mailbox_t* mailbox = nullptr;
   int ret = email_get_mailbox_by_mailbox_id(mailbox_id, &mailbox);
   if (EMAIL_ERROR_NONE != ret) {
index 24b26f4823f335e67f250a87236ec9df7713ab33..231f7cc7efc2775e81420733d8bc7eff7155dcf1 100644 (file)
@@ -34,8 +34,8 @@ namespace extension {
 namespace messaging {
 namespace native {
 
-common::PlatformResult EmailGetMailboxList(int account_id, std::vector<MessageFolder>& folders);
-common::PlatformResult EmailGetMailboxById(int mailbox_id, MessageFolder& folder);
+common::PlatformResult EmailGetMailboxList(int account_id, picojson::array& folders);
+common::PlatformResult EmailGetMailboxById(int mailbox_id, picojson::value& folder);
 
 }  // namespace native
 }  // namespace messaging