Active
Project:
OAuth 1.0
Version:
6.x-3.0-beta2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
30 Sep 2010 at 20:18 UTC
Updated:
22 Mar 2012 at 21:45 UTC
In the function "oauth_common_authorization_add" it is supposedly being passed two arguments $consumer and $provider. In the body of the function though the variable $consumer is not used, and the variable $account is.
function oauth_common_authorization_add($consumer, $provider) {
$token = new DrupalOAuthToken(user_password(32), user_password(32), array(
'provider_token' => $provider,
'uid' => $account->uid,
));
return drupal_get_form('oauth_common_form_authorization', $token);
}
It seems as if that might be a problem.
Comments
Comment #1
voxpelli commentedSeems like the problem is bigger - both the menu callbacks 'user/%user/oauth/consumer/%oauth_common_consumer/delete' and 'user/%user/oauth/consumer/%oauth_common_consumer/add-authorization' seems to not get added to the menu and thus this function is pretty hard to find and use - that's probably why this error haven't been discovered.
Comment #2
voxpelli commentedWon't have time for this - deassigning myself to not give a false impression.