Merge lp:~cyphermox/ubuntu-system-settings/providePinCode into lp:ubuntu-system-settings

Proposed by Mathieu Trudel-Lapierre
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 940
Merged at revision: 950
Proposed branch: lp:~cyphermox/ubuntu-system-settings/providePinCode
Merge into: lp:ubuntu-system-settings
Diff against target: 31 lines (+9/-6)
1 file modified
plugins/bluetooth/PageComponent.qml (+9/-6)
To merge this branch: bzr merge lp:~cyphermox/ubuntu-system-settings/providePinCode
Reviewer Review Type Date Requested Status
Sebastien Bacher (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+232270@code.launchpad.net

Commit message

Fix PIN code or passcode requests by properly matching the tag for the original request.

Description of the change

Fix PIN code or passcode requests by properly matching the tag for the original request.

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 :

Looks fine to me, thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/bluetooth/PageComponent.qml'
2--- plugins/bluetooth/PageComponent.qml 2014-08-21 13:29:08 +0000
3+++ plugins/bluetooth/PageComponent.qml 2014-08-26 17:17:38 +0000
4@@ -51,19 +51,22 @@
5 Connections {
6 target: backend.agent
7 onPasskeyConfirmationNeeded: {
8+ var request_tag = tag
9 var popup = PopupUtils.open(confirmPasskeyDialog, root, {passkey: passkey, name: device.name})
10- popup.canceled.connect(function() {target.confirmPasskey(tag, false)})
11- popup.confirmed.connect(function() {target.confirmPasskey(tag, true)})
12+ popup.canceled.connect(function() {target.confirmPasskey(request_tag, false)})
13+ popup.confirmed.connect(function() {target.confirmPasskey(request_tag, true)})
14 }
15 onPasskeyNeeded: {
16+ var request_tag = tag
17 var popup = PopupUtils.open(providePasskeyDialog, root, {name: device.name})
18- popup.canceled.connect(function() {target.providePasskey(tag, false, 0)})
19- popup.provided.connect(function(passkey) {target.providePasskey(tag, true, passkey)})
20+ popup.canceled.connect(function() {target.providePasskey(request_tag, false, 0)})
21+ popup.provided.connect(function(passkey) {target.providePasskey(request_tag, true, passkey)})
22 }
23 onPinCodeNeeded: {
24+ var request_tag = tag
25 var popup = PopupUtils.open(providePinCodeDialog, root, {name: device.name})
26- popup.canceled.connect(function() {target.providePinCode(tag, false, "")})
27- popup.provided.connect(function(pinCode) {target.providePinCode(tag, true, pinCode)})
28+ popup.canceled.connect(function() {target.providePinCode(request_tag, false, "")})
29+ popup.provided.connect(function(pinCode) {target.providePinCode(request_tag, true, pinCode)})
30 }
31 }
32

Subscribers

People subscribed via source and target branches