This patch resolves a couple different issues:
1) The form is not properly built in the messaging_sendto_js() callback, meaning that form values such as node are not available. I copied the code at http://drupal.org/node/331941 so hopefully it's correct (or at least more correct).
2) sms_validate_number() is not called correctly. This function returns any error messages so you want to check that it returns nothing, not that it returns TRUE. Also in the next line, there was a typo, $number should be $destination.
| Comment | File | Size | Author |
|---|---|---|---|
| messaging_sendto.patch | 2.25 KB | mfb |
Comments
Comment #1
David Goode commentedHey,
Sorry everybody, I've had finals for the past month so I haven't been working on this project nearly as much as I should have :-( I'm done now though, and I'll review this ASAP.
Thanks,
David
Comment #2
David Goode commentedcommitted