Merge lp:~mardy/ubuntu-system-settings-online-accounts/request-destroyed-1517253 into lp:ubuntu-system-settings-online-accounts

Proposed by Alberto Mardegan
Status: Merged
Approved by: David Barth
Approved revision: 312
Merged at revision: 318
Proposed branch: lp:~mardy/ubuntu-system-settings-online-accounts/request-destroyed-1517253
Merge into: lp:ubuntu-system-settings-online-accounts
Diff against target: 12 lines (+2/-0)
1 file modified
online-accounts-ui/provider-request.cpp (+2/-0)
To merge this branch: bzr merge lp:~mardy/ubuntu-system-settings-online-accounts/request-destroyed-1517253
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
David Barth (community) Approve
Review via email: mp+277796@code.launchpad.net

Commit message

Don't handle window closing more than once.

Description of the change

Don't handle window closing more than once.

To post a comment you must log in.
Revision history for this message
David Barth (dbarth) :
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 'online-accounts-ui/provider-request.cpp'
2--- online-accounts-ui/provider-request.cpp 2015-06-16 14:43:58 +0000
3+++ online-accounts-ui/provider-request.cpp 2015-11-18 09:41:25 +0000
4@@ -164,6 +164,8 @@
5 {
6 Q_Q(ProviderRequest);
7
8+ QObject::disconnect(m_view, SIGNAL(visibleChanged(bool)),
9+ this, SLOT(onWindowVisibleChanged(bool)));
10 if (!visible) {
11 q->setResult(QVariantMap());
12 }

Subscribers

People subscribed via source and target branches