Merge branch 'jb51663' into 'master'

[commhistoryd] Fix regression with the missed call secondary actions. Fixes JB#51663 See merge request mer-core/commhistory-daemon!48
sailfishos · Oct 20, 2020 · e13f2b5 · e13f2b5
2 parents 53f8dca + 9c4cb50
commit e13f2b5
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/personalnotification.cpp b/src/personalnotification.cpp
@@ -299,10 +299,15 @@ bool PersonalNotification::hidden() const
 
 bool PersonalNotification::hasPhoneNumber() const
 {
-    if (m_eventType == CommHistory::Event::SMSEvent || m_eventType == CommHistory::Event::MMSEvent || m_eventType == VOICEMAIL_SMS_EVENT_TYPE) {
+    switch (m_eventType) {
+    case CommHistory::Event::SMSEvent:
+    case CommHistory::Event::MMSEvent:
+    case CommHistory::Event::CallEvent:
+    case VOICEMAIL_SMS_EVENT_TYPE:
         return CommHistory::normalizePhoneNumber(m_remoteUid, true).length() > 0;
+    default:
+        return false;
     }
-    return false;
 }
 
 void PersonalNotification::setRemoteUid(const QString& remoteUid)