Merge lp:~ken-vandine/account-plugins/credentialsId into lp:account-plugins

Proposed by Ken VanDine
Status: Merged
Approved by: David King
Approved revision: 90
Merged at revision: 90
Proposed branch: lp:~ken-vandine/account-plugins/credentialsId
Merge into: lp:account-plugins
Diff against target: 26 lines (+9/-1)
2 files modified
debian/changelog (+8/-0)
tools/account-console (+1/-1)
To merge this branch: bzr merge lp:~ken-vandine/account-plugins/credentialsId
Reviewer Review Type Date Requested Status
David King (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+147410@code.launchpad.net

Commit message

account-console: fixed a bug where it was failing to store the credentialsId because it was using the wrong type.

Description of the change

When using account-console to create an account, it was failing to store the CredentialsId because it was using the wrong type. This branch uses set_variant and a Variant.

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
David King (amigadave) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2012-12-10 21:44:24 +0000
+++ debian/changelog 2013-02-08 16:12:43 +0000
@@ -1,3 +1,11 @@
1account-plugins (0.10bzr13.02.08-0ubuntu1) UNRELEASED; urgency=low
2
3 [ Ken VanDine ]
4 * account-console: fixed a bug where it was failing to store the
5 credentialsId because it was using the wrong type.
6
7 -- Ken VanDine <ken.vandine@canonical.com> Fri, 08 Feb 2013 11:06:35 -0500
8
1account-plugins (0.10bzr12.12.10-0ubuntu1) raring; urgency=low9account-plugins (0.10bzr12.12.10-0ubuntu1) raring; urgency=low
210
3 [ Ken VanDine ]11 [ Ken VanDine ]
412
=== modified file 'tools/account-console'
--- tools/account-console 2012-11-23 10:43:30 +0000
+++ tools/account-console 2013-02-08 16:12:43 +0000
@@ -124,7 +124,7 @@
124124
125125
126 def on_credentials_stored(self, identity, id, error, account):126 def on_credentials_stored(self, identity, id, error, account):
127 account.set_value(self.args.signon_id_field, int(id))127 account.set_variant(self.args.signon_id_field, GLib.Variant('u', id))
128 account.store(self.on_account_stored, None)128 account.store(self.on_account_stored, None)
129129
130130

Subscribers

People subscribed via source and target branches