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
=== modified file 'src/plugin.cpp'
--- src/plugin.cpp 2012-07-20 05:46:13 +0000
+++ src/plugin.cpp 2014-10-29 14:26:58 +0000
@@ -39,7 +39,7 @@
39Plugin::Plugin(QObject *parent):39Plugin::Plugin(QObject *parent):
40 AuthPluginInterface(parent),40 AuthPluginInterface(parent),
41 impl(0),41 impl(0),
42 m_networkAccessManager(new QNetworkAccessManager(this))42 m_networkAccessManager(0)
43{43{
44 TRACE();44 TRACE();
45}45}
@@ -75,6 +75,10 @@
75{75{
76 if (impl != 0) delete impl;76 if (impl != 0) delete impl;
7777
78 if (!m_networkAccessManager) {
79 m_networkAccessManager = new QNetworkAccessManager(this);
80 }
81
78 if (OAuth1Plugin::mechanisms().contains(mechanism)) {82 if (OAuth1Plugin::mechanisms().contains(mechanism)) {
79 impl = new OAuth1Plugin(this);83 impl = new OAuth1Plugin(this);
80 } else if (OAuth2Plugin::mechanisms().contains(mechanism)) {84 } else if (OAuth2Plugin::mechanisms().contains(mechanism)) {

Subscribers

People subscribed via source and target branches

to all changes: