fix(friend): Add save friend alias on change

Fix #4706
qTox · Nov 24, 2017 · c0a7488 · c0a7488
1 parent 5bc8ef4
commit c0a7488
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/src/widget/friendwidget.cpp b/src/widget/friendwidget.cpp
@@ -429,8 +429,4 @@ void FriendWidget::setAlias(const QString& _alias)
     // Hack to avoid edit const Friend. TODO: Repalce on emit
     Friend* f = FriendList::findFriend(frnd->getId());
     f->setAlias(alias);
-
-    Settings& s = Settings::getInstance();
-    s.setFriendAlias(frnd->getPublicKey(), alias);
-    s.savePersonal();
 }
diff --git a/src/widget/widget.cpp b/src/widget/widget.cpp
@@ -1099,17 +1099,22 @@ void Widget::onFriendAliasChanged(uint32_t friendId, const QString& alias)
         GUI::setWindowTitle(alias);
     }
 
-    Status s = f->getStatus();
-    contactListWidget->moveWidget(friendWidget, s);
+    Status status = f->getStatus();
+    contactListWidget->moveWidget(friendWidget, status);
     FilterCriteria criteria = getFilterCriteria();
-    bool filter = s == Status::Offline ? filterOffline(criteria) : filterOnline(criteria);
+    bool filter = status == Status::Offline ? filterOffline(criteria) : filterOnline(criteria);
     friendWidget->searchName(ui->searchContactText->text(), filter);
 
     ChatForm* friendForm = chatForms[friendId];
     friendForm->setName(alias);
     for (Group* g : GroupList::getAllGroups()) {
         g->regeneratePeerList();
     }
+
+    const ToxPk& pk = f->getPublicKey();
+    Settings& s = Settings::getInstance();
+    s.setFriendAlias(pk, alias);
+    s.savePersonal();
 }
 
 void Widget::onChatroomWidgetClicked(GenericChatroomWidget* widget)