Attachment 118471 Details for Bug 92153 – Reverse notify / emit pairs to fix stale account properties

Reverse notify / emit pairs to fix stale account properties

file_92153.txt (text/plain), 9.22 KB, created by James Smith on 2015-09-28 06:43:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: James Smith

Created: 2015-09-28 06:43:41 UTC

Size: 9.22 KB

patch

obsolete

>diff --git TelepathyQt/account.cpp TelepathyQt/account.cpp
>index 8d4c7a7..f48d46e 100644
>--- TelepathyQt/account.cpp
>+++ TelepathyQt/account.cpp
>@@ -4336,16 +4336,16 @@ void Account::Private::updateProperties(const QVariantMap &props)
>         debug() << " Service Name:" << parent->serviceName();
>         /* use parent->serviceName() here as if the service name is empty we are going to use the
>          * protocol name */
>-        emit parent->serviceNameChanged(parent->serviceName());
>         parent->notify("serviceName");
>+        emit parent->serviceNameChanged(parent->serviceName());
> 
>         /* if we had a profile and the service changed, it means the profile also changed */
>         if (parent->isReady(Account::FeatureProfile)) {
>             /* service name changed, let's recreate profile */
>             profileChanged = true;
>             profile.reset();
>-            emit parent->profileChanged(parent->profile());
>             parent->notify("profile");
>+            emit parent->profileChanged(parent->profile());
>         }
>     }
> 
>@@ -4353,8 +4353,8 @@ void Account::Private::updateProperties(const QVariantMap &props)
>         displayName != qdbus_cast<QString>(props[QLatin1String("DisplayName")])) {
>         displayName = qdbus_cast<QString>(props[QLatin1String("DisplayName")]);
>         debug() << " Display Name:" << displayName;
>-        emit parent->displayNameChanged(displayName);
>         parent->notify("displayName");
>+        emit parent->displayNameChanged(displayName);
>     }
> 
>     if ((props.contains(QLatin1String("Icon")) &&
>@@ -4368,8 +4368,8 @@ void Account::Private::updateProperties(const QVariantMap &props)
>         QString newIconName = parent->iconName();
>         if (oldIconName != newIconName) {
>             debug() << " Icon:" << newIconName;
>-            emit parent->iconNameChanged(newIconName);
>             parent->notify("iconName");
>+            emit parent->iconNameChanged(newIconName);
>         }
>     }
> 
>@@ -4377,32 +4377,32 @@ void Account::Private::updateProperties(const QVariantMap &props)
>         nickname != qdbus_cast<QString>(props[QLatin1String("Nickname")])) {
>         nickname = qdbus_cast<QString>(props[QLatin1String("Nickname")]);
>         debug() << " Nickname:" << nickname;
>-        emit parent->nicknameChanged(nickname);
>         parent->notify("nickname");
>+        emit parent->nicknameChanged(nickname);
>     }
> 
>     if (props.contains(QLatin1String("NormalizedName")) &&
>         normalizedName != qdbus_cast<QString>(props[QLatin1String("NormalizedName")])) {
>         normalizedName = qdbus_cast<QString>(props[QLatin1String("NormalizedName")]);
>         debug() << " Normalized Name:" << normalizedName;
>-        emit parent->normalizedNameChanged(normalizedName);
>         parent->notify("normalizedName");
>+        emit parent->normalizedNameChanged(normalizedName);
>     }
> 
>     if (props.contains(QLatin1String("Valid")) &&
>         valid != qdbus_cast<bool>(props[QLatin1String("Valid")])) {
>         valid = qdbus_cast<bool>(props[QLatin1String("Valid")]);
>         debug() << " Valid:" << (valid ? "true" : "false");
>-        emit parent->validityChanged(valid);
>         parent->notify("valid");
>+        emit parent->validityChanged(valid);
>     }
> 
>     if (props.contains(QLatin1String("Enabled")) &&
>         enabled != qdbus_cast<bool>(props[QLatin1String("Enabled")])) {
>         enabled = qdbus_cast<bool>(props[QLatin1String("Enabled")]);
>         debug() << " Enabled:" << (enabled ? "true" : "false");
>-        emit parent->stateChanged(enabled);
>         parent->notify("enabled");
>+        emit parent->stateChanged(enabled);
>     }
> 
>     if (props.contains(QLatin1String("ConnectAutomatically")) &&
>@@ -4411,8 +4411,8 @@ void Account::Private::updateProperties(const QVariantMap &props)
>         connectsAutomatically =
>                 qdbus_cast<bool>(props[QLatin1String("ConnectAutomatically")]);
>         debug() << " Connects Automatically:" << (connectsAutomatically ? "true" : "false");
>-        emit parent->connectsAutomaticallyPropertyChanged(connectsAutomatically);
>         parent->notify("connectsAutomatically");
>+        emit parent->connectsAutomaticallyPropertyChanged(connectsAutomatically);
>     }
> 
>     if (props.contains(QLatin1String("HasBeenOnline")) &&
>@@ -4423,17 +4423,17 @@ void Account::Private::updateProperties(const QVariantMap &props)
>         // don't emit firstOnline unless we're already ready, that would be
>         // misleading - we'd emit it just before any already-used account
>         // became ready
>+        parent->notify("hasBeenOnline");
>         if (parent->isReady(Account::FeatureCore)) {
>             emit parent->firstOnline();
>         }
>-        parent->notify("hasBeenOnline");
>     }
> 
>     if (props.contains(QLatin1String("Parameters")) &&
>         parameters != qdbus_cast<QVariantMap>(props[QLatin1String("Parameters")])) {
>         parameters = qdbus_cast<QVariantMap>(props[QLatin1String("Parameters")]);
>-        emit parent->parametersChanged(parameters);
>         parent->notify("parameters");
>+        emit parent->parametersChanged(parameters);
>     }
> 
>     if (props.contains(QLatin1String("AutomaticPresence")) &&
>@@ -4443,8 +4443,8 @@ void Account::Private::updateProperties(const QVariantMap &props)
>                 props[QLatin1String("AutomaticPresence")]));
>         debug() << " Automatic Presence:" << automaticPresence.type() <<
>             "-" << automaticPresence.status();
>-        emit parent->automaticPresenceChanged(automaticPresence);
>         parent->notify("automaticPresence");
>+        emit parent->automaticPresenceChanged(automaticPresence);
>     }
> 
>     if (props.contains(QLatin1String("CurrentPresence")) &&
>@@ -4454,10 +4454,10 @@ void Account::Private::updateProperties(const QVariantMap &props)
>                 props[QLatin1String("CurrentPresence")]));
>         debug() << " Current Presence:" << currentPresence.type() <<
>             "-" << currentPresence.status();
>-        emit parent->currentPresenceChanged(currentPresence);
>         parent->notify("currentPresence");
>-        emit parent->onlinenessChanged(parent->isOnline());
>+        emit parent->currentPresenceChanged(currentPresence);
>         parent->notify("online");
>+        emit parent->onlinenessChanged(parent->isOnline());
>     }
> 
>     if (props.contains(QLatin1String("RequestedPresence")) &&
>@@ -4467,8 +4467,8 @@ void Account::Private::updateProperties(const QVariantMap &props)
>                 props[QLatin1String("RequestedPresence")]));
>         debug() << " Requested Presence:" << requestedPresence.type() <<
>             "-" << requestedPresence.status();
>-        emit parent->requestedPresenceChanged(requestedPresence);
>         parent->notify("requestedPresence");
>+        emit parent->requestedPresenceChanged(requestedPresence);
>     }
> 
>     if (props.contains(QLatin1String("ChangingPresence")) &&
>@@ -4477,8 +4477,8 @@ void Account::Private::updateProperties(const QVariantMap &props)
>         changingPresence = qdbus_cast<bool>(
>                 props[QLatin1String("ChangingPresence")]);
>         debug() << " Changing Presence:" << changingPresence;
>-        emit parent->changingPresence(changingPresence);
>         parent->notify("changingPresence");
>+        emit parent->changingPresence(changingPresence);
>     }
> 
>     if (props.contains(QLatin1String("Connection"))) {
>@@ -4571,9 +4571,9 @@ void Account::Private::updateProperties(const QVariantMap &props)
> 
>                 checkCapabilitiesChanged(profileChanged);
> 
>-                emit parent->connectionStatusChanged(connectionStatus);
>                 parent->notify("connectionError");
>                 parent->notify("connectionErrorDetails");
>+                emit parent->connectionStatusChanged(connectionStatus);
>             } else {
>                 connectionStatusChanged = false;
>             }
>@@ -4605,9 +4605,9 @@ bool Account::Private::processConnQueue()
>                 debug() << "Dropping connection for account" << parent->objectPath();
> 
>                 connection.reset();
>-                emit parent->connectionChanged(connection);
>                 parent->notify("connection");
>                 parent->notify("connectionObjectPath");
>+                emit parent->connectionChanged(connection);
>             }
> 
>             connObjPathQueue.dequeue();
>@@ -4716,8 +4716,8 @@ void Account::gotAvatar(QDBusPendingCallWatcher *watcher)
>             mPriv->readinessHelper->setIntrospectCompleted(FeatureAvatar, true);
>         }
> 
>-        emit avatarChanged(mPriv->avatar);
>         notify("avatar");
>+        emit avatarChanged(mPriv->avatar);
>     } else {
>         // check if the feature is already there, and for some reason retrieveAvatar
>         // failed when called the second time
>@@ -4796,9 +4796,9 @@ void Account::onConnectionBuilt(PendingOperation *op)
> 
>         if (!mPriv->connection.isNull()) {
>             mPriv->connection.reset();
>-            emit connectionChanged(mPriv->connection);
>             notify("connection");
>             notify("connectionObjectPath");
>+            emit connectionChanged(mPriv->connection);
>         }
>     } else {
>         ConnectionPtr prevConn = mPriv->connection;

Actions: View

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.