This module works great. Some behaviour notes:
- Super fast repeated clicks work awesome. After all that clicking the page redirects to the destination properly with the results made only once. (Like adding a new comment or page).
- Forward and back buttons work optimal (though maybe not how one might expect). If I fill out a form, completely submit, then hit the back button and return to the form, I can submit the form again. Even though the token hasn't changed in the form. Great! Although this might be considered a down-side (since the user might intentionally submit the same information twice), I think this behavior is perfect
- "Slowly" repeating a submission does not work as I expect. For instance if a user tries to create a new page: Fill in information, click submit, then hit stop on the page to prevent the browser from being redirected. Then the user clicks submit again. S/he is returned to the new page form, with the Drupal message reporting "Your page was created", but is facing an empty form and no idea where their new node went.
Ideally, we'd really just want the last submission made by the user to take effect rather than the first, but that's obviously impractical. If we're lucky enough to have a $_REQUEST['destination'] set then that should be used in the drupal_goto. Now any ideas on how we could at least direct the user to the correct page if they submit the same form twice "slowly" without a destination specified?
Comments
Comment #1
robertdouglass commentedThanks for testing! I can add the destination check to the last case you mentioned, but I'm not yet sure how to handle the "slow" case.