Merge lp:~mhall119/wordpress-launchpad-integration/fixes-737338 into lp:wordpress-launchpad-integration

Proposed by Michael Hall
Status: Merged
Merged at revision: 20
Proposed branch: lp:~mhall119/wordpress-launchpad-integration/fixes-737338
Merge into: lp:wordpress-launchpad-integration
Diff against target: 12 lines (+1/-2)
1 file modified
openid-launchpad.php (+1/-2)
To merge this branch: bzr merge lp:~mhall119/wordpress-launchpad-integration/fixes-737338
Reviewer Review Type Date Requested Status
Matthew Nuzum (community) Approve
Review via email: mp+69129@code.launchpad.net

Commit message

Convert to using the correct arguments for openid_start_login

Description of the change

Convert to using the correct arguments for openid_start_login

To post a comment you must log in.
Revision history for this message
Matthew Nuzum (newz) wrote :

LOoks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'openid-launchpad.php'
2--- openid-launchpad.php 2010-04-26 13:47:48 +0000
3+++ openid-launchpad.php 2011-07-25 16:48:35 +0000
4@@ -169,8 +169,7 @@
5 function openid_launchpad_wp_authenticate(&$credentials) {
6 if (isset($_REQUEST['launchpad'])) {
7 openid_start_login(openid_launchpad_get_server(), 'login',
8- array('redirect_to' => $_REQUEST['redirect_to']),
9- site_url('/wp-login.php', 'login_post'));
10+ $_REQUEST['redirect_to']);
11 }
12 }
13

Subscribers

People subscribed via source and target branches