a7db3ab5e1
Summary: When a new message is delivered, the conversation list should update by changing the preview text and re-sorting the conversations Bonus bug discovered and fixed: previously, when the conversations list was being populated, it made a request for the first message in every conversation. This would be fine if the conversationdbusinterface pulled from local cache. However, this actually triggers a request to the phone for *every* conversation. This should be handled differently in conversationdbusinterface's requestConversation as well, but that's a project for a later day (TODO comments added) Test Plan: - Launch SMS app - Verify conversations list appears - Verify lack of massive stream of debug output indicating lots of messages for the wrong conversation are being received - Verify that opening a particular conversation shows the messages after a short delay while the backend fetches the content from the phone - Verify that receiving a new message into an existing conversation updates the conversation list Reviewers: #kde_connect, nicolasfella Reviewed By: #kde_connect, nicolasfella Subscribers: nicolasfella, apol, kdeconnect Tags: #kde_connect Differential Revision: https://phabricator.kde.org/D15608
107 lines
3.4 KiB
C++
107 lines
3.4 KiB
C++
/**
|
|
* Copyright 2013 Albert Vaca <albertvaka@gmail.com>
|
|
* Copyright 2018 Simon Redman <simon@ergotech.com>
|
|
*
|
|
* This program is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU General Public License as
|
|
* published by the Free Software Foundation; either version 2 of
|
|
* the License or (at your option) version 3 or any later version
|
|
* accepted by the membership of KDE e.V. (or its successor approved
|
|
* by the membership of KDE e.V.), which shall act as a proxy
|
|
* defined in Section 14 of version 3 of the license.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
|
*/
|
|
|
|
#ifndef CONVERSATIONSDBUSINTERFACE_H
|
|
#define CONVERSATIONSDBUSINTERFACE_H
|
|
|
|
#include <QDBusAbstractAdaptor>
|
|
#include <QHash>
|
|
#include <QList>
|
|
#include <QMap>
|
|
#include <QString>
|
|
#include <QStringList>
|
|
#include <QDir>
|
|
#include <QPointer>
|
|
|
|
#include "interfaces/conversationmessage.h"
|
|
#include "interfaces/dbusinterfaces.h"
|
|
|
|
class KdeConnectPlugin;
|
|
class Device;
|
|
|
|
Q_DECLARE_LOGGING_CATEGORY(KDECONNECT_CONVERSATIONS)
|
|
|
|
class ConversationsDbusInterface
|
|
: public QDBusAbstractAdaptor
|
|
{
|
|
Q_OBJECT
|
|
Q_CLASSINFO("D-Bus Interface", "org.kde.kdeconnect.device.conversations")
|
|
|
|
public:
|
|
explicit ConversationsDbusInterface(KdeConnectPlugin* plugin);
|
|
~ConversationsDbusInterface() override;
|
|
|
|
void addMessage(const ConversationMessage &message);
|
|
void removeMessage(const QString& internalId);
|
|
|
|
public Q_SLOTS:
|
|
/**
|
|
* Return a list of the first message in every conversation
|
|
*
|
|
* Note that the return value is a list of QVariants, which in turn have a value of
|
|
* QVariantMap created from each message
|
|
*/
|
|
QVariantList activeConversations();
|
|
|
|
void requestConversation(const QString &conversationID, int start, int end);
|
|
|
|
/**
|
|
* Send a new message to this conversation
|
|
*/
|
|
void replyToConversation(const QString& conversationID, const QString& message);
|
|
|
|
/**
|
|
* Send the request to the Telephony plugin to update the list of conversation threads
|
|
*/
|
|
void requestAllConversationThreads();
|
|
|
|
Q_SIGNALS:
|
|
Q_SCRIPTABLE void conversationCreated(const QVariantMap& msg);
|
|
Q_SCRIPTABLE void conversationRemoved(const QString& threadID);
|
|
Q_SCRIPTABLE void conversationUpdated(const QVariantMap& msg) const;
|
|
Q_SCRIPTABLE void conversationMessageReceived(const QVariantMap& msg, int pos) const;
|
|
|
|
private /*methods*/:
|
|
QString newId(); //Generates successive identifitiers to use as public ids
|
|
|
|
private /*attributes*/:
|
|
const Device* m_device;
|
|
KdeConnectPlugin* m_plugin;
|
|
|
|
/**
|
|
* Mapping of threadID to the messages which make up that thread
|
|
*
|
|
* The messages are stored as a QMap of the timestamp to the actual message object so that
|
|
* we can use .values() to get a sorted list of messages from least- to most-recent
|
|
*/
|
|
QHash<QString, QMap<qint64, ConversationMessage>> m_conversations;
|
|
|
|
/**
|
|
* Mapping of threadID to the set of uIDs known in the corresponding conversation
|
|
*/
|
|
QHash<QString, QSet<qint32>> m_known_messages;
|
|
|
|
int m_lastId;
|
|
|
|
SmsDbusInterface m_smsInterface;
|
|
};
|
|
|
|
#endif // CONVERSATIONSDBUSINTERFACE_H
|