From 3c0aca70e0068fe2d0e9289896a12dd0d6682c77 Mon Sep 17 00:00:00 2001 From: Matt Lowe Date: Tue, 27 Jan 2015 10:55:52 +0100 Subject: [PATCH] Updated method name + updated method name + added some doc --- cockatrice/src/chatview.cpp | 9 ++++++--- cockatrice/src/chatview.h | 2 +- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/cockatrice/src/chatview.cpp b/cockatrice/src/chatview.cpp index 161a8a7f..ff36a091 100644 --- a/cockatrice/src/chatview.cpp +++ b/cockatrice/src/chatview.cpp @@ -205,7 +205,7 @@ void ChatView::appendMessage(QString message, QString sender, UserLevelFlags use QString userMention = message.left(mentionEndIndex); QString userName = userMention.right(userMention.size()-1).normalized(QString::NormalizationForm_D); QMap userList = tabSupervisor->getUserListsTab()->getAllUsersList()->getUsers(); - QString correctUserName = getCorrectUserName(userList, userName); + QString correctUserName = getNameFromUserList(userList, userName); if (!correctUserName.isEmpty()) { UserListTWI *vlu = userList.value(correctUserName); mentionFormatOtherUser.setAnchorHref("user://" + QString::number(vlu->getUserInfo().user_level()) + "_" + correctUserName); @@ -225,8 +225,11 @@ void ChatView::appendMessage(QString message, QString sender, UserLevelFlags use verticalScrollBar()->setValue(verticalScrollBar()->maximum()); } - -QString ChatView::getCorrectUserName(QMap &userList, QString &userName) { +/** + Returns the correct case version of the provided username, if no correct casing version + was found then the provided name is not available and will return an empty QString. + */ +QString ChatView::getNameFromUserList(QMap &userList, QString &userName) { QMap::iterator i; QString lowerUserName = userName.toLower(); for (i = userList.begin(); i != userList.end(); ++i) { diff --git a/cockatrice/src/chatview.h b/cockatrice/src/chatview.h index c5e1eb4d..17cdd398 100644 --- a/cockatrice/src/chatview.h +++ b/cockatrice/src/chatview.h @@ -36,7 +36,7 @@ private: QTextCursor prepareBlock(bool same = false); void appendCardTag(QTextCursor &cursor, const QString &cardName); void appendUrlTag(QTextCursor &cursor, QString url); - QString getCorrectUserName(QMap &userList, QString &userName); + QString getNameFromUserList(QMap &userList, QString &userName); private slots: void openLink(const QUrl &link); public: