There are three related problems in the _submit hook for the admin/user/notify page.
- The two buttons currently do the same thing. That is, clicking 'save' also invokes the send-now part of the routine
- For save-and-send-now, the settings are saved after, rather than before _notify_send() is invoked, so any new settings will be ignored. If this is really the intended behavior, the button should be named 'send now'.
- After submitting the form, the user is taken to the home page.
The last one may be WAD, but I find it really, really irritating and believe the user should be left on the admin/user/notify page.
This patch fixes all three issues.
| Comment | File | Size | Author |
|---|---|---|---|
| notify-saveandsend.path | 1.29 KB | kthagen |
Comments
Comment #1
RobRoy commentedGood catch. Committed modified version to HEAD and DRUPAL-4-7 since I had moved some stuff around prior. Thanks!
Comment #2
(not verified) commented