In the current formsapi model, directly using drupal_goto on your _submit is wrong; instead, return the location to go to.

This turns out to be a problem for me since form_alter can't accept additional data in other _submit functions because that drupal_goto effectively ends execution.

CommentFileSizeAuthor
submit_goto_0.patch2.4 KBmerlinofchaos

Comments

merlinofchaos’s picture

merlinofchaos commented

In case you're wondering why I care about form_alter:

http://cvs.drupal.org/viewcvs/*checkout*/drupal/contributions/modules/na...

jvandyk’s picture

jvandyk commented
Status: Needs review » Fixed

Committed. Thanks.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)