Merge lp:~alecu/ubuntu-system-settings/not-call-but-signal into lp:ubuntu-system-settings

Proposed by Alejandro J. Cura
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 1020
Merged at revision: 1026
Proposed branch: lp:~alecu/ubuntu-system-settings/not-call-but-signal
Merge into: lp:ubuntu-system-settings
Diff against target: 24 lines (+6/-8)
1 file modified
plugins/system-update/update_manager.cpp (+6/-8)
To merge this branch: bzr merge lp:~alecu/ubuntu-system-settings/not-call-but-signal
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Sebastien Bacher (community) Approve
dobey (community) Approve
Review via email: mp+234202@code.launchpad.net

Commit message

Fixing the refreshing of the apps scope

To post a comment you must log in.
Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Sebastien Bacher (seb128) wrote :

thanks

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/system-update/update_manager.cpp'
2--- plugins/system-update/update_manager.cpp 2014-09-04 15:46:30 +0000
3+++ plugins/system-update/update_manager.cpp 2014-09-10 20:04:31 +0000
4@@ -291,14 +291,12 @@
5 void UpdateManager::updateClickScope()
6 {
7 // Refresh click scope
8- QDBusInterface iface("com.canonical.unity.scopes",
9- "/com/canonical/unity/scopes",
10- "com.canonical.unity.scopes",
11- QDBusConnection::sessionBus(), 0);
12-
13- if (iface.isValid()) {
14- iface.call(QLatin1String("InvalidateResults"), QLatin1String("clickscope"));
15- }
16+ QDBusMessage signal = QDBusMessage::createSignal(
17+ "/com/canonical/unity/scopes",
18+ "com.canonical.unity.scopes",
19+ "InvalidateResults");
20+ signal << "clickscope";
21+ QDBusConnection::sessionBus().send(signal);
22 }
23
24 }

Subscribers

People subscribed via source and target branches