Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Arguments for oauth_common_form_authorization() which are declared as
function oauth_common_form_authorization($form_state, $token) {
should be declared as
function oauth_common_form_authorization($form, &$form_state, $token) {
Comment | File | Size | Author |
---|---|---|---|
#2 | oauth.auth-forms.2.patch | 2.58 KB | sun |
Comments
Comment #1
nevets CreditAttribution: nevets commentedIn addition, the code expects $user so the function should probably be declared as
and the menu item updated to pass $user.
Comment #2
sunIn D7 all form constructors take $form and &$form_state as the first arguments.
Attached patch fixes those authorization forms.
Comment #3
jobeirne CreditAttribution: jobeirne commentedNevets, thanks much for the work. Sun, thank you for the patch. It wouldn't apply cleanly to HEAD, so I investigated, re-rolled it myself, and committed. I've done a second commit containing your cancel link fix and I've attributed it to you.