[Bug 92153] Account properties notify signals are dispatched before the account property is updated

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=92153

--- Comment #1 from David Edmundson <kde at davidedmundson.co.uk> ---
I'm not sure I understand

parent->notify just emits propertyChanged("propertyName")


 * Notify that a property named \a propertyName changed.
 *
 * This method will emit propertyChanged() for \a propertyName.
 *
 * \todo Use for more classes beyond Account. Most importantly Contact.
 */
void Object::notify(const char *propertyName)
{
    emit propertyChanged(QLatin1String(propertyName));
}


As far as I can tell, the only user of this is AccountSet and some unit tests.

git grep propertyChanged
TelepathyQt/account-set.cpp:            SIGNAL(propertyChanged(QString)),
TelepathyQt/object.cpp: * This method will emit propertyChanged() for \a
propertyName.
TelepathyQt/object.cpp:    emit propertyChanged(QLatin1String(propertyName));
TelepathyQt/object.h:    void propertyChanged(const QString &propertyName);
tests/dbus/account-connection-factory.cpp:               
SIGNAL(propertyChanged(QString)),
tests/dbus/account-connection-factory.cpp:               
SIGNAL(propertyChanged(QString)),


As for requested presence; it's set on line 4466 and the emit is on line 4470,
so that's always set before the emit.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.