Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
The OAuth Consumer add/edit form does a drupal_goto at the end of the form submit function. This should actually use $form_state['redirect'] instead, per drupal_redirect_form. I've attached a patch to resolve this issue.
Without the patch, it isn't possible to prevent redirection by any of the methods outlined in drupal_redirect_form.
Comment | File | Size | Author |
---|---|---|---|
#1 | oauth_consumer_drupal_goto.patch | 868 bytes | omegamonk |
Comments
Comment #1
omegamonk CreditAttribution: omegamonk commentedI just noticed that the delete form had the same issue, so I updated that, as well. Also, the original path listed in the drupal_goto, was actually incorrect, so the redirect wasn't working at all. It was previously pointing to 'user/%d/oauth/consumers', when it should be 'user/%d/oauth/consumer'. Anyway, attached is the updated patch.
Comment #2
omegamonk CreditAttribution: omegamonk commentedComment #3
sjerdoLGTM! +1