Merge lp:~online-accounts/signon-plugin-oauth2/packaging into lp:signon-plugin-oauth2/rtm-14.09

Proposed by David Barth
Status: Merged
Merged at revision: 66
Proposed branch: lp:~online-accounts/signon-plugin-oauth2/packaging
Merge into: lp:signon-plugin-oauth2/rtm-14.09
Diff against target: 23 lines (+5/-1)
1 file modified
src/plugin.cpp (+5/-1)
To merge this branch: bzr merge lp:~online-accounts/signon-plugin-oauth2/packaging
Reviewer Review Type Date Requested Status
David Barth (community) Approve
Review via email: mp+240001@code.launchpad.net

Commit message

RTM 14.09 backport of the fix for Bug #1377181: Account creation fails repeatedly, then it works

Description of the change

RTM 14.09 backport of the fix for Bug #1377181: Account creation fails repeatedly, then it works

To post a comment you must log in.
Revision history for this message
David Barth (dbarth) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/plugin.cpp'
2--- src/plugin.cpp 2012-07-20 05:46:13 +0000
3+++ src/plugin.cpp 2014-10-29 14:26:58 +0000
4@@ -39,7 +39,7 @@
5 Plugin::Plugin(QObject *parent):
6 AuthPluginInterface(parent),
7 impl(0),
8- m_networkAccessManager(new QNetworkAccessManager(this))
9+ m_networkAccessManager(0)
10 {
11 TRACE();
12 }
13@@ -75,6 +75,10 @@
14 {
15 if (impl != 0) delete impl;
16
17+ if (!m_networkAccessManager) {
18+ m_networkAccessManager = new QNetworkAccessManager(this);
19+ }
20+
21 if (OAuth1Plugin::mechanisms().contains(mechanism)) {
22 impl = new OAuth1Plugin(this);
23 } else if (OAuth2Plugin::mechanisms().contains(mechanism)) {

Subscribers

People subscribed via source and target branches

to all changes: