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(props[QLatin1String("DisplayName")])) { displayName = qdbus_cast(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(props[QLatin1String("Nickname")])) { nickname = qdbus_cast(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(props[QLatin1String("NormalizedName")])) { normalizedName = qdbus_cast(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(props[QLatin1String("Valid")])) { valid = qdbus_cast(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(props[QLatin1String("Enabled")])) { enabled = qdbus_cast(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(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(props[QLatin1String("Parameters")])) { parameters = qdbus_cast(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( 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;