Attachment 128618 Details for Bug 92153 – Reorder connection / presence properties

[patch] Reorder connection / presence properties

propertyordering.patch (text/plain), 2.28 KB, created by James Smith on 2016-12-21 20:04:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: James Smith

Created: 2016-12-21 20:04:34 UTC

Size: 2.28 KB

patch

obsolete

>From 59c8973714d31e719e2dd2b86b33645d52de27e9 Mon Sep 17 00:00:00 2001
>From: "James D. Smith" <smithjd15@gmail.com>
>Date: Wed, 21 Dec 2016 13:00:30 -0700
>Subject: [PATCH] Initial commit.
>
>---
> TelepathyQt/account.cpp | 26 +++++++++++++-------------
> 1 file changed, 13 insertions(+), 13 deletions(-)
>
>diff --git TelepathyQt/account.cpp TelepathyQt/account.cpp
>index f6635e1..ec85e25 100644
>--- TelepathyQt/account.cpp
>+++ TelepathyQt/account.cpp
>@@ -4388,19 +4388,6 @@ void Account::Private::updateProperties(const QVariantMap &props)
>         parent->notify("automaticPresence");
>     }
> 
>-    if (props.contains(QLatin1String("CurrentPresence")) &&
>-        currentPresence.barePresence() != qdbus_cast<SimplePresence>(
>-                props[QLatin1String("CurrentPresence")])) {
>-        currentPresence = Presence(qdbus_cast<SimplePresence>(
>-                props[QLatin1String("CurrentPresence")]));
>-        debug() << " Current Presence:" << currentPresence.type() <<
>-            "-" << currentPresence.status();
>-        emit parent->currentPresenceChanged(currentPresence);
>-        parent->notify("currentPresence");
>-        emit parent->onlinenessChanged(parent->isOnline());
>-        parent->notify("online");
>-    }
>-
>     if (props.contains(QLatin1String("RequestedPresence")) &&
>         requestedPresence.barePresence() != qdbus_cast<SimplePresence>(
>                 props[QLatin1String("RequestedPresence")])) {
>@@ -4524,6 +4511,19 @@ void Account::Private::updateProperties(const QVariantMap &props)
>     if (!connectionStatusChanged && profileChanged) {
>         checkCapabilitiesChanged(profileChanged);
>     }
>+
>+    if (props.contains(QLatin1String("CurrentPresence")) &&
>+        currentPresence.barePresence() != qdbus_cast<SimplePresence>(
>+                props[QLatin1String("CurrentPresence")])) {
>+        currentPresence = Presence(qdbus_cast<SimplePresence>(
>+                props[QLatin1String("CurrentPresence")]));
>+        debug() << " Current Presence:" << currentPresence.type() <<
>+            "-" << currentPresence.status();
>+        emit parent->currentPresenceChanged(currentPresence);
>+        parent->notify("currentPresence");
>+        emit parent->onlinenessChanged(parent->isOnline());
>+        parent->notify("online");
>+    }
> }
> 
> void Account::Private::retrieveAvatar()
>-- 
>2.10.2
>

Actions: View | Splinter Review

Attachments on bug 92153: 118471 | 119368 | 128461 | 128535 | 128601 | 128618

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.