Merge lp:~dbarth/ubuntu-system-settings/rtm-fix-1378678 into lp:ubuntu-system-settings/rtm-14.09

Proposed by David Barth
Status: Rejected
Rejected by: Sebastien Bacher
Proposed branch: lp:~dbarth/ubuntu-system-settings/rtm-fix-1378678
Merge into: lp:ubuntu-system-settings/rtm-14.09
Diff against target: 76 lines (+15/-0)
5 files modified
plugins/system-update/PageComponent.qml (+5/-0)
plugins/system-update/update_manager.cpp (+4/-0)
plugins/system-update/update_manager.h (+1/-0)
tests/plugins/system-update/fakessoservice.cpp (+2/-0)
tests/plugins/system-update/fakessoservice.h (+3/-0)
To merge this branch: bzr merge lp:~dbarth/ubuntu-system-settings/rtm-fix-1378678
Reviewer Review Type Date Requested Status
Sebastien Bacher (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+251055@code.launchpad.net

This proposal supersedes a proposal from 2015-02-26.

Commit message

cherry picking the fix for bug #1378678 (aka bug #1413655), ie prompting the user when older u1 accounts are deleted by the system

Description of the change

cherry picking the fix for bug #1378678 (aka bug #1413655), ie prompting the user when older u1 accounts are deleted by the system

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Sebastien Bacher (seb128) wrote :

That's a cherry pick from trunk r1324, looks fine to me, thanks

review: Approve
Revision history for this message
Sebastien Bacher (seb128) wrote :

seems like we are not doing more rtm landing, so cleaning the list a bit, that one is rather going to be including with the vivid rebase

Unmerged revisions

998. By David Barth

cherry picking the fix for bug #1378678, ie prompting the user when older u1 accounts are deleted by the system

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/system-update/PageComponent.qml'
2--- plugins/system-update/PageComponent.qml 2015-01-30 18:36:21 +0000
3+++ plugins/system-update/PageComponent.qml 2015-02-26 10:50:36 +0000
4@@ -199,6 +199,11 @@
5 credentialsNotification.visible = true;
6 }
7
8+ onCredentialsDeleted: {
9+ credentialsNotification.visible = false;
10+ uoaConfig.exec();
11+ }
12+
13 onSystemUpdateDownloaded: {
14 root.installAll = false;
15 }
16
17=== modified file 'plugins/system-update/update_manager.cpp'
18--- plugins/system-update/update_manager.cpp 2014-12-16 15:38:34 +0000
19+++ plugins/system-update/update_manager.cpp 2015-02-26 10:50:36 +0000
20@@ -63,6 +63,10 @@
21 this, SIGNAL(credentialsNotFound()));
22 QObject::connect(&m_service, SIGNAL(credentialsNotFound()),
23 this, SLOT(clickUpdateNotAvailable()));
24+ QObject::connect(&m_service, SIGNAL(credentialsDeleted()),
25+ this, SIGNAL(credentialsDeleted()));
26+ QObject::connect(&m_service, SIGNAL(credentialsDeleted()),
27+ this, SLOT(clickUpdateNotAvailable()));
28 // PROCESS
29 QObject::connect(&m_process, SIGNAL(finished(int)),
30 this, SLOT(processOutput()));
31
32=== modified file 'plugins/system-update/update_manager.h'
33--- plugins/system-update/update_manager.h 2014-12-10 14:25:19 +0000
34+++ plugins/system-update/update_manager.h 2015-02-26 10:50:36 +0000
35@@ -68,6 +68,7 @@
36 void modelChanged();
37 void updatesNotFound();
38 void credentialsNotFound();
39+ void credentialsDeleted();
40 void updateAvailableFound(bool downloading);
41 void errorFound();
42 void networkError();
43
44=== modified file 'tests/plugins/system-update/fakessoservice.cpp'
45--- tests/plugins/system-update/fakessoservice.cpp 2014-05-07 15:37:22 +0000
46+++ tests/plugins/system-update/fakessoservice.cpp 2015-02-26 10:50:36 +0000
47@@ -13,6 +13,8 @@
48 if(m_validCredentials) {
49 Token token("token_key", "token_secret", "consumer_key", "consumer_secret");
50 emit this->credentialsFound(token);
51+ } else if (m_deletedCredentials) {
52+ emit this->credentialsDeleted();
53 } else {
54 emit this->credentialsNotFound();
55 }
56
57=== modified file 'tests/plugins/system-update/fakessoservice.h'
58--- tests/plugins/system-update/fakessoservice.h 2014-05-05 18:18:37 +0000
59+++ tests/plugins/system-update/fakessoservice.h 2015-02-26 10:50:36 +0000
60@@ -17,13 +17,16 @@
61 void getCredentials();
62
63 void setValidCredentials(bool value) { m_validCredentials = value; }
64+ void setDeletedCredentials(bool value) { m_deletedCredentials = value; }
65
66 signals:
67 void credentialsFound(const Token&);
68 void credentialsNotFound();
69+ void credentialsDeleted();
70
71 private:
72 bool m_validCredentials;
73+ bool m_deletedCredentials;
74 };
75
76 }

Subscribers

People subscribed via source and target branches