From dbe0a1596376a3edc4c1c10d604c9aaa54c8af73 Mon Sep 17 00:00:00 2001
From: Diadlo <polsha3@gmail.com>
Date: Mon, 18 Jul 2016 16:11:17 +0300
Subject: [PATCH] fix(chatform): Fixed call buttons

Fix #3521.
---
 src/widget/form/chatform.cpp | 21 +++++++++++----------
 src/widget/form/chatform.h   |  1 +
 2 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/src/widget/form/chatform.cpp b/src/widget/form/chatform.cpp
index 428817822..64b5e8522 100644
--- a/src/widget/form/chatform.cpp
+++ b/src/widget/form/chatform.cpp
@@ -136,7 +136,7 @@ ChatForm::ChatForm(Friend* chatFriend)
     } );
 
     setAcceptDrops(true);
-
+    disableCallButtons();
     retranslateUi();
     Translator::registerHandler(std::bind(&ChatForm::retranslateUi, this), this);
 }
@@ -628,10 +628,17 @@ void ChatForm::onFileSendFailed(uint32_t FriendId, const QString &fname)
 void ChatForm::onFriendStatusChanged(uint32_t friendId, Status status)
 {
     // Disable call buttons if friend is offline
-    if(friendId == f->getFriendID() && status == Status::Offline)
+    if(friendId != f->getFriendID())
+        return;
+
+    Status old = oldStatus.value(friendId, Status::Offline);
+
+    if (old != Status::Offline && status == Status::Offline)
         disableCallButtons();
-    else
-        onEnableCallButtons();
+    else if (old == Status::Offline && status != Status::Offline)
+        enableCallButtons();
+
+    oldStatus[friendId] = status;
 }
 
 void ChatForm::onAvatarChange(uint32_t FriendId, const QPixmap &pic)
@@ -935,12 +942,6 @@ void ChatForm::show(ContentLayout* contentLayout)
 {
     GenericChatForm::show(contentLayout);
 
-    // Disable call buttons if friend is offline
-    if(f->getStatus() == Status::Offline)
-        disableCallButtons();
-    else
-        onEnableCallButtons();
-
     if (callConfirm)
         callConfirm->show();
 }
diff --git a/src/widget/form/chatform.h b/src/widget/form/chatform.h
index 7e1b3a1d3..a7a5d99c7 100644
--- a/src/widget/form/chatform.h
+++ b/src/widget/form/chatform.h
@@ -122,6 +122,7 @@ private:
     QAction* copyStatusAction;
 
     QHash<uint, FileTransferInstance*> ftransWidgets;
+    QMap<uint32_t, Status> oldStatus;
     CallConfirmWidget *callConfirm;
     bool isTyping;
 };