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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

omegamonk’s picture

I 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.

omegamonk’s picture

sjerdo’s picture

Status: Active » Reviewed & tested by the community

LGTM! +1