Attachment 130684 Details for Bug 100363 – Add property averaging, presence setting and account lookup by unique identifier to AccountSet, <> operators to Presence.

[patch] Add property averaging, presence setting and account lookup by unique identifier to AccountSet, <> operators to Presence.

0001-AccountSet-account-connection-properties-averages-an.patch (text/plain), 15.50 KB, created by James Smith on 2017-04-05 06:22:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: James Smith

Created: 2017-04-05 06:22:41 UTC

Size: 15.50 KB

patch

obsolete

>rom 6d3864d2d555271d98730edc877e6018a1528f54 Mon Sep 17 00:00:00 2001
>From: "James D. Smith" <smithjd15@gmail.com>
>Date: Sat, 4 Mar 2017 13:47:33 -0700
>Subject: [PATCH 1/2] Implement presence < and > operators.
>
>---
> TelepathyQt/presence.cpp | 23 +++++++++++++++++++++++
> TelepathyQt/presence.h   |  2 ++
> 2 files changed, 25 insertions(+)
>
>diff --git TelepathyQt/presence.cpp TelepathyQt/presence.cpp
>index 823067d..d5a9c6b 100644
>--- TelepathyQt/presence.cpp
>+++ TelepathyQt/presence.cpp
>@@ -150,6 +150,29 @@ bool Presence::operator!=(const Presence &other) const
>     return mPriv->sp != other.mPriv->sp;
> }
> 
>+bool Presence::operator <(const Presence &other) const
>+{
>+    if (!isValid() || !other.isValid()) {
>+        if (!isValid() && !other.isValid()) {
>+            return false;
>+        }
>+        return true;
>+    }
>+
>+    if (type() > other.type()) {
>+        return true;
>+    } else if (type() == other.type()) {
>+        return (statusMessage() < other.statusMessage());
>+    } else {
>+        return false;
>+    }
>+}
>+
>+bool Presence::operator >(const Presence &other) const
>+{
>+    return (other < *this);
>+}
>+
> ConnectionPresenceType Presence::type() const
> {
>     if (!isValid()) {
>diff --git TelepathyQt/presence.h TelepathyQt/presence.h
>index 31fd056..ef4e848 100644
>--- TelepathyQt/presence.h
>+++ TelepathyQt/presence.h
>@@ -57,6 +57,8 @@ public:
>     Presence &operator=(const Presence &other);
>     bool operator==(const Presence &other) const;
>     bool operator!=(const Presence &other) const;
>+    bool operator <(const Presence &other) const;
>+    bool operator >(const Presence &other) const;
> 
>     ConnectionPresenceType type() const;
>     QString status() const;
>-- 
>2.10.2
>
>From adae8a5f9db3ba6ec53799747174d25455a73988 Mon Sep 17 00:00:00 2001
>From: "James D. Smith" <smithjd15@gmail.com>
>Date: Thu, 23 Mar 2017 14:56:04 -0600
>Subject: [PATCH] AccountSet account connection properties averages and
> setPresence capability. Add account lookup by uniqueIdentifier.
>
>---
> TelepathyQt/account-set-internal.h |  11 ++
> TelepathyQt/account-set.cpp        | 274 ++++++++++++++++++++++++++++++++++++-
> TelepathyQt/account-set.h          |  21 +++
> 3 files changed, 305 insertions(+), 1 deletion(-)
>
>diff --git TelepathyQt/account-set-internal.h TelepathyQt/account-set-internal.h
>index f7dbf9e0..290d093b 100644
>--- TelepathyQt/account-set-internal.h
>+++ TelepathyQt/account-set-internal.h
>@@ -43,6 +43,11 @@ struct TP_QT_NO_EXPORT AccountSet::Private
>     void removeAccount(const AccountPtr &account);
>     void wrapAccount(const AccountPtr &account);
>     void filterAccount(const AccountPtr &account);
>+    void requestedPresenceChanged();
>+    void currentPresenceChanged();
>+    void connectionStatusChanged();
>+    void changingPresence();
>+    void onlinenessChanged();
>     bool accountMatchFilter(AccountWrapper *account);
> 
>     AccountSet *parent;
>@@ -50,6 +55,12 @@ struct TP_QT_NO_EXPORT AccountSet::Private
>     AccountFilterConstPtr filter;
>     QHash<QString, AccountWrapper *> wrappers;
>     QHash<QString, AccountPtr> accounts;
>+
>+    Tp::Presence requestedPresence;
>+    Tp::Presence currentPresence;
>+    Tp::ConnectionStatus connectionStatus;
>+    bool isChangingPresence;
>+    bool isOnline;
>     bool ready;
> };
> 
>diff --git TelepathyQt/account-set.cpp TelepathyQt/account-set.cpp
>index fce92a39..da0286c0 100644
>--- TelepathyQt/account-set.cpp
>+++ TelepathyQt/account-set.cpp
>@@ -33,6 +33,7 @@
> #include <TelepathyQt/AccountManager>
> #include <TelepathyQt/ConnectionCapabilities>
> #include <TelepathyQt/ConnectionManager>
>+#include <TelepathyQt/Presence>
> 
> namespace Tp
> {
>@@ -66,6 +67,12 @@ AccountSet::Private::Private(AccountSet *parent,
> 
> void AccountSet::Private::init()
> {
>+    requestedPresence = Tp::Presence::offline();
>+    currentPresence = Tp::Presence::offline();
>+    connectionStatus = Tp::ConnectionStatusDisconnected;
>+    isChangingPresence = false;
>+    isOnline = false;
>+
>     if (filter->isValid()) {
>         connectSignals();
>         insertAccounts();
>@@ -93,6 +100,12 @@ void AccountSet::Private::insertAccount(const Tp::AccountPtr &account)
>     Q_ASSERT(!wrappers.contains(accountPath));
>     wrapAccount(account);
>     filterAccount(account);
>+
>+    requestedPresenceChanged();
>+    currentPresenceChanged();
>+    connectionStatusChanged();
>+    changingPresence();
>+    onlinenessChanged();
> }
> 
> void AccountSet::Private::removeAccount(const Tp::AccountPtr &account)
>@@ -106,6 +119,97 @@ void AccountSet::Private::removeAccount(const Tp::AccountPtr &account)
>     wrapper->deleteLater();
> 
>     emit parent->accountRemoved(account);
>+
>+    requestedPresenceChanged();
>+    currentPresenceChanged();
>+    connectionStatusChanged();
>+    changingPresence();
>+    onlinenessChanged();
>+}
>+
>+void AccountSet::Private::requestedPresenceChanged()
>+{
>+    Tp::Presence highestRequestedPresence = Tp::PresenceSpec::unknown().presence();
>+
>+    foreach (const Tp::AccountPtr &account, accounts) {
>+        if (account->requestedPresence() > highestRequestedPresence) {
>+            highestRequestedPresence = account->requestedPresence();
>+        }
>+    }
>+
>+    if (requestedPresence != highestRequestedPresence) {
>+        requestedPresence = highestRequestedPresence;
>+        emit parent->requestedPresenceChanged(requestedPresence);
>+    }
>+}
>+
>+void AccountSet::Private::currentPresenceChanged()
>+{
>+    Tp::Presence highestCurrentPresence = Tp::PresenceSpec::unknown().presence();
>+
>+    foreach (const Tp::AccountPtr &account, accounts) {
>+        if (account->currentPresence() > highestCurrentPresence) {
>+            highestCurrentPresence = account->currentPresence();
>+        }
>+    }
>+
>+    if (currentPresence != highestCurrentPresence) {
>+        currentPresence = highestCurrentPresence;
>+        emit parent->currentPresenceChanged(currentPresence);
>+    }
>+}
>+
>+void AccountSet::Private::connectionStatusChanged()
>+{
>+    Tp::ConnectionStatus status = Tp::ConnectionStatusDisconnected;
>+    QList<Tp::ConnectionStatus> allConnectionStatuses;
>+
>+    foreach (const Tp::AccountPtr &account, accounts) {
>+        allConnectionStatuses << account->connectionStatus();
>+    }
>+
>+    if (allConnectionStatuses.contains(Tp::ConnectionStatusConnecting)) {
>+        status = Tp::ConnectionStatusConnecting;
>+    } else if (allConnectionStatuses.contains(Tp::ConnectionStatusConnected)) {
>+        status = Tp::ConnectionStatusConnected;
>+    }
>+
>+    if (connectionStatus != status) {
>+        connectionStatus = status;
>+        emit parent->connectionStatusChanged(connectionStatus);
>+    }
>+}
>+
>+void AccountSet::Private::changingPresence()
>+{
>+    bool changing = false;
>+    foreach (const Tp::AccountPtr &account, accounts) {
>+        changing = account->isChangingPresence();
>+        if (account->isChangingPresence()) {
>+            break;
>+        }
>+    }
>+
>+    if (isChangingPresence != changing) {
>+        isChangingPresence = changing;
>+        emit parent->isChangingPresence(isChangingPresence);
>+    }
>+}
>+
>+void AccountSet::Private::onlinenessChanged()
>+{
>+    bool online = false;
>+    foreach (const Tp::AccountPtr &account, accounts) {
>+        online = account->isOnline();
>+        if (!account->isOnline()) {
>+            break;
>+        }
>+    }
>+
>+    if (isOnline != online) {
>+        isOnline = online;
>+        emit parent->onlinenessChanged(isOnline);
>+    }
> }
> 
> void AccountSet::Private::wrapAccount(const AccountPtr &account)
>@@ -116,7 +220,7 @@ void AccountSet::Private::wrapAccount(const AccountPtr &account)
>             SLOT(onAccountRemoved(Tp::AccountPtr)));
>     parent->connect(wrapper,
>             SIGNAL(accountPropertyChanged(Tp::AccountPtr,QString)),
>-            SLOT(onAccountChanged(Tp::AccountPtr)));
>+            SLOT(onAccountPropertyChanged(Tp::AccountPtr,QString)));
>     parent->connect(wrapper,
>             SIGNAL(accountCapabilitiesChanged(Tp::AccountPtr,Tp::ConnectionCapabilities)),
>             SLOT(onAccountChanged(Tp::AccountPtr)));
>@@ -381,6 +485,102 @@ QList<AccountPtr> AccountSet::accounts() const
> }
> 
> /**
>+ * Return the account object that matches uniqueIdentifier.
>+ *
>+ * \param uniqueIdentifier The unique identifier of the account.
>+ *
>+ * \return A pointer to an Account object.
>+ */
>+AccountPtr AccountSet::account(const QString &uniqueIdentifier) const
>+{
>+    Tp::AccountPtr account = Tp::AccountPtr();
>+
>+    if (mPriv->accounts.contains(uniqueIdentifier)) {
>+        account = mPriv->accounts.value(uniqueIdentifier);
>+    }
>+
>+    return account;
>+}
>+
>+/**
>+ * Set the requested presence of all accounts that match filter.
>+ *
>+ * \param presence The requested presence.
>+ */
>+void AccountSet::setPresence(const Tp::Presence &presence)
>+{
>+    foreach (const Tp::AccountPtr &account, mPriv->accounts.values()) {
>+        account->setRequestedPresence(presence);
>+    }
>+}
>+
>+/**
>+ * The requested presence of all accounts that match filter.
>+ *
>+ * Change notification is via the requestedPresenceChanged() signal.
>+ *
>+ * \return The highest requested presence of the accounts that match filter.
>+ * \sa requestedPresenceChanged()
>+ */
>+Tp::Presence AccountSet::requestedPresence() const
>+{
>+    return mPriv->requestedPresence;
>+}
>+
>+/**
>+ * The current presence of all accounts that match filter.
>+ *
>+ * Change notification is via the currentPresenceChanged() signal.
>+ *
>+ * \return The highest current presence of the accounts that match filter.
>+ * \sa currentPresenceChanged()
>+ */
>+Tp::Presence AccountSet::currentPresence() const
>+{
>+    return mPriv->currentPresence;
>+}
>+
>+/**
>+ * The connection status of all accounts that match filter.
>+ *
>+ * Change notification is via the connectionStatusChanged() signal.
>+ *
>+ * \return connecting if any account is connecting, connected if at
>+ * least one account is connected, or disconnected.
>+ * \sa connectionStatusChanged()
>+*/
>+Tp::ConnectionStatus AccountSet::connectionStatus() const
>+{
>+    return mPriv->connectionStatus;
>+}
>+
>+/**
>+ * Of all accounts that match filter, if any are changing presence.
>+ *
>+ * Change notification is via the isChangingPresence() signal.
>+ *
>+ * \return true if any accounts that match filter are changing presence.
>+ * \sa isChangingPresence()
>+ */
>+bool AccountSet::changingPresence() const
>+{
>+    return mPriv->isChangingPresence;
>+}
>+
>+/**
>+ * Of all accounts that match filter, if all are online.
>+ *
>+ * Change notification is via the onlinenessChanged() signal.
>+ *
>+ * \return false if all accounts that match filter are not online.
>+ * \sa onlinenessChanged()
>+ */
>+bool AccountSet::online() const
>+{
>+    return mPriv->isOnline;
>+}
>+
>+/**
>  * \fn void AccountSet::accountAdded(const Tp::AccountPtr &account)
>  *
>  * Emitted whenever an account that matches filter is added to
>@@ -400,6 +600,51 @@ QList<AccountPtr> AccountSet::accounts() const
>  * \sa accounts()
>  */
> 
>+/**
>+ * \fn void AccountSet::requestedPresenceChanged(const Tp::Presence &requestedPresence)
>+ *
>+ * Emitted whenever the highest requested presence of this set of accounts changes.
>+ *
>+ * \param requestedPresence The highest requested presence.
>+ * \sa requestedPresence()
>+ */
>+
>+/**
>+ * \fn void AccountSet::currentPresenceChanged(const Tp::Presence &currentPresence)
>+ *
>+ * Emitted whenever the highest current presence of this set of accounts changes.
>+ *
>+ * \param requestedPresence The highest requested presence.
>+ * \sa currentPresence()
>+ */
>+
>+/**
>+ * \fn void AccountSet::connectionStatusChanged(Tp::ConnectionStatus connectionStatus)
>+ *
>+ * Emitted whenever the connection status of this set of accounts changes.
>+ *
>+ * \param connectionStatus The connection status.
>+ * \sa connectionStatus()
>+ */
>+
>+/**
>+ * \fn void AccountSet::isChangingPresence(bool changingPresence)
>+ *
>+ * Emitted whenever any of this set of accounts are changing presence.
>+ *
>+ * \param changingPresence If any account is changing presence.
>+ * \sa changingPresence()
>+ */
>+
>+/**
>+ * \fn void AccountSet::onlinenessChanged(bool online)
>+ *
>+ * Emitted whenever any of this set of accounts onlineness changes.
>+ *
>+ * \param online If all accounts are online.
>+ * \sa online()
>+ */
>+
> void AccountSet::onNewAccount(const AccountPtr &account)
> {
>     mPriv->insertAccount(account);
>@@ -415,4 +660,31 @@ void AccountSet::onAccountChanged(const AccountPtr &account)
>     mPriv->filterAccount(account);
> }
> 
>+void AccountSet::onAccountPropertyChanged(const Tp::AccountPtr &account, const QString &propertyName)
>+{
>+    mPriv->filterAccount(account);
>+
>+    if (propertyName == QLatin1String("requestedPresence")) {
>+        if (mPriv->requestedPresence != account->requestedPresence()) {
>+            mPriv->requestedPresenceChanged();
>+        }
>+    } else if (propertyName == QLatin1String("currentPresence")) {
>+        if (mPriv->currentPresence != account->currentPresence()) {
>+            mPriv->currentPresenceChanged();
>+        }
>+    } else if (propertyName == QLatin1String("connectionStatus")) {
>+        if (mPriv->connectionStatus != account->connectionStatus()) {
>+            mPriv->connectionStatusChanged();
>+        }
>+    } else if (propertyName == QLatin1String("changingPresence")) {
>+        if (mPriv->isChangingPresence != account->isChangingPresence()) {
>+            mPriv->changingPresence();
>+        }
>+    } else if (propertyName == QLatin1String("online")) {
>+        if (mPriv->isOnline != account->isOnline()) {
>+            mPriv->onlinenessChanged();
>+        }
>+    }
>+}
>+
> } // Tp
>diff --git TelepathyQt/account-set.h TelepathyQt/account-set.h
>index 40dd79af..92b68e28 100644
>--- TelepathyQt/account-set.h
>+++ TelepathyQt/account-set.h
>@@ -30,6 +30,7 @@
> #include <TelepathyQt/Filter>
> #include <TelepathyQt/Object>
> #include <TelepathyQt/Types>
>+#include <TelepathyQt/Presence>
> 
> #include <QList>
> #include <QString>
>@@ -45,6 +46,11 @@ class TP_QT_EXPORT AccountSet : public Object
>     Q_PROPERTY(AccountManagerPtr accountManager READ accountManager)
>     Q_PROPERTY(AccountFilterConstPtr filter READ filter)
>     Q_PROPERTY(QList<AccountPtr> accounts READ accounts)
>+    Q_PROPERTY(Tp::Presence requestedPresence READ requestedPresence WRITE setPresence NOTIFY requestedPresenceChanged)
>+    Q_PROPERTY(Tp::Presence currentPresence READ currentPresence NOTIFY currentPresenceChanged)
>+    Q_PROPERTY(Tp::ConnectionStatus connectionStatus READ connectionStatus NOTIFY connectionStatusChanged)
>+    Q_PROPERTY(bool changingPresence READ changingPresence NOTIFY isChangingPresence)
>+    Q_PROPERTY(bool online READ online NOTIFY onlinenessChanged)
> 
> public:
>     AccountSet(const AccountManagerPtr &accountManager,
>@@ -58,15 +64,30 @@ public:
>     AccountFilterConstPtr filter() const;
> 
>     QList<AccountPtr> accounts() const;
>+    AccountPtr account(const QString &uniqueIdentifier) const;
>+
>+    void setPresence(const Tp::Presence &presence);
>+
>+    Tp::Presence requestedPresence() const;
>+    Tp::Presence currentPresence() const;
>+    Tp::ConnectionStatus connectionStatus() const;
>+    bool changingPresence() const;
>+    bool online() const;
> 
> Q_SIGNALS:
>     void accountAdded(const Tp::AccountPtr &account);
>     void accountRemoved(const Tp::AccountPtr &account);
>+    void requestedPresenceChanged(const Tp::Presence &requestedPresence);
>+    void currentPresenceChanged(const Tp::Presence &currentPresence);
>+    void connectionStatusChanged(Tp::ConnectionStatus connectionStatus);
>+    void isChangingPresence(bool changingPresence);
>+    void onlinenessChanged(bool online);
> 
> private Q_SLOTS:
>     TP_QT_NO_EXPORT void onNewAccount(const Tp::AccountPtr &account);
>     TP_QT_NO_EXPORT void onAccountRemoved(const Tp::AccountPtr &account);
>     TP_QT_NO_EXPORT void onAccountChanged(const Tp::AccountPtr &account);
>+    TP_QT_NO_EXPORT void onAccountPropertyChanged(const Tp::AccountPtr &account, const QString &propertyName);
> 
> private:
>     struct Private;
>-- 
>2.12.0
>

Actions: View | Splinter Review

Attachments on bug 100363: 130405 | 130684 | 130685 | 130887 | 131717 | 131718 | 131719 | 131720 | 131728 | 137599

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.