From: Denis Dolzhenko <d.dolzhenko@samsung.com>
Date: Tue, 4 Oct 2016 15:09:33 +0000 (+0300)
Subject: TSAM-8335 Fixed: read report
X-Git-Tag: submit/tizen/20161005.131726~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3cc58037aab5a8683aff22b38e471708c963d630;p=profile%2Fmobile%2Fapps%2Fnative%2Fmessage.git

TSAM-8335 Fixed: read report

Change-Id: I86e00a8ee849c86deedfaa3383b1925417f9261a
Signed-off-by: Denis Dolzhenko <d.dolzhenko@samsung.com>
---

diff --git a/src/Conversation/ConvList/Controller/src/ConvListItem.cpp b/src/Conversation/ConvList/Controller/src/ConvListItem.cpp
index 88cc1b99..4d92479b 100644
--- a/src/Conversation/ConvList/Controller/src/ConvListItem.cpp
+++ b/src/Conversation/ConvList/Controller/src/ConvListItem.cpp
@@ -62,10 +62,10 @@ ConvListItem::ConvListItem(const MsgConversationItem &item,
     , m_IsRestrictedByDpm(item.isRestrictedByDpm())
     , m_IsNeededReadReport(false)
 {
-
     m_IsNeededReadReport = m_Direction == Message::MD_Received &&
-                           m_Type == MsgUtils::isMms(m_Type) &&
+                           m_Type == Message::MT_MMS &&
                            !item.isRead();
+
     prepareBubble(item, searchWord);
 }
 
@@ -81,12 +81,12 @@ ConvListItem::~ConvListItem()
 void ConvListItem::updateStatus()
 {
     MessageRef msg = m_App.getMsgEngine().getStorage().getMessage(m_MsgId);
-    if(msg)
-    {
-        m_Time = msg->getTime();
-        m_NetworkStatus = msg->getNetworkStatus();
-        m_IsRestrictedByDpm = msg->isRestrictedByDpm();
-    }
+    if(!msg)
+        return;
+
+    m_Time = msg->getTime();
+    m_NetworkStatus = msg->getNetworkStatus();
+    m_IsRestrictedByDpm = msg->isRestrictedByDpm();
 
     if(m_NetworkStatus == Message::NS_Send_Fail)
     {