mirrors/linphone-desktop
1
0
Fork 0
mirror of https://gitlab.linphone.org/BC/public/linphone-desktop.git synced 2026-04-17 20:08:28 +00:00
Code Issues Projects Releases Packages Wiki Activity

Quick code review

Browse source
This commit is contained in:
Sylvain Berfini 2025-10-01 14:09:19 +02:00
parent faa8f73230
commit e203df4012
2 changed files with 10 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
Linphone/model/chat/ChatModel.cpp
View file

@ -61,14 +61,17 @@ std::list<std::shared_ptr<linphone::Content>> ChatModel::getSharedDocuments() co
}
std::list<std::shared_ptr<linphone::EventLog>> ChatModel::getHistory() const {
// Pas besoin de faire le filtre au niveau app, tu peux utiliser juste getHistory(count) qui renvoie ce qu'il faut, ça sera sans doute plus efficace
int filter = mMonitor->hasCapability((int)linphone::ChatRoom::Capabilities::Conference)
? static_cast<int>(linphone::ChatRoom::HistoryFilter::ChatMessage) |
static_cast<int>(linphone::ChatRoom::HistoryFilter::InfoNoDevice)
: static_cast<int>(linphone::ChatRoom::HistoryFilter::ChatMessage);
return mMonitor->getHistory(0, filter);
// Ne fais pas le getHistory(0), ça va charger tout l'histoire d'une chat room pour rien (ce qui explique le spinner quand tu ouvres une conversation)
// A la place ne prend que 20-30 items et rajoute un scroll listener pour les charger à la volée avec getHistoryRangeEvents(currentCount, step)
return mMonitor->getHistory(30, filter);
}
std::list<std::shared_ptr<linphone::ChatMessage>> ChatModel::getChatMessageHistory() const {
/*std::list<std::shared_ptr<linphone::ChatMessage>> ChatModel::getChatMessageHistory() const {
auto history = mMonitor->getHistory(0, (int)linphone::ChatRoom::HistoryFilter::ChatMessage);
std::list<std::shared_ptr<linphone::ChatMessage>> res;
for (auto &eventLog : history) {
@ -76,7 +79,7 @@ std::list<std::shared_ptr<linphone::ChatMessage>> ChatModel::getChatMessageHisto
if (chatMessage) res.push_back(chatMessage);
}
return res;
}
}*/
QString ChatModel::getIdentifier() const {
return Utils::coreStringToAppString(mMonitor->getIdentifier());
@ -123,9 +126,10 @@ int ChatModel::getUnreadMessagesCount() const {
void ChatModel::markAsRead() {
mMonitor->markAsRead();
for (auto &message : getChatMessageHistory()) {
/*for (auto &message : getChatMessageHistory()) {
message->markAsRead();
}
}*/
mMonitor->markAsRead(); // pas besoin d'itérer sur tout l'historique des messages, ça fera gagner du temps
emit messagesRead();
}

2
Linphone/model/chat/ChatModel.hpp
View file

@ -49,7 +49,7 @@ public:
std::list<std::shared_ptr<linphone::Content>> getSharedMedias() const;
std::list<std::shared_ptr<linphone::Content>> getSharedDocuments() const;
std::list<std::shared_ptr<linphone::EventLog>> getHistory() const;
std::list<std::shared_ptr<linphone::ChatMessage>> getChatMessageHistory() const;
//std::list<std::shared_ptr<linphone::ChatMessage>> getChatMessageHistory() const;
QString getIdentifier() const;
void deleteHistory();
void deleteMessage(std::shared_ptr<linphone::ChatMessage> message);