on xenial help sign in
Merge from wallpaper branch
commit3609dc04f077d7c5a5f6e697ac6a6f7efbfc3de5 authorMichael Terry <michael.terry@canonical.com> dateFri 6 Sep 2013, 15:21 (13 years ago) parentc2da3c77417ab0
19 files changed+343−342
Shell.qml+1−1 plugins/AccountsService/AccountsBindings.cpp−125 plugins/AccountsService/AccountsBindings.h−80 plugins/AccountsService/AccountsService.cpp+79−80 plugins/AccountsService/AccountsService.h+40−15 plugins/AccountsService/AccountsServiceDBusAdaptor.cpp+126 plugins/AccountsService/AccountsServiceDBusAdaptor.h+54 plugins/AccountsService/CMakeLists.txt+1−1 plugins/AccountsService/plugin.cpp+3−3 plugins/Unity/Launcher/CMakeLists.txt+1−1 plugins/Unity/Launcher/backend/launcherbackend.cpp+2−2 plugins/Unity/Launcher/backend/launcherbackend.h+2−2 tests/mocks/AccountsService/AccountsBindings.cpp → tests/mocks/AccountsService/AccountsService.cpp+11−9 tests/mocks/AccountsService/AccountsBindings.h → tests/mocks/AccountsService/AccountsService.h+5−5 tests/mocks/AccountsService/CMakeLists.txt+1−1 tests/mocks/AccountsService/plugin.cpp+3−3 tests/plugins/AccountsService/CMakeLists.txt+1−1 tests/plugins/AccountsService/client.cpp+11−11 tests/plugins/Unity/Launcher/CMakeLists.txt+2−2
Shell.qml+1−1
@@ -15,11 +15,11 @@
1515 */
1616
1717import QtQuick 2.0
18+import GSettings 1.0
1819import Ubuntu.Application 0.1
1920import Ubuntu.Components 0.1
2021import Ubuntu.Gestures 0.1
2122import Unity.Launcher 0.1
22−import GSettings 1.0
2323import Powerd 0.1
2424import SessionManager 0.1
2525import "Dash"
plugins/AccountsService/AccountsBindings.cppdeleted−125
@@ -1,125 +0,0 @@
1−/*
2− * Copyright (C) 2013 Canonical, Ltd.
3− *
4− * This program is free software; you can redistribute it and/or modify
5− * it under the terms of the GNU General Public License as published by
6− * the Free Software Foundation; version 3.
7− *
8− * This program is distributed in the hope that it will be useful,
9− * but WITHOUT ANY WARRANTY; without even the implied warranty of
10− * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11− * GNU General Public License for more details.
12− *
13− * You should have received a copy of the GNU General Public License
14− * along with this program. If not, see <http://www.gnu.org/licenses/>.
15− *
16− * Author: Michael Terry <michael.terry@canonical.com>
17− */
18−
19−#include "AccountsBindings.h"
20−#include "AccountsService.h"
21−
22−#include <QStringList>
23−
24−AccountsBindings::AccountsBindings(QObject* parent)
25− : QObject(parent),
26− m_service(new AccountsService(this))
27−{
28− connect(m_service, SIGNAL(propertiesChanged(const QString &, const QString &, const QStringList &)),
29− this, SLOT(propertiesChanged(const QString &, const QString &, const QStringList &)));
30− connect(m_service, SIGNAL(maybeChanged(const QString &)),
31− this, SLOT(maybeChanged(const QString &)));
32−
33− setUser(qgetenv("USER"));
34− updateDemoEdgesForCurrentUser();
35−}
36−
37−QString AccountsBindings::getUser()
38−{
39− return m_user;
40−}
41−
42−void AccountsBindings::setUser(const QString &user)
43−{
44− m_user = user;
45− Q_EMIT userChanged();
46−
47− updateDemoEdges();
48− updateBackgroundFile();
49−}
50−
51−bool AccountsBindings::getDemoEdges()
52−{
53− return m_demoEdges;
54−}
55−
56−void AccountsBindings::setDemoEdges(bool demoEdges)
57−{
58− m_demoEdges = demoEdges;
59− m_service->setUserProperty(m_user, "com.canonical.unity.AccountsService", "demo-edges", demoEdges);
60−}
61−
62−bool AccountsBindings::getDemoEdgesForCurrentUser()
63−{
64− return m_demoEdgesForCurrentUser;
65−}
66−
67−void AccountsBindings::setDemoEdgesForCurrentUser(bool demoEdgesForCurrentUser)
68−{
69− m_demoEdgesForCurrentUser = demoEdgesForCurrentUser;
70− m_service->setUserProperty(qgetenv("USER"), "com.canonical.unity.AccountsService", "demo-edges", demoEdgesForCurrentUser);
71−}
72−
73−QString AccountsBindings::getBackgroundFile()
74−{
75− return m_backgroundFile;
76−}
77−
78−void AccountsBindings::updateDemoEdges()
79−{
80− auto demoEdges = m_service->getUserProperty(m_user, "com.canonical.unity.AccountsService", "demo-edges").toBool();
81− if (m_demoEdges != demoEdges) {
82− m_demoEdges = demoEdges;
83− Q_EMIT demoEdgesChanged();
84− }
85−}
86−
87−void AccountsBindings::updateDemoEdgesForCurrentUser()
88−{
89− auto demoEdgesForCurrentUser = m_service->getUserProperty(qgetenv("USER"), "com.canonical.unity.AccountsService", "demo-edges").toBool();
90− if (m_demoEdgesForCurrentUser != demoEdgesForCurrentUser) {
91− m_demoEdgesForCurrentUser = demoEdgesForCurrentUser;
92− Q_EMIT demoEdgesForCurrentUserChanged();
93− }
94−}
95−
96−void AccountsBindings::updateBackgroundFile()
97−{
98− auto backgroundFile = m_service->getUserProperty(m_user, "org.freedesktop.Accounts.User", "BackgroundFile").toString();
99− if (m_backgroundFile != backgroundFile) {
100− m_backgroundFile = backgroundFile;
101− Q_EMIT backgroundFileChanged();
102− }
103−}
104−
105−void AccountsBindings::propertiesChanged(const QString &user, const QString &interface, const QStringList &changed)
106−{
107− if (interface == "com.canonical.unity.AccountsService") {
108− if (changed.contains("demo-edges")) {
109− if (qgetenv("USER") == user) {
110− updateDemoEdgesForCurrentUser();
111− }
112− if (m_user != user) {
113− updateDemoEdges();
114− }
115− }
116− }
117−}
118−
119−void AccountsBindings::maybeChanged(const QString &user)
120−{
121− if (m_user == user) {
122− // Standard properties might have changed
123− updateBackgroundFile();
124− }
125−}
plugins/AccountsService/AccountsBindings.hdeleted−80
@@ -1,80 +0,0 @@
1−/*
2− * Copyright (C) 2012,2013 Canonical, Ltd.
3− *
4− * This program is free software; you can redistribute it and/or modify
5− * it under the terms of the GNU General Public License as published by
6− * the Free Software Foundation; version 3.
7− *
8− * This program is distributed in the hope that it will be useful,
9− * but WITHOUT ANY WARRANTY; without even the implied warranty of
10− * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11− * GNU General Public License for more details.
12− *
13− * You should have received a copy of the GNU General Public License
14− * along with this program. If not, see <http://www.gnu.org/licenses/>.
15− *
16− * Authors: Michael Terry <michael.terry@canonical.com>
17− */
18−
19−#ifndef UNITY_ACCOUNTSBINDINGS_H
20−#define UNITY_ACCOUNTSBINDINGS_H
21−
22−#include <QObject>
23−#include <QString>
24−
25−class AccountsService;
26−
27−class AccountsBindings: public QObject
28−{
29− Q_OBJECT
30− Q_PROPERTY (QString user
31− READ getUser
32− WRITE setUser
33− NOTIFY userChanged)
34− Q_PROPERTY (bool demoEdges
35− READ getDemoEdges
36− WRITE setDemoEdges
37− NOTIFY demoEdgesChanged)
38− Q_PROPERTY (bool demoEdgesForCurrentUser
39− READ getDemoEdgesForCurrentUser
40− WRITE setDemoEdgesForCurrentUser
41− NOTIFY demoEdgesForCurrentUserChanged)
42− Q_PROPERTY (QString backgroundFile
43− READ getBackgroundFile
44− NOTIFY backgroundFileChanged)
45−
46−public:
47− explicit AccountsBindings(QObject *parent = 0);
48−
49− QString getUser();
50− void setUser(const QString &user);
51− bool getDemoEdges();
52− void setDemoEdges(bool demoEdges);
53− bool getDemoEdgesForCurrentUser();
54− void setDemoEdgesForCurrentUser(bool demoEdgesForCurrentUser);
55−
56− QString getBackgroundFile();
57−
58−Q_SIGNALS:
59− void userChanged();
60− void demoEdgesChanged();
61− void demoEdgesForCurrentUserChanged();
62− void backgroundFileChanged();
63−
64−private Q_SLOTS:
65− void propertiesChanged(const QString &user, const QString &interface, const QStringList &changed);
66− void maybeChanged(const QString &user);
67−
68−private:
69− void updateDemoEdges();
70− void updateDemoEdgesForCurrentUser();
71− void updateBackgroundFile();
72−
73− AccountsService *m_service;
74− QString m_user;
75− bool m_demoEdges;
76− bool m_demoEdgesForCurrentUser;
77− QString m_backgroundFile;
78−};
79−
80−#endif
plugins/AccountsService/AccountsService.cpp+79−80
@@ -17,110 +17,109 @@
1717 */
1818
1919#include "AccountsService.h"
20−#include <QDBusConnection>
21−#include <QDBusConnectionInterface>
22−#include <QDBusMessage>
23−#include <QDBusVariant>
20+#include "AccountsServiceDBusAdaptor.h"
21+
22+#include <QStringList>
2423
2524AccountsService::AccountsService(QObject* parent)
2625 : QObject(parent),
27− accounts_manager(NULL),
28− users()
26+ m_service(new AccountsServiceDBusAdaptor(this))
2927{
30− auto connection = QDBusConnection::SM_BUSNAME();
31− auto interface = connection.interface();
32− interface->startService("org.freedesktop.Accounts");
33− accounts_manager = new QDBusInterface("org.freedesktop.Accounts",
34− "/org/freedesktop/Accounts",
35− "org.freedesktop.Accounts",
36− connection, this);
28+ connect(m_service, SIGNAL(propertiesChanged(const QString &, const QString &, const QStringList &)),
29+ this, SLOT(propertiesChanged(const QString &, const QString &, const QStringList &)));
30+ connect(m_service, SIGNAL(maybeChanged(const QString &)),
31+ this, SLOT(maybeChanged(const QString &)));
32+
33+ setUser(qgetenv("USER"));
34+ updateDemoEdgesForCurrentUser();
3735}
3836
39−QVariant AccountsService::getUserProperty(const QString &user, const QString &interface, const QString &property)
37+QString AccountsService::getUser()
4038{
41− auto iface = getUserInterface(user);
42− if (iface != nullptr && iface->isValid()) {
43− auto answer = iface->call("Get", interface, property);
44− if (answer.type() == QDBusMessage::ReplyMessage) {
45− return answer.arguments()[0].value<QDBusVariant>().variant();
46− }
47− }
48− return QVariant();
39+ return m_user;
4940}
5041
51−void AccountsService::setUserProperty(const QString &user, const QString &interface, const QString &property, const QVariant &value)
42+void AccountsService::setUser(const QString &user)
5243{
53− auto iface = getUserInterface(user);
54− if (iface != nullptr && iface->isValid()) {
55− // The value needs to be carefully wrapped
56− iface->call("Set", interface, property, QVariant::fromValue(QDBusVariant(value)));
57− }
44+ m_user = user;
45+ Q_EMIT userChanged();
46+
47+ updateDemoEdges();
48+ updateBackgroundFile();
49+}
50+
51+bool AccountsService::getDemoEdges()
52+{
53+ return m_demoEdges;
5854}
5955
60−void AccountsService::propertiesChangedSlot(const QString &interface, const QVariantMap &changed, const QStringList &invalid)
56+void AccountsService::setDemoEdges(bool demoEdges)
6157{
62− // Merge changed and invalidated together
63− QStringList combined;
64− combined << invalid;
65− combined << changed.keys();
66− combined.removeDuplicates();
58+ m_demoEdges = demoEdges;
59+ m_service->setUserProperty(m_user, "com.canonical.unity.AccountsService", "demo-edges", demoEdges);
60+}
6761
68− Q_EMIT propertiesChanged(getUserForPath(message().path()), interface, combined);
62+bool AccountsService::getDemoEdgesForCurrentUser()
63+{
64+ return m_demoEdgesForCurrentUser;
6965}
7066
71−void AccountsService::maybeChangedSlot()
67+void AccountsService::setDemoEdgesForCurrentUser(bool demoEdgesForCurrentUser)
7268{
73− Q_EMIT maybeChanged(getUserForPath(message().path()));
69+ m_demoEdgesForCurrentUser = demoEdgesForCurrentUser;
70+ m_service->setUserProperty(qgetenv("USER"), "com.canonical.unity.AccountsService", "demo-edges", demoEdgesForCurrentUser);
7471}
7572
76−QString AccountsService::getUserForPath(const QString &path)
73+QString AccountsService::getBackgroundFile()
7774{
78− QMap<QString, QDBusInterface *>::const_iterator i;
79− for (i = users.constBegin(); i != users.constEnd(); ++i) {
80− if (i.value()->path() == path) {
81− return i.key();
82− }
75+ return m_backgroundFile;
76+}
77+
78+void AccountsService::updateDemoEdges()
79+{
80+ auto demoEdges = m_service->getUserProperty(m_user, "com.canonical.unity.AccountsService", "demo-edges").toBool();
81+ if (m_demoEdges != demoEdges) {
82+ m_demoEdges = demoEdges;
83+ Q_EMIT demoEdgesChanged();
8384 }
84− return QString();
8585}
8686
87−QDBusInterface *AccountsService::getUserInterface(const QString &user)
87+void AccountsService::updateDemoEdgesForCurrentUser()
8888{
89− auto iface = users.value(user);
90− if (iface == nullptr && accounts_manager->isValid()) {
91− auto answer = accounts_manager->call("FindUserByName", user);
92− if (answer.type() == QDBusMessage::ReplyMessage) {
93− auto path = answer.arguments()[0].value<QDBusObjectPath>().path();
94−
95− iface = new QDBusInterface("org.freedesktop.Accounts",
96− path,
97− "org.freedesktop.DBus.Properties",
98− accounts_manager->connection(), this);
99−
100− // With its own pre-defined properties, AccountsService is oddly
101− // close-lipped. It won't send out proper DBus.Properties notices,
102− // but it does have one catch-all Changed() signal. So let's
103− // listen to that.
104− iface->connection().connect(
105− iface->service(),
106− path,
107− "org.freedesktop.Accounts.User",
108− "Changed",
109− this,
110− SLOT(maybeChangedSlot()));
111−
112− // But custom properties do send out the right notifications, so
113− // let's still listen there.
114− iface->connection().connect(
115− iface->service(),
116− path,
117− "org.freedesktop.DBus.Properties",
118− "PropertiesChanged",
119− this,
120− SLOT(propertiesChangedSlot(QString, QVariantMap, QStringList)));
121−
122− users.insert(user, iface);
89+ auto demoEdgesForCurrentUser = m_service->getUserProperty(qgetenv("USER"), "com.canonical.unity.AccountsService", "demo-edges").toBool();
90+ if (m_demoEdgesForCurrentUser != demoEdgesForCurrentUser) {
91+ m_demoEdgesForCurrentUser = demoEdgesForCurrentUser;
92+ Q_EMIT demoEdgesForCurrentUserChanged();
93+ }
94+}
95+
96+void AccountsService::updateBackgroundFile()
97+{
98+ auto backgroundFile = m_service->getUserProperty(m_user, "org.freedesktop.Accounts.User", "BackgroundFile").toString();
99+ if (m_backgroundFile != backgroundFile) {
100+ m_backgroundFile = backgroundFile;
101+ Q_EMIT backgroundFileChanged();
102+ }
103+}
104+
105+void AccountsService::propertiesChanged(const QString &user, const QString &interface, const QStringList &changed)
106+{
107+ if (interface == "com.canonical.unity.AccountsService") {
108+ if (changed.contains("demo-edges")) {
109+ if (qgetenv("USER") == user) {
110+ updateDemoEdgesForCurrentUser();
111+ }
112+ if (m_user != user) {
113+ updateDemoEdges();
114+ }
123115 }
124116 }
125− return iface;
117+}
118+
119+void AccountsService::maybeChanged(const QString &user)
120+{
121+ if (m_user == user) {
122+ // Standard properties might have changed
123+ updateBackgroundFile();
124+ }
126125}
plugins/AccountsService/AccountsService.h+40−15
@@ -1,5 +1,5 @@
11/*
2− * Copyright (C) 2012,2013 Canonical, Ltd.
2+ * Copyright (C) 2013 Canonical, Ltd.
33 *
44 * This program is free software; you can redistribute it and/or modify
55 * it under the terms of the GNU General Public License as published by
@@ -19,36 +19,61 @@
1919#ifndef UNITY_ACCOUNTSSERVICE_H
2020#define UNITY_ACCOUNTSSERVICE_H
2121
22−#include <QDBusContext>
23−#include <QDBusInterface>
24−#include <QMap>
2522#include <QObject>
2623#include <QString>
2724
28−class AccountsService: public QObject, public QDBusContext
25+class AccountsServiceDBusAdaptor;
26+
27+class AccountsService: public QObject
2928{
3029 Q_OBJECT
30+ Q_PROPERTY (QString user
31+ READ getUser
32+ WRITE setUser
33+ NOTIFY userChanged)
34+ Q_PROPERTY (bool demoEdges
35+ READ getDemoEdges
36+ WRITE setDemoEdges
37+ NOTIFY demoEdgesChanged)
38+ Q_PROPERTY (bool demoEdgesForCurrentUser
39+ READ getDemoEdgesForCurrentUser
40+ WRITE setDemoEdgesForCurrentUser
41+ NOTIFY demoEdgesForCurrentUserChanged)
42+ Q_PROPERTY (QString backgroundFile
43+ READ getBackgroundFile
44+ NOTIFY backgroundFileChanged)
3145
3246public:
3347 explicit AccountsService(QObject *parent = 0);
3448
35− Q_INVOKABLE QVariant getUserProperty(const QString &user, const QString &interface, const QString &property);
36− Q_INVOKABLE void setUserProperty(const QString &user, const QString &interface, const QString &property, const QVariant &value);
49+ QString getUser();
50+ void setUser(const QString &user);
51+ bool getDemoEdges();
52+ void setDemoEdges(bool demoEdges);
53+ bool getDemoEdgesForCurrentUser();
54+ void setDemoEdgesForCurrentUser(bool demoEdgesForCurrentUser);
55+ QString getBackgroundFile();
3756
3857Q_SIGNALS:
39− void propertiesChanged(const QString &user, const QString &interface, const QStringList &changed);
40− void maybeChanged(const QString &user); // Standard properties might have changed
58+ void userChanged();
59+ void demoEdgesChanged();
60+ void demoEdgesForCurrentUserChanged();
61+ void backgroundFileChanged();
4162
4263private Q_SLOTS:
43− void propertiesChangedSlot(const QString &interface, const QVariantMap &changed, const QStringList &invalid);
44− void maybeChangedSlot();
64+ void propertiesChanged(const QString &user, const QString &interface, const QStringList &changed);
65+ void maybeChanged(const QString &user);
4566
4667private:
47− QDBusInterface *getUserInterface(const QString &user);
48− QString getUserForPath(const QString &path);
68+ void updateDemoEdges();
69+ void updateDemoEdgesForCurrentUser();
70+ void updateBackgroundFile();
4971
50− QDBusInterface *accounts_manager;
51− QMap<QString, QDBusInterface *> users;
72+ AccountsServiceDBusAdaptor *m_service;
73+ QString m_user;
74+ bool m_demoEdges;
75+ bool m_demoEdgesForCurrentUser;
76+ QString m_backgroundFile;
5277};
5378
5479#endif
plugins/AccountsService/AccountsServiceDBusAdaptor.cppnew file+126
@@ -0,0 +1,126 @@
1+/*
2+ * Copyright (C) 2013 Canonical, Ltd.
3+ *
4+ * This program is free software; you can redistribute it and/or modify
5+ * it under the terms of the GNU General Public License as published by
6+ * the Free Software Foundation; version 3.
7+ *
8+ * This program is distributed in the hope that it will be useful,
9+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
10+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11+ * GNU General Public License for more details.
12+ *
13+ * You should have received a copy of the GNU General Public License
14+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
15+ *
16+ * Author: Michael Terry <michael.terry@canonical.com>
17+ */
18+
19+#include "AccountsServiceDBusAdaptor.h"
20+#include <QDBusConnection>
21+#include <QDBusConnectionInterface>
22+#include <QDBusMessage>
23+#include <QDBusVariant>
24+
25+AccountsServiceDBusAdaptor::AccountsServiceDBusAdaptor(QObject* parent)
26+ : QObject(parent),
27+ accounts_manager(NULL),
28+ users()
29+{
30+ auto connection = QDBusConnection::SM_BUSNAME();
31+ auto interface = connection.interface();
32+ interface->startService("org.freedesktop.Accounts");
33+ accounts_manager = new QDBusInterface("org.freedesktop.Accounts",
34+ "/org/freedesktop/Accounts",
35+ "org.freedesktop.Accounts",
36+ connection, this);
37+}
38+
39+QVariant AccountsServiceDBusAdaptor::getUserProperty(const QString &user, const QString &interface, const QString &property)
40+{
41+ auto iface = getUserInterface(user);
42+ if (iface != nullptr && iface->isValid()) {
43+ auto answer = iface->call("Get", interface, property);
44+ if (answer.type() == QDBusMessage::ReplyMessage) {
45+ return answer.arguments()[0].value<QDBusVariant>().variant();
46+ }
47+ }
48+ return QVariant();
49+}
50+
51+void AccountsServiceDBusAdaptor::setUserProperty(const QString &user, const QString &interface, const QString &property, const QVariant &value)
52+{
53+ auto iface = getUserInterface(user);
54+ if (iface != nullptr && iface->isValid()) {
55+ // The value needs to be carefully wrapped
56+ iface->call("Set", interface, property, QVariant::fromValue(QDBusVariant(value)));
57+ }
58+}
59+
60+void AccountsServiceDBusAdaptor::propertiesChangedSlot(const QString &interface, const QVariantMap &changed, const QStringList &invalid)
61+{
62+ // Merge changed and invalidated together
63+ QStringList combined;
64+ combined << invalid;
65+ combined << changed.keys();
66+ combined.removeDuplicates();
67+
68+ Q_EMIT propertiesChanged(getUserForPath(message().path()), interface, combined);
69+}
70+
71+void AccountsServiceDBusAdaptor::maybeChangedSlot()
72+{
73+ Q_EMIT maybeChanged(getUserForPath(message().path()));
74+}
75+
76+QString AccountsServiceDBusAdaptor::getUserForPath(const QString &path)
77+{
78+ QMap<QString, QDBusInterface *>::const_iterator i;
79+ for (i = users.constBegin(); i != users.constEnd(); ++i) {
80+ if (i.value()->path() == path) {
81+ return i.key();
82+ }
83+ }
84+ return QString();
85+}
86+
87+QDBusInterface *AccountsServiceDBusAdaptor::getUserInterface(const QString &user)
88+{
89+ auto iface = users.value(user);
90+ if (iface == nullptr && accounts_manager->isValid()) {
91+ auto answer = accounts_manager->call("FindUserByName", user);
92+ if (answer.type() == QDBusMessage::ReplyMessage) {
93+ auto path = answer.arguments()[0].value<QDBusObjectPath>().path();
94+
95+ iface = new QDBusInterface("org.freedesktop.Accounts",
96+ path,
97+ "org.freedesktop.DBus.Properties",
98+ accounts_manager->connection(), this);
99+
100+ // With its own pre-defined properties, AccountsService is oddly
101+ // close-lipped. It won't send out proper DBus.Properties notices,
102+ // but it does have one catch-all Changed() signal. So let's
103+ // listen to that.
104+ iface->connection().connect(
105+ iface->service(),
106+ path,
107+ "org.freedesktop.Accounts.User",
108+ "Changed",
109+ this,
110+ SLOT(maybeChangedSlot()));
111+
112+ // But custom properties do send out the right notifications, so
113+ // let's still listen there.
114+ iface->connection().connect(
115+ iface->service(),
116+ path,
117+ "org.freedesktop.DBus.Properties",
118+ "PropertiesChanged",
119+ this,
120+ SLOT(propertiesChangedSlot(QString, QVariantMap, QStringList)));
121+
122+ users.insert(user, iface);
123+ }
124+ }
125+ return iface;
126+}
plugins/AccountsService/AccountsServiceDBusAdaptor.hnew file+54
@@ -0,0 +1,54 @@
1+/*
2+ * Copyright (C) 2013 Canonical, Ltd.
3+ *
4+ * This program is free software; you can redistribute it and/or modify
5+ * it under the terms of the GNU General Public License as published by
6+ * the Free Software Foundation; version 3.
7+ *
8+ * This program is distributed in the hope that it will be useful,
9+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
10+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11+ * GNU General Public License for more details.
12+ *
13+ * You should have received a copy of the GNU General Public License
14+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
15+ *
16+ * Authors: Michael Terry <michael.terry@canonical.com>
17+ */
18+
19+#ifndef UNITY_ACCOUNTSSERVICEDBUSADAPTOR_H
20+#define UNITY_ACCOUNTSSERVICEDBUSADAPTOR_H
21+
22+#include <QDBusContext>
23+#include <QDBusInterface>
24+#include <QMap>
25+#include <QObject>
26+#include <QString>
27+
28+class AccountsServiceDBusAdaptor: public QObject, public QDBusContext
29+{
30+ Q_OBJECT
31+
32+public:
33+ explicit AccountsServiceDBusAdaptor(QObject *parent = 0);
34+
35+ Q_INVOKABLE QVariant getUserProperty(const QString &user, const QString &interface, const QString &property);
36+ Q_INVOKABLE void setUserProperty(const QString &user, const QString &interface, const QString &property, const QVariant &value);
37+
38+Q_SIGNALS:
39+ void propertiesChanged(const QString &user, const QString &interface, const QStringList &changed);
40+ void maybeChanged(const QString &user); // Standard properties might have changed
41+
42+private Q_SLOTS:
43+ void propertiesChangedSlot(const QString &interface, const QVariantMap &changed, const QStringList &invalid);
44+ void maybeChangedSlot();
45+
46+private:
47+ QDBusInterface *getUserInterface(const QString &user);
48+ QString getUserForPath(const QString &path);
49+
50+ QDBusInterface *accounts_manager;
51+ QMap<QString, QDBusInterface *> users;
52+};
53+
54+#endif
plugins/AccountsService/CMakeLists.txt+1−1
@@ -3,8 +3,8 @@ include(Plugins)
33add_definitions(-DSM_BUSNAME=systemBus)
44
55add_library(AccountsService-qml MODULE
6− AccountsBindings.cpp
76 AccountsService.cpp
7+ AccountsServiceDBusAdaptor.cpp
88 plugin.cpp
99 )
1010
plugins/AccountsService/plugin.cpp+3−3
@@ -17,7 +17,7 @@
1717 */
1818
1919#include "plugin.h"
20−#include "AccountsBindings.h"
20+#include "AccountsService.h"
2121
2222#include <QDBusMetaType>
2323#include <QtQml>
@@ -26,12 +26,12 @@ static QObject *service_provider(QQmlEngine *engine, QJSEngine *scriptEngine)
2626{
2727 Q_UNUSED(engine)
2828 Q_UNUSED(scriptEngine)
29− return new AccountsBindings();
29+ return new AccountsService();
3030}
3131
3232void AccountsServicePlugin::registerTypes(const char *uri)
3333{
3434 Q_ASSERT(uri == QLatin1String("AccountsService"));
3535 qDBusRegisterMetaType<QList<QVariantMap>>();
36− qmlRegisterSingletonType<AccountsBindings>(uri, 0, 1, "AccountsService", service_provider);
36+ qmlRegisterSingletonType<AccountsService>(uri, 0, 1, "AccountsService", service_provider);
3737}
plugins/Unity/Launcher/CMakeLists.txt+1−1
@@ -17,7 +17,7 @@ set(QMLLAUNCHERPLUGIN_SRC
1717 quicklistmodel.cpp
1818 common/quicklistentry.cpp
1919 backend/launcherbackend.cpp
20− ${CMAKE_SOURCE_DIR}/plugins/AccountsService/AccountsService.cpp
20+ ${CMAKE_SOURCE_DIR}/plugins/AccountsService/AccountsServiceDBusAdaptor.cpp
2121 ${LAUNCHER_API_INCLUDEDIR}/unity/shell/launcher/LauncherItemInterface.h
2222 ${LAUNCHER_API_INCLUDEDIR}/unity/shell/launcher/LauncherModelInterface.h
2323 ${LAUNCHER_API_INCLUDEDIR}/unity/shell/launcher/QuickListModelInterface.h
plugins/Unity/Launcher/backend/launcherbackend.cpp+2−2
@@ -18,7 +18,7 @@
1818 * along with this program. If not, see <http://www.gnu.org/licenses/>.
1919 */
2020
21−#include "AccountsService.h"
21+#include "AccountsServiceDBusAdaptor.h"
2222#include "launcherbackend.h"
2323
2424#include <QDir>
@@ -43,7 +43,7 @@ LauncherBackend::LauncherBackend(bool useStorage, QObject *parent):
4343 m_accounts(nullptr)
4444{
4545 if (useStorage) {
46− m_accounts = new AccountsService(this);
46+ m_accounts = new AccountsServiceDBusAdaptor(this);
4747 }
4848 m_user = qgetenv("USER");
4949 syncFromAccounts();
plugins/Unity/Launcher/backend/launcherbackend.h+2−2
@@ -25,7 +25,7 @@
2525#include <QSettings>
2626#include <QStringList>
2727
28−class AccountsService;
28+class AccountsServiceDBusAdaptor;
2929
3030/**
3131 * @brief An interface that provides all the data needed by the launcher.
@@ -156,7 +156,7 @@ private:
156156 QList<QString> m_storedApps;
157157 mutable QHash<QString, LauncherBackendItem*> m_itemCache;
158158
159− AccountsService *m_accounts;
159+ AccountsServiceDBusAdaptor *m_accounts;
160160 QString m_user;
161161};
162162
tests/mocks/AccountsService/AccountsBindings.cpp → tests/mocks/AccountsService/AccountsService.cpprenamed+11−9
@@ -16,41 +16,43 @@
1616 * Author: Michael Terry <michael.terry@canonical.com>
1717 */
1818
19−#include "AccountsBindings.h"
19+#include "AccountsService.h"
2020
21−AccountsBindings::AccountsBindings(QObject* parent)
21+AccountsService::AccountsService(QObject* parent)
2222 : QObject(parent)
2323{
2424}
2525
26−QString AccountsBindings::getUser()
26+QString AccountsService::getUser()
2727{
2828 return "testuser";
2929}
3030
31−void AccountsBindings::setUser(const QString &)
31+void AccountsService::setUser(const QString &user)
3232{
33+ Q_UNUSED(user)
3334}
3435
35−bool AccountsBindings::getDemoEdges()
36+bool AccountsService::getDemoEdges()
3637{
3738 return false;
3839}
3940
40−void AccountsBindings::setDemoEdges(bool)
41+void AccountsService::setDemoEdges(bool demoEdges)
4142{
43+ Q_UNUSED(demoEdges)
4244}
4345
44−bool AccountsBindings::getDemoEdgesForCurrentUser()
46+bool AccountsService::getDemoEdgesForCurrentUser()
4547{
4648 return false;
4749}
4850
49−void AccountsBindings::setDemoEdgesForCurrentUser(bool)
51+void AccountsService::setDemoEdgesForCurrentUser(bool)
5052{
5153}
5254
53−QString AccountsBindings::getBackgroundFile()
55+QString AccountsService::getBackgroundFile()
5456{
5557 return TOP_SRCDIR "/graphics/phone_background.jpg";
5658}
tests/mocks/AccountsService/AccountsBindings.h → tests/mocks/AccountsService/AccountsService.hrenamed+5−5
@@ -1,5 +1,5 @@
11/*
2− * Copyright (C) 2012,2013 Canonical, Ltd.
2+ * Copyright (C) 2013 Canonical, Ltd.
33 *
44 * This program is free software; you can redistribute it and/or modify
55 * it under the terms of the GNU General Public License as published by
@@ -17,14 +17,14 @@
1717 * Michael Terry <michael.terry@canonical.com>
1818 */
1919
20−#ifndef UNITY_MOCK_ACCOUNTSBINDINGS_H
21−#define UNITY_MOCK_ACCOUNTSBINDINGS_H
20+#ifndef UNITY_MOCK_ACCOUNTSSERVICE_H
21+#define UNITY_MOCK_ACCOUNTSSERVICE_H
2222
2323#include <QObject>
2424#include <QString>
2525#include <QVariant>
2626
27−class AccountsBindings: public QObject
27+class AccountsService: public QObject
2828{
2929 Q_OBJECT
3030 Q_PROPERTY (QString user
@@ -44,7 +44,7 @@ class AccountsBindings: public QObject
4444 NOTIFY backgroundFileChanged)
4545
4646public:
47− explicit AccountsBindings(QObject *parent = 0);
47+ explicit AccountsService(QObject *parent = 0);
4848
4949 QString getUser();
5050 void setUser(const QString &user);
tests/mocks/AccountsService/CMakeLists.txt+1−1
@@ -1,7 +1,7 @@
11add_definitions(-DTOP_SRCDIR="${CMAKE_SOURCE_DIR}")
22
33add_library(MockAccountsService-qml MODULE
4− AccountsBindings.cpp
4+ AccountsService.cpp
55 plugin.cpp
66 )
77
tests/mocks/AccountsService/plugin.cpp+3−3
@@ -18,7 +18,7 @@
1818 */
1919
2020#include "plugin.h"
21−#include "AccountsBindings.h"
21+#include "AccountsService.h"
2222
2323#include <QtQml>
2424
@@ -26,11 +26,11 @@ static QObject *service_provider(QQmlEngine *engine, QJSEngine *scriptEngine)
2626{
2727 Q_UNUSED(engine)
2828 Q_UNUSED(scriptEngine)
29− return new AccountsBindings();
29+ return new AccountsService();
3030}
3131
3232void AccountsServicePlugin::registerTypes(const char *uri)
3333{
3434 Q_ASSERT(uri == QLatin1String("AccountsService"));
35− qmlRegisterSingletonType<AccountsBindings>(uri, 0, 1, "AccountsService", service_provider);
35+ qmlRegisterSingletonType<AccountsService>(uri, 0, 1, "AccountsService", service_provider);
3636}
tests/plugins/AccountsService/CMakeLists.txt+1−1
@@ -25,7 +25,7 @@ qt5_use_modules(mock-server Core DBus)
2525
2626add_executable(test-accountsservice
2727 ${CMAKE_SOURCE_DIR}/plugins/AccountsService/AccountsService.cpp
28− ${CMAKE_SOURCE_DIR}/plugins/AccountsService/AccountsBindings.cpp
28+ ${CMAKE_SOURCE_DIR}/plugins/AccountsService/AccountsServiceDBusAdaptor.cpp
2929 client.cpp
3030 )
3131qt5_use_modules(test-accountsservice Core DBus Test)
tests/plugins/AccountsService/client.cpp+11−11
@@ -17,8 +17,8 @@
1717 * Authored by: Michael Terry <michael.terry@canonical.com>
1818 */
1919
20−#include "AccountsBindings.h"
2120#include "AccountsService.h"
21+#include "AccountsServiceDBusAdaptor.h"
2222#include <QSignalSpy>
2323#include <QTest>
2424
@@ -31,28 +31,28 @@ private Q_SLOTS:
3131 void testInvalids()
3232 {
3333 // Test various invalid calls
34− AccountsService session;
34+ AccountsServiceDBusAdaptor session;
3535 QCOMPARE(session.getUserProperty("NOPE", "com.canonical.unity.AccountsService", "demo-edges"), QVariant());
3636 QCOMPARE(session.getUserProperty("testuser", "com.canonical.unity.AccountsService", "NOPE"), QVariant());
3737 }
3838
39− void testGetSetService()
39+ void testGetSetServiceDBusAdaptor()
4040 {
41− AccountsService session;
41+ AccountsServiceDBusAdaptor session;
4242 session.setUserProperty("testuser", "com.canonical.unity.AccountsService", "demo-edges", QVariant(true));
4343 QCOMPARE(session.getUserProperty("testuser", "com.canonical.unity.AccountsService", "demo-edges"), QVariant(true));
4444 session.setUserProperty("testuser", "com.canonical.unity.AccountsService", "demo-edges", QVariant(false));
4545 QCOMPARE(session.getUserProperty("testuser", "com.canonical.unity.AccountsService", "demo-edges"), QVariant(false));
4646 }
4747
48− void testGetSetBindings()
48+ void testGetSetService()
4949 {
50− AccountsBindings bindings;
51− bindings.setUser("testuser");
52− bindings.setDemoEdges(true);
53− QCOMPARE(bindings.getDemoEdges(), true);
54− bindings.setDemoEdges(false);
55− QCOMPARE(bindings.getDemoEdges(), false);
50+ AccountsService session;
51+ session.setUser("testuser");
52+ session.setDemoEdges(true);
53+ QCOMPARE(session.getDemoEdges(), true);
54+ session.setDemoEdges(false);
55+ QCOMPARE(session.getDemoEdges(), false);
5656 }
5757};
5858
tests/plugins/Unity/Launcher/CMakeLists.txt+2−2
@@ -21,7 +21,7 @@ add_test(NAME launcherbackendtest COMMAND ${testBackendCommand})
2121add_custom_target(launcherbackendtest ${testBackendCommand})
2222add_executable(launcherbackendtestExec
2323 launcherbackendtest.cpp
24− ${CMAKE_SOURCE_DIR}/plugins/AccountsService/AccountsService.cpp
24+ ${CMAKE_SOURCE_DIR}/plugins/AccountsService/AccountsServiceDBusAdaptor.cpp
2525 ${CMAKE_SOURCE_DIR}/plugins/Unity/Launcher/backend/launcherbackend.cpp
2626 ${CMAKE_SOURCE_DIR}/plugins/Unity/Launcher/common/quicklistentry.cpp
2727 )
@@ -39,7 +39,7 @@ add_executable(launchermodeltestExec
3939 ${CMAKE_SOURCE_DIR}/plugins/Unity/Launcher/launchermodel.cpp
4040 ${CMAKE_SOURCE_DIR}/plugins/Unity/Launcher/launcheritem.cpp
4141 ${CMAKE_SOURCE_DIR}/plugins/Unity/Launcher/quicklistmodel.cpp
42− ${CMAKE_SOURCE_DIR}/plugins/AccountsService/AccountsService.cpp
42+ ${CMAKE_SOURCE_DIR}/plugins/AccountsService/AccountsServiceDBusAdaptor.cpp
4343 ${CMAKE_SOURCE_DIR}/plugins/Unity/Launcher/backend/launcherbackend.cpp
4444 ${CMAKE_SOURCE_DIR}/plugins/Unity/Launcher/common/quicklistentry.cpp
4545 ${LAUNCHER_API_INCLUDEDIR}/unity/shell/launcher/LauncherItemInterface.h